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

Merged

Conversation

KWSony
Copy link
Collaborator

@KWSony KWSony commented Apr 26, 2024

요구사항

기본 요구사항

  • 피그마 디자인에 맞게 페이지를 만들어 주세요.

  • React와 같은 UI 라이브러리를 사용하지 않고 진행합니다.

  • HTML, CSS 파일을 Netlify로 배포해 주세요.

  • PC사이즈만 고려해 주어진 디자인으로 구현합니다.

체크리스트 [기본]

  • 랜딩 페이지의 url path는 루트(‘/’)로 설정합니다.

  • title은 “판다마켓”로 설정합니다.

  • 화면의 너비가 1920px 이상이면 하늘색 배경색은 너비를 꽉 채우도록 채워지고, 내부 요소들의 위치는 고정되고, 여백만 커지도록 합니다.

  • 화면의 너비가 1920px 보다 작아질 때, “판다마켓” 로고의 왼쪽 여백 200px“로그인" 버튼의 오른쪽 여백 200px이 유지되고, 화면의 너비가 작아질수록 두 요소간 거리가 가까워지도록 설정합니다.

  • 클릭으로 기능이 동작해야 하는 경우, 사용자가 클릭할 수 있는 요소임을 알 수 있도록 CSS 속성 cursor: pointer 로 설정합니다.

  • “판다마켓” 클릭 시 루트 페이지(‘/’)로 이동합니다.

  • '로그인'버튼 클릭 시 로그인 페이지(‘/login’)로 이동합니다 (빈 페이지)

  • “구경하러가기”버튼 클릭 시(’/items’)로 이동합니다.(빈 페이지)

  • “Privacy Policy”, “FAQ”는 클릭 시 각각 Privacy 페이지(‘/privacy’), FAQ 페이지(‘/faq’)로 이동합니다.(모두 빈 페이지)

  • 페이스북, 트위터, 유튜브, 인스타그램 아이콘을 클릭 시 각각의 홈페이지로 새로운 창이 열리면서 이동합니다.

심화

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

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

스크린샷

sprintmisson7-1

멘토에게

  • 시맨틱 태그 사용과 페이지 구조 설계가 익숙하지 않아 배치가 다소 난잡한듯 합니다..
  • 구글링으로 말씀해주신 rem 관련 내용 담아 최소한의 페이지 크기 반응 구현만 시도해보았습니다.
  • 셀프 코드 리뷰를 통해 질문 이어가겠습니다.

@KWSony KWSony added the 매운맛🔥 뒤는 없습니다. 그냥 필터 없이 말해주세요. 책임은 제가 집니다. label Apr 26, 2024
@KWSony KWSony requested a review from jlstgt April 26, 2024 08:25
@jlstgt jlstgt changed the title [손권우]sprint 7 [손권우] Sprint1 Apr 27, 2024
Copy link
Collaborator

@jlstgt jlstgt left a comment

Choose a reason for hiding this comment

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

CSS에 많이 익숙해지신 것 같습니다. 그리고 레이아웃도 적절히 잘 구현해주셨습니다.
헤더와 font-size를 62.5%로 지정하는 것과 관련된 얘기는 멘토링 시간에 마저 말씀드리겠습니다.
고생 많으셨습니다! 😊

<div class="virtualbox"></div>
<button type="button" onclick="location.href='login.html'">로그인</button>
</header>
<main>
Copy link
Collaborator

Choose a reason for hiding this comment

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

레이아웃 잘 지정해주셨습니다. 브라우저 가로 넓이가 넓어질 때에도 배경이 안 짤리고 잘 나오네요.
조금 더 보완하자면, 브라우저 가로 크기가 줄어들 때를 대비해서,
image
"일상의 모든 물건을~"이라는 글자 옆에 왼쪽 padding이 조금 들어가 있으면 더 좋을 것 같습니다.

<meta name="viewport" content="width=device-width, initial-scale=1.0" />
<link rel="preconnect" href="https://fonts.googleapis.com" />
<link rel="preconnect" href="https://fonts.gstatic.com" crossorigin />
<link href="https://fonts.googleapis.com/css2?family=Noto+Sans+KR:[email protected]&display=swap" rel="stylesheet" />
Copy link
Collaborator

Choose a reason for hiding this comment

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

피그마 시안에서는 Noto Sans가 아닌 Pretendard로 되어 있습니다. Pretendard로 적용하셔야 할 듯 합니다.

</head>
<body>
<header>
<div>
Copy link
Collaborator

Choose a reason for hiding this comment

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

의미상 버튼이기 때문에 div가 아닌 button 요소를 사용해야 합니다. 그리고 로고를 클릭할 때 메인 페이지로 이동해야하는 조건이 있기 때문에 a 요소로 링크도 걸어주셔야 합니다.

<img src="image\logo.png" alt="로고" />
</div>
<div class="virtualbox"></div>
<button type="button" onclick="location.href='login.html'">로그인</button>
Copy link
Collaborator

Choose a reason for hiding this comment

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

자바스크립트로 링크를 걸지 말고, a를 이용하는 게 좋습니다. 단적인 예로 버튼에 오른쪽 마우스를 클릭해서 "새 탭으로 열기"를 하고 싶을 때가 있는데, 이렇게 자바스크립트로 링크를 걸면 해당 기능이 동작하지 않습니다.

<body>
<header>
<div>
<img src="image\logo.png" alt="로고" />
Copy link
Collaborator

Choose a reason for hiding this comment

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

image/logo.png 처럼 역슬래시가 아닌 슬래시가 일반적인 표현입니다.

@@ -0,0 +1,290 @@
:root {
--color-primary: #3692FF;
Copy link
Collaborator

Choose a reason for hiding this comment

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

👍

line-height: 100%;
}

@media (min-width: 1521px) { *{ font-size: 62.5%; }}
Copy link
Collaborator

Choose a reason for hiding this comment

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

사실 미디어 쿼리 없이 모든 width에서 font-size62.5%로 지정하는 게 이 스프린트 미션의 의도였습니다.
이렇게 62.5%로 지정해야 10px을 1rem으로 적을 수 있기 때문입니다. 멘토링 시간에 더 말씀드리겠습니다.

}

button:hover{
cursor: pointer;
Copy link
Collaborator

Choose a reason for hiding this comment

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

button 요소는 그냥 쓰기만 해도 마우스를 올렸을 때 커서가 바뀌므로 필요 없는 속성입니다.

display: flex;
align-items:center;
justify-content: center;
gap: 8.59px
Copy link
Collaborator

Choose a reason for hiding this comment

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

끝에 세미콜론(;)이 없습니다!

height: 160px;
display: flex;
flex-direction: row;
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.

잘 하셨네요 👍

@jlstgt jlstgt merged commit c7b717e into codeit-bootcamp-frontend:Basic-손권우 Apr 29, 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