Skip to content

Conversation

@tansxx
Copy link
Collaborator

@tansxx tansxx commented Apr 18, 2025

요구사항

기본

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

심화

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

주요 변경사항

  • 초기 프로젝트 구조 디렉토리 구성
  • index.html 구현
  • style/reset.css, main.css, variables.css 작성

스크린샷

스크린샷 2025-04-18 오전 11 06 54
스크린샷 2025-04-18 오전 11 07 12
스크린샷 2025-04-18 오전 11 07 23

멘토에게

  • 마지막 카드 밑에 회색 여백이 있는데 저것도 꼭 맞춰서 구현을 해야 하는 건지 구현을 해야 한다면 어떤 식으로 구현을 해야 하는지 궁금합니다
  • 반응형을 일단 main-banner에만 적용을 해 뒀는데 반응형을 어떤 식으로 어떻게 구현을 해야 하는 건지 잘 이해가 안 됩니다.
  • 셀프 코드 리뷰를 통해 질문 이어가겠습니다.

@tansxx tansxx requested a review from dongqui April 18, 2025 02:09
@tansxx tansxx added the 매운맛🔥 뒤는 없습니다. 그냥 필터 없이 말해주세요. 책임은 제가 집니다. label Apr 18, 2025
Copy link
Collaborator

@dongqui dongqui left a comment

Choose a reason for hiding this comment

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

희정님..!
요구 사항도 꼼꼼하게 잘 챙겨주셨고 특별히 리뷰 드릴 거 없이 깔끔하게 잘 해주셨습니다! 시멘틱 태그 조금만 챙겨주시면 더욱 좋을 거 같습니다 💯

마지막 카드 밑에 회색 여백이 있는데 저것도 꼭 맞춰서 구현을 해야 하는 건지 구현을 해야 한다면 어떤 식으로 구현을 해야 하는지 궁금합니다
-> 피그마 기본 프레임에 들어간 배경 색상 같네요. 중요한 부분은 아니라.. 그냥 넘기셔도 될 거 같습니다..!(?) 만약 구현한다면, section 부모에 특별히 스타일 들어가고 있는 게 없어서 간단하게 div하나 추가 하셔도 될 거 같아요 :)

반응형을 일단 main-banner에만 적용을 해 뒀는데 반응형을 어떤 식으로 어떻게 구현을 해야 하는 건지 잘 이해가 안 됩니다.
-> 반응형은 미션 3에 본격적으로 시안과 요구사항이 주어집니다~! 심화 요구 사항 때문이시라면, 유연한 사이즈 단위(rem, %, clamp...)를 사용하라는 의도 같은데, 충분히 잘 하셨으니 넘어가셔도 좋습니다 :)

Copy link
Collaborator

Choose a reason for hiding this comment

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

reset을 사용하셨군요! 👍

Copy link
Collaborator

Choose a reason for hiding this comment

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

공통 스타일을 따로 정리해 주셨네요! 👍

@@ -0,0 +1,140 @@
<!DOCTYPE html>
<html lang="ko">
Copy link
Collaborator

Choose a reason for hiding this comment

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

label을 잘 적용해주셨네요! 👍

<header class="header">
<div class="header-inner">
<a href="/panda-market/" class="logo">
<img src="./assets/images/panda_logo.svg" alt="판다마켓 로고" />
Copy link
Collaborator

Choose a reason for hiding this comment

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

alt를 잘 넣어주셨네요~! 👍

<img src="./assets/images/Img_home_02.svg" alt="상품 검색 이미지" />
</div>
<div class="info-card-text">
<p class="info-card-label text-2lg-bold">Search</p>
Copy link
Collaborator

Choose a reason for hiding this comment

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

P는 문단을 나타내기 위해 사용됩니다! 여기서는 부적절한 거 같네요 🤔

align-items: center;
width: 100%;
max-width: 1200px;
padding: 0 clamp(16px, 6vw, 24px);
Copy link
Collaborator

Choose a reason for hiding this comment

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

clamp를 쓰셨군요!? 😮

max-width: 1200px;
padding: 0 clamp(16px, 6vw, 24px);
margin: 0 auto;
box-sizing: border-box;
Copy link
Collaborator

Choose a reason for hiding this comment

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

보통 box-sizing은 일관성을 위해

* {
  box-sizing: border-box;
}

이런식으로 공통 스타일로 맞추고 시작합니다 :)

white-space: nowrap;
}

.info-card.reverse {
Copy link
Collaborator

Choose a reason for hiding this comment

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

유연하게 잘 처리해 주셨네요! 👍

<a href="https://x.com/?lang=ko" target="_blank">
<img src="./assets/icons/ic_twitter.svg" alt="Twitter" />
</a>
<a
Copy link
Collaborator

Choose a reason for hiding this comment

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

외부 링크를 target="_blank" 속성을 사용하여 새 탭에서 열리도록 설정되어 있습니다. 이 경우 보안상 rel="noopener noreferrer" 속성을 추가하는 것이 좋습니다.
https://velog.io/@parkseonup/noopener%EC%99%80-noreferrer

요즘 현대 브라우저는 기본으로 처리 되긴 하지만.. 참고해 주세요 🤣

</body>

<!-- 푸터 -->
<footer class="footer">
Copy link
Collaborator

Choose a reason for hiding this comment

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

body는 문서의 콘텐츠를 포함하는 부분으로, 모든 시각적 요소와 콘텐츠는 body 내에 있어야 합니다~! footer도 문서의 콘텐츠 중 하나이므로 body 안으로 들어가야 합니다!

@dongqui dongqui merged commit ca45a25 into codeit-bootcamp-frontend:Basic-강희정 Apr 18, 2025
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