-
Notifications
You must be signed in to change notification settings - Fork 18
[김가희] sprint1 #1
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
The head ref may contain hidden characters: "Basic-\uAE40\uAC00\uD76C"
[김가희] sprint1 #1
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
멘토링 시간에도 말씀 드렸던 것처럼 시멘틱 태그를 적극적으로 사용해보시면 좋을 것 같네요.
페이지의 푸터 영역에 footer
태그를 사용하신 것처럼 헤더, 메인 콘텐츠, 네비게이션 영역에 각각 적합한 태그를 사용해주시면 더 좋을 것 같습니다! 👍
</div> | ||
<div> | ||
<button class="navbtn"> | ||
<a href="login.html">로그인</a> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<button/>
, <a/>
태그 콘텐츠는 상호작용 가능한 요소는 들어갈 수 없습니다!
다른 페이지로 링크를 따라 이동하시도록 만드는게 목적이면 <a/>
태그만 사용하셔도 됩니다.
자세한 내용은 Stackoverflow의 다음 답변을 참고해 주세요.
<img src="facebook.png"> | ||
<img src="twitter.png"> | ||
<img src="youtube.png"> | ||
<img src="instagram.png"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
푸터의 SNS 아이콘들은 링크 기능이 있는 요소로 보이는데, <a/>
태그 사용 후 <img/>
에 alt 속성으로 대체 텍스트까지 제공해주시면 더 좋을 것 같네요. :)
margin-left: 400px; | ||
margin-right: 400px; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
위클리 과제는 반응형으로 작업하는 것으로 알고 있는데요. 화면이 1000px 정도만 되어도 네비게이션 콘텐츠가 적절하게 렌더링되지 않을 것 같습니다!
border: 1px; | ||
border-color: #DFDFDF; | ||
border-radius: 8px; | ||
font-family: 'Pretendard Variable'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
글꼴은 공통으로 사용하는 경우 :root(=html) 또는 body에서 제공해주셔도 됩니다.
CSS(Cascading Style Sheets)이라는 이름에 맞게 하위 요소로 상속이 되거든요. :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
반응형 작업 시에 필요한 미디어 쿼리 쉽게 보실 수 있는 아티클이 있어서 공유 드립니다. 참고 부탁드려요~!!
.secondimg, | ||
.thirdimg, | ||
.fourthimg { | ||
height: 444px; | ||
} | ||
|
||
.secondimg { | ||
margin: 138px 605px 138px 360px; | ||
width: 955px; | ||
} | ||
|
||
.thirdimg { | ||
margin: 138px 360px 138px 591px; | ||
width: 969px; | ||
} | ||
|
||
.fourthimg { | ||
margin: 138px 573px 138px 360px; | ||
width: 987px; | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
동일한 박스 형태의 이미지면 요소 크기와 높이 등을 지정하는 공통 클래스를 지정해주고,
레이아웃 배치만 별도 클래스로 지정해주시면 좋을 것 같네요. :)
첫 번째 위클리 과제 고생 많으셨습니다! 👍 |
요구사항
기본
심화
주요 변경사항
스크린샷
멘토에게