Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[assignment#1] 코드리뷰 #1

Open
seojisoosoo opened this issue Mar 29, 2022 · 0 comments
Open

[assignment#1] 코드리뷰 #1

seojisoosoo opened this issue Mar 29, 2022 · 0 comments

Comments

@seojisoosoo
Copy link

세상에 넘 잘했어요!!!!
당신을 최고의 사자로 임명합니다(진지)
스타일을 줄 때, 웬만해서는 css파일로 주는 것이 나중에 코드가 길어질 때, 좀 더 편합니다! 이 점 유의하면 좋을 것 같고, 다양한 태그를 이용하려 한 것 아주 좋아용!

Q1

        <p> 멋쟁이사자처럼 10기로 백엔드 개발과 기획에 관심이 많습니다.</p>

        <p> 중앙대학교 소프트웨어학부 2학년입니다.<br> 23살로 00년생이고 앞으로 잘 부탁드립니다 ! </p>

  • 오 줄바꿈으로 쓸 수 있는 p태그와 br태그를 모두 사용하셨군요! 좋습니다ㅎㅎ통일성 있게 둘 중에 하나만 사용하셔도 좋을 것 같아요!

<h3 style="color: gray;"> 소프트웨어학부 2학년 재학생 </h3>
<img src="rok.png" width="100px" align="left" style="margin-top: 20px; margin-left: 25px; margin-right: 10px;"/>

  • 스타일을 줄 때 어떤 부분은 css파일에 주고, 어떤 부분은 직접 주셨더라고요! 여러 스타일 방식에 접근하신 점 아주 좋습니다! 다만, 앞으로 코드가 더 길어지는 작업을 해야할 때는, 최대한 css파일로 따로 작성해주시면 더 좋을 것같아요!!

Q4
스크린샷 2022-03-29 오후 8 20 31

  • 이미지가 깨졌어요ㅜㅜ 이미지가 파일 안에 넣어져있지 않아서 발생한 문제같습니다! 이미지 삽입할 때는, 꼭 파일 안에 이미지를 넣어주세요!
#card {
    width: 290px;
    height: fit-content;
    border-radius: 25px;
    border-style: solid;
    border-width: 2px;
    background:linear-gradient(rgb(209, 227, 236) 70%, rgb(139, 18, 196) 30%);
}

#country{
    width: fit-content;
    display: inline-block;
    height: fit-content;
    border-radius: 25px;
    border-style: solid;
    border-width: 2px;
    margin-top: 20px;
    background-color: rgb(243, 226, 194);
}
  • width, height, border-radius, border-style, border-width이 동일하게 존재하죠? 이렇게 여러번 사용되는 코드는 id보다는 class로 주는 것이 더 효율적입니다! class의 경우, css에서 #이 아니라 . 을 이용해요!!
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant