Skip to content

Conversation

@asksa1256
Copy link
Collaborator

@asksa1256 asksa1256 commented Apr 2, 2025

요구사항

기본

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

심화

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

주요 변경사항

배포 링크

glowing-chaja-f558c3.netlify.app

스크린샷

screencapture-glowing-chaja-f558c3-netlify-app-2025-04-02-14_04_56

멘토에게

  • 심화 요구사항 구현을 위해 크기, 폰트사이즈 관련 요소들을 모두 rem단위로 설정하고, 미디어 쿼리로 화면 크기가 작아짐에 따라 html에서 font-size를 줄였는데 문제에서 의도한 건 이 방식이 아닌 것 같아서요.. 다른 방법이 있다면 무엇인지 알고 싶습니다!
  • 추천하시는 class 작명법이 있다면 알려주시면 감사하겠습니다.
  • 셀프 코드 리뷰를 통해 질문 이어가겠습니다.

@asksa1256 asksa1256 requested a review from addiescode-sj April 2, 2025 05:11
Copy link
Collaborator

@addiescode-sj addiescode-sj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨습니다! 코멘트 보시고 조금만 다듬어보면 멋진 결과물 나올것같네요 :)

주요 리뷰 포인트

  • 접근성 향상을 위한 속성 추가
  • alt 속성 사용
  • 네이밍케이스, 네이밍 컨벤션

@addiescode-sj
Copy link
Collaborator

질문에 대한 답변

심화 요구사항 구현을 위해 크기, 폰트사이즈 관련 요소들을 모두 rem단위로 설정하고, 미디어 쿼리로 화면 크기가 작아짐에 따라 html에서 font-size를 줄였는데 문제에서 의도한 건 이 방식이 아닌 것 같아서요.. 다른 방법이 있다면 무엇인지 알고 싶습니다!

추천하시는 class 작명법이 있다면 알려주시면 감사하겠습니다.

두개 질문에 대한 답변 모두 본문 내 코멘트에서 피드백 드렸습니다! :)

@addiescode-sj addiescode-sj merged commit 8302966 into codeit-bootcamp-frontend:Basic-이상달 Apr 4, 2025
addiescode-sj pushed a commit that referenced this pull request Apr 25, 2025
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

Successfully merging this pull request may close these issues.

2 participants