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

[박지민] sprint3 #47

Conversation

jiminp00
Copy link
Collaborator

요구사항

기본

  • 브라우저에 현재 보이는 화면의 영역(viewport) 너비를 기준으로 분기되는 반응형 디자인을 적용합니다.

  • Tablet 사이즈로 작아질 때 “판다마켓” 로고의 왼쪽에 여백 24px, “로그인” 버튼 오른쪽 여백 24px을 유지할 수 있도록 “판다마켓” 로고와 “로그인" 버튼의 간격이 가까워집니다.

  • Mobile 사이즈로 작아질 때 “판다마켓” 로고의 왼쪽에 여백 16px, “로그인” 버튼 오른쪽 여백 16px을 유지할 수 있도록 “판다마켓” 로고와 “로그인" 버튼의 간격이 가까워집니다.

  • 화면 영역이 줄어들면 “Privacy Policy”, “FAQ”, “codeit-2024”이 있는 영역과 SNS 아이콘들이 있는 영역의 간격이 줄어듭니다.

  • Tablet 사이즈에서 내부 디자인은 PC사이즈와 동일합니다.

  • Mobile 사이즈에서 좌우 여백 16px 제외하고 내부 요소들이 너비를 모두 차지합니다.

  • Mobile 사이즈에서 내부 요소들의 너비는 기기의 너비가 커지는 만큼 커지지만 400px을 넘지 않습니다.

심화

  • 페이스북, 카카오톡, 디스코드, 트위터 등 SNS에서 Linkbrary 랜딩 페이지(“/”) 공유 시 좌측 예시와 같은 미리보기를 볼 수 있도록 랜딩 페이지 메타 태그를 설정해 주세요.

주요 변경사항

스크린샷

태블릿 랜딩페이지
tablet_landingpage_1
모바일 랜딩페이지
mobile_landingpage_1
모바일 로그인페이지
mobile_login

멘토에게

  • 스프린트 미션 2에서 주신 피드백을 반영했습니다
  • css 파일을 용도별로 분리했습니다
  • 일부 코드를 개선하거나 반응형을 고려하는 과정에서 html 구조 자체를 변경한것이 있습니다. 코드 안에 주석으로 달아두었습니다.

@jiminp00 jiminp00 self-assigned this Aug 23, 2024
@jiminp00 jiminp00 added the 매운맛🔥 뒤는 없습니다. 그냥 필터 없이 말해주세요. 책임은 제가 집니다. label Aug 23, 2024
Copy link
Collaborator

Choose a reason for hiding this comment

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

PNG -> SVG로 바꾸셨네요. 잘 하셨습니다. 👍

<link rel="stylesheet" href="style.css">
<link
rel="stylesheet"
href="https://cdnjs.cloudflare.com/ajax/libs/normalize/8.0.1/normalize.min.css"
Copy link
Collaborator

Choose a reason for hiding this comment

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

�외부 CDN에 올려져 있는 파일을 직접 가져오셔도 되지만, 이 경우 해당 CDN에 문제가 생겼을 때 대응이 어렵습니다!
가능하면 직접 가져오셔서 사용해 주세요~!

Comment on lines +31 to +32
<p>일상의 모든 물건을</p>
<p>거래해 보세요</p>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Paragraph 요소는 문단 구분을 위해 사용하는데요. 일상의 모든 물건을 거래해 보세요는 하나의 문단으로 보여지네요.
공백 문자열을 넣고 white-space 속성으로 줄바꿈 처리를 하시거나, 정 어려우시다면<br/>을 사용하시는 걸 권장드립니다.

<div class="card-text even">
<h1 class="card-header">Search</h1>
<h2 class="card-title">구매를 원하는<br>상품을 검색하세요</h2>
<h3 class="card-explain">구매하고 싶은 물품은 검색해서<br>쉽게 찾아보세요</h3>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Heading은 제목 요소에 해당하는 요소입니다. 모든 요소를 heading으로 처리하실 필요는 없어요!
해당 영역 내에서 제목에 해당되는 요소만 적용하시고, 나머지는 <p>, <span> 등을 사용하셔도 됩니다.

Copy link
Collaborator

Choose a reason for hiding this comment

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

파일을 분기점으로 나누시면, import 순서에 따라 스타일 적용이 의도하지 않게 적용될 수도 있을 것 같네요. (그리고 관리 차원에서도 어려워 보이고요.)

@hoody-jellybean
Copy link
Collaborator

hoody-jellybean commented Aug 26, 2024

전반적으로 잘 해주셨습니다. 👍

다만 반응형 디버깅 시 모바일 디바이스에 필요한 메타 정보가 누락되어 있어 올바르게 그려지지 않는데요.
다음 문서 참고하셔서 넣어주시면 좋을 것 같습니다.

@hoody-jellybean hoody-jellybean merged commit 507f9ca into codeit-bootcamp-frontend:Basic-박지민 Aug 26, 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