Conversation
There was a problem hiding this comment.
크게 거슬리는 점 없이 잘 하신 것 같습니다. 👍
다음 PR부터는 기본과, 심화의 어떤 항목이 된것인지 같이 나오면 좋을 것 같습니다.
LGTM
질문 답변
화면의 너비가 1920px 이상이면 하늘색 배경색은 너비를 꽉 채우도록 채워지고, 내부 요소들의 위치는 고정되고, 여백만 커지도록 합니다, 화면의 너비가 1920px 보다 작아질 때, “판다마켓” 로고의 왼쪽 여백 200px“로그인" 버튼의 오른쪽 여백 200px이 유지되고, 화면의 너비가 작아질수록 두 요소간 거리가 가까워지도록 설정합니다. 두 조건이 배우는 챕터에 나오는 건 아닌 것 같아서 하지 못했습니다.
반응형에 관해서는 멘토링때 설명드렸고, 이해도 하신 것 같아서 따로 답변은 추가하지않겠습니다.
| <div class="main_title"> | ||
| 일상의 모든 물건을<br /> | ||
| 거래해 보세요 | ||
| </div> |
There was a problem hiding this comment.
text같은 것들은 h, p, span등을 활용해보시면 좋을 것 같습니다.
|
|
||
| <section class="container"> | ||
| <div class="subtitle"> | ||
| <img src="hotitem.png" class="image" 인기 상품" /> |
There was a problem hiding this comment.
img에 alt속성이 비정상적으로 들어간 것들이 몇개 있는 것 같습니다.
| <section class="container"> | ||
| <div class="subtitle"> | ||
| <img src="hotitem.png" class="image" 인기 상품" /> | ||
| <div class="mini_title">Hot item</div> | ||
| <div class="main_subtitle">인기 상품을<br />확인해 보세요</div> | ||
| <div class="explanation"> | ||
| 가장 HOT한 중고거래 물품을<br />판다 마켓에서 확인해 보세요 | ||
| </div> | ||
| </div> | ||
|
|
||
| <div class="container2"> | ||
| <img src="search.png" class="right_img" 인기 상품" /> | ||
| <div class="search">Search</div> | ||
| <div class="main_subtitle_buy"> | ||
| 구매를 원하는<br />상품을 검색하세요 | ||
| </div> | ||
| <div class="explanation_p"> | ||
| 구매하고 싶은 물품은 검색해서<br />쉽게 찾아보세요 | ||
| </div> | ||
| </div> | ||
|
|
||
| <div class="container3"> | ||
| <img src="register.png" class="image" 인기 상품" /> | ||
| <div class="register">Register</div> | ||
| <div class="main_subtitle_sell"> | ||
| 판매를 원하는<br />상품을 등록하세요 | ||
| </div> | ||
| <div class="explanation_p_p"> | ||
| 어떤 물건이든 판매하고 싶은 상품을<br />쉽게 등록하세요 | ||
| </div> | ||
| </div> | ||
| </section> |
There was a problem hiding this comment.
section을 container로 잡았는데 내부 div들이 container2, 3이 되는게 좀 부자연스러워 보입니다.
| <div class="main_subtitle_buy"> | ||
| 구매를 원하는<br />상품을 검색하세요 | ||
| </div> | ||
| <div class="explanation_p"> |
There was a problem hiding this comment.
무조건은 아니지만 저는 최대한 줄임말을 안쓰는쪽으로 하는걸 추천드립니다.
There was a problem hiding this comment.
스타일이 비슷한 요소들도 전부 별도로 분리되어있어서, 중복코드가 많은 것 같습니다.
같은 스타일은 같은 클래스로 묶고, 몇몇 달라지는 부분만 별도 클래스를 적용하는 식으로 해보시면 좋을 것 같습니다.
There was a problem hiding this comment.
전역적으로 사용하는 클래스가 아닌이상 태그 위계에 맞춰서 만들면 좀더 가독성과 클래스 중복에 도움이 됩니다.
main_section_title 같은 느낌으로 작성하시면 도움이 될 것 같습니다.
혹은 logo같은 단일로 사용되는 경우 id를 활용할 수도 있습니다.
요구사항
기본
심화
주요 변경사항
스크린샷
멘토에게