Skip to content
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 #8

Merged

Conversation

dang-96
Copy link
Collaborator

@dang-96 dang-96 commented May 24, 2024

요구사항

기본

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

심화

  • palette에 있는 color값들을 css 변수로 등록하고 사용해 주세요.

사이트 링크

스크린샷

스프린트1

멘토에게

  • 로그인, 구경하러 가기 버튼의 텍스트의 밑에 살짝의 여백이 있어서 세로 정렬이 안맞는것 같아서요 혹시 높이 지정할떄 height로 지정하지않고 padding로 지정해서 그런걸까요?

@dang-96 dang-96 requested a review from lisarnjs May 24, 2024 07:26
@dang-96 dang-96 added the 매운맛🔥 뒤는 없습니다. 그냥 필터 없이 말해주세요. 책임은 제가 집니다. label May 24, 2024
Copy link
Collaborator

@lisarnjs lisarnjs left a comment

Choose a reason for hiding this comment

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

스크린샷 2024-05-27 오후 10 20 44

제가 로그인 버튼에 몇 개 코드를 추가해봤던 것인데
차이를 비교해보세요!
일단 기본적으로 a태그 내부의 텍스트라서 아주 살짝 올라가보이는 게 있긴 합니다 ㅜㅜ
하지만 보통은 신경이 쓰이지 않을 정도여서 크게 의미를 두진 않지만
신경쓰인다면 a 태그의 텍스트를 span 태그로 묶고 중앙 정렬 코드를 사용해야겠죠?!

이번 미션도 고생하셨습니다~!


<main>
<div class="main-wrap">
<article class="keyvisual">
Copy link
Collaborator

Choose a reason for hiding this comment

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

제목태그인 h2태그가 내부에 있다면 section 태그를 사용하셔도 의미있을 것 같아요 👍

text-align: right;
}

main .main-wrap .card .card-wrap .card-txt span {
Copy link
Collaborator

Choose a reason for hiding this comment

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

자식 선택자 " > 사용 " 을 더 애용하시는 걸 추천드려요!
나중에 예상치 못한 오류가 발생할 확률을 줄여준답니다 :)

Copy link
Collaborator

Choose a reason for hiding this comment

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

오 역시 ㅎ ㅎ reset.css를 작성하시는군요 👍

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

리뷰와 피드백 감사합니다! 참고하여 코드를 수정해보겠습니다!

@lisarnjs lisarnjs merged commit 9f79ead into codeit-bootcamp-frontend:Basic-염정훈 May 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
매운맛🔥 뒤는 없습니다. 그냥 필터 없이 말해주세요. 책임은 제가 집니다.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants