-
Notifications
You must be signed in to change notification settings - Fork 46
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
[한정빈] Sprint1 #34
The head ref may contain hidden characters: "Basic-\uD55C\uC815\uBE48-sprint1"
[한정빈] Sprint1 #34
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.
잘 작성해주셨습니다!.
- em, rem 단위 이외에도 vw, vh, fr 등을 사용하는 것 같습니다. 고정 컨텐츠의 경우 px도 사용하기도 합니다.
- 텍스트 줄바꿈은 css속성으로 구현할 수도 있을 것 같습니다. word-break, white-space 속성이 있을 것 같습니다.
- 최대한 그룹화 하시거나, 보통 위에서 아래 순서대로 읽기 때문에, 위에서부터 속성을 정리해주시면 좋을 것 같습니다.
고생하셨습니다!
<body> | ||
<header> | ||
<a href="/" class="logo"> | ||
<img src="logo.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.
img 태그에서 보통 alt 속성을 놓치기 쉬운데, 작성해주시는 습관을 가져주시면 좋을 것 같습니다!
Search | ||
</p> | ||
<h2 class="card-title"> | ||
구매를 원하는<br>상품을 검색하세요 |
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.
텍스트 줄바꿈은
이 일반적이긴 합니다!
믿을 수 있는<br> | ||
판다마켓 중고 거래 | ||
</h2> | ||
</div> |
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.
해당 div 구조는 꼭 필요할까요 ?!
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.
상단 배너랑 비슷하게 생겨서 복붙해서 쓰다보니 이렇게 된 것 같습니다! 수정하는게 좋을 것 같아서 다음번에 반영하겠습니다
</div> | ||
</footer> | ||
</body> | ||
</html> |
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.
시멘틱 태그 구조로도 구조를 알면 좋지만, css 클래스명을 통해서도 구조를 파악할 수 있다고 생각해요. css 클래스명도 구조 표현되도록 잘 작성해주신 것 같습니다.
img 태그에 alt 속성 잊지말아주시면 좋을 것 같습니다!
.ic-sns img { | ||
width: 20px; | ||
height: 20px; | ||
} |
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.
css 내부에 그래도 주석으로 나눠주신 것은 좋은 것 같습니다
요구사항
기본
심화
주요 변경사항
스크린샷
멘토에게