[홍지윤] sprint11

[h-zhirun]

요구사항

기본

• 랜딩 페이지의 url path는 루트('/')로 설정합니다. [x]
• title은 "판다마켓"로 설정합니다. [x]
• 화면의 너비가 1920px 이상이면 하늘색 배경색은 너비를 꽉 채우도록 채워지고, 내부 요소들의 위치는 고정되고, 여백만 커지도록 합니다. []
• 화면의 너비가 1920px 보다 작아질 때, "판다마켓" 로고의 왼쪽 여백 200px"로그인" 버튼의 오른쪽 여백 200px이 유지되고, 화면의 너비가 작아질수록 두 요소간 거리가 가까워지도록 설정합니다. []
• 클릭으로 기능이 동작해야 하는 경우, 사용자가 클릭할 수 있는 요소임을 알 수 있도록 CSS 속성 cursor: pointer 로 설정합니다. []
• "판다마켓" 클릭 시 루트 페이지('/')로 이동합니다. [x]
• '로그인'버튼 클릭 시 로그인 페이지('/login')로 이동합니다 (빈 페이지) [x]
• "구경하러가기"버튼 클릭 시('/items')로 이동합니다.(빈 페이지) [x]
• "Privacy Policy", "FAQ"는 클릭 시 각각 Privacy 페이지('/privacy'), FAQ 페이지('/faq')로 이동합니다.(모두 빈 페이지) [x]
• 페이스북, 트위터, 유튜브, 인스타그램 아이콘을 클릭 시 각각의 홈페이지로 새로운 창이 열리면서 이동합니다. [x]

심화

• 사용자의 브라우저가 크고 작아짐에 따라 페이지의 요소간 간격, 요소의 크기, font-size 등 모든 크기와 관련된 값이 크고 작아지도록 설정해 보세요.(설정값은 자유입니다) []

주요 변경사항

• Create main home

스크린샷

멘토에게

• 체크리스트에서 체크 되지 않은 사항에 대해서는 아직 이해도가 너무 부족해서 이해한 부분만 작성하여 냅니다.
• 제가 쓴 코드를 제가 못 알아봐서 다시 만들어야 하나 걱정입니다.
• 코드리뷰 이후에는 수업 진도와 맞게 제출하겠습니다 죄송합니다.

[devym-37]

지윤님
코드 구조는 잘 작성해주신 것 같습니다.
하지만 잘 이해가 되지 않으시는건, 아마도 css의 class네이밍으로만으로도 구조가 보일 수 있으면 좋은데, 현재 네이밍이 first, second 등 구조를 알 수 없는 네이밍이여서이지 않을까 생각됩니다!
해당 네이밍을 수정해보시면 다시 코드를 읽으시더라도 구조가 보이실 것 같습니다.
그리고 커밋도 어떤 작업을 하셨는지 내용으로 작성해서 push해주시면 좋을 것 같습니다!
고생하셨습니다 !!

[devym-37]

보통 img태그 사용시 alt 값을 놓치는 경우가 있는데, 잊지 않고 잘 작성해주셨습니다!

[devym-37]

해당 태그에서는 alt값이 빠져있네요..! 추후에는 잊지 않는 습관을 가지시면 좋을 것 같습니다!

[devym-37]

class 네이밍으로 해당 구조를 알 수 있으면 좋을 것 같습니다. first second third로 나누기 보다 해당 구조를 알 수 있는 네이밍을 고민해보시면 어떨까요 ?

[devym-37]

해당 class네이밍도 위와 동일하게 고려해주시면 좋을 것 같습니다

[devym-37]

머지가 막혀있는 것 같은데 확인해주세요!! ㅎㅎ

[h-zhirun]

찾아봐도 어떻게 해결해야 하는지 모르겠습니다 ㅠㅠ