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 #7

Conversation

KingsMinn
Copy link
Collaborator

@KingsMinn KingsMinn commented Jan 8, 2025

요구사항

기본

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

심화

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

주요 변경사항

  • 판다마켓 랜딩페이지 생성

스크린샷

멘토에게

  • 어찌저찌 만들긴 했지만 이게 좋은 코드인지는 늘 의문이 듭니다..

@KingsMinn KingsMinn requested a review from dongqui January 8, 2025 05:56
@KingsMinn KingsMinn added the 매운맛🔥 뒤는 없습니다. 그냥 필터 없이 말해주세요. 책임은 제가 집니다. label Jan 8, 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.

승민님 고생 많으셨습니다~!
대체로 잘 구현해주셨고 요구 사항을 좀 더 꼼꼼하게 보시면 더욱 좋을 거 같습니다 :)

어찌저찌 만들긴 했지만 이게 좋은 코드인지는 늘 의문이 듭니다..
-> 클린 코드 자체가 애매하기도 하고 늘 어렵죠 ㅎㅎ 우선은 개념을 확실히 익히는 것에 더 집중하셔도 좋습니다~! :)

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

Choose a reason for hiding this comment

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

<html lang="ko"> lang에 신경써주세요.
SEO, 접근성에 도움이 됩니다!

<body>
<header>
<div class="header-container">
<a href="/" class="logo"><img src="images/logo.svg" alt="판다마켓 로고"></a>
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="images/landing_02.svg" alt="판다마켓 랜딩 이미지">
<div class="texts">
<p class="tag">Search</p>
<h1>구매를 원하는<br>상품을 검색하세요</h1>
Copy link
Collaborator

Choose a reason for hiding this comment

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

h tag 는 계층 구조로 사용하시는 것이 접근성에 좋습니다! h1은 페이지에 하나가 되어야합니다 :)

https://developer.mozilla.org/ko/docs/Web/HTML/Element/Heading_Elements#사용_일람

<section class="land land-normal">
<img src="images/landing_01.svg" alt="판다마켓 랜딩 이미지">
<div class="texts">
<p class="tag">Hot item</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는 문단을 나타내기 위해 사용됩니다! 여기서는 부적절한 거 같네요!

</section>
</main>
<footer>
<div><p style="color: #9CA3AF;">©codeit - 2024</p></div>
Copy link
Collaborator

Choose a reason for hiding this comment

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

inline style은 일관성이 떨어지고 유지 보수에 좋지 않아 지양하시는 것이 좋습니다~! :)

</section>
</main>
<footer>
<div><p style="color: #9CA3AF;">©codeit - 2024</p></div>
Copy link
Collaborator

Choose a reason for hiding this comment

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

여기도 위와 마찬가지로 p tag는 부적절합니다!

Copy link
Collaborator

Choose a reason for hiding this comment

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

벌써 scss를 활용하시는군요! 😮

overflow-x: hidden;
}

header {
Copy link
Collaborator

Choose a reason for hiding this comment

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

지금은 css 파일이 하나라 괜찮지만, 태그에 직접 스타일을 주게 되면 페이지 전체에 영향을 주기 때문에 지양하셔야합니다! 되도록 클래스를 이용해주세요~ :)

<img src="images/landing_top.svg" alt="판다마켓 랜딩 이미지">
<div class="texts">
<h1 style="margin-bottom: 32px;">일상의 모든 물건을<br>거래해 보세요</h1>
<a href="/items" class="button">구경하러 가기</a>
Copy link
Collaborator

Choose a reason for hiding this comment

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

image
해당 버튼은 다시 스타일 작성해보세요~!

피그마에서 제공하는 스타일이 늘 제대로 동작하지는 않습니다 :)

width: 988px;
height: 444px;
margin: 138px 100%;
justify-content: space-between;
Copy link
Collaborator

Choose a reason for hiding this comment

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

image
요소간에 간격을 확인해주세요! space-between은 부적절한 거 같네요!

Copy link
Collaborator

Choose a reason for hiding this comment

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

현재 각각 요소마다 간격이 다릅니다~!
스크린샷 2025-01-08 164033

@dongqui dongqui merged commit f2da460 into codeit-bootcamp-frontend:Basic-김승민 Jan 9, 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