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

Merged

Conversation

kwonjin2
Copy link
Collaborator

@kwonjin2 kwonjin2 commented Feb 3, 2025

요구사항

  • Github에 PR(Pull Request)을 만들어서 미션을 제출합니다.
  • 피그마 디자인에 맞게 페이지를 만들어 주세요.
  • React와 같은 UI 라이브러리를 사용하지 않고 진행합니다.

기본

  • 브라우저에 현재 보이는 화면의 영역(viewport) 너비를 기준으로 분기되는 반응형 디자인을 적용합니다.
  • Tablet 사이즈로 작아질 때 “판다마켓” 로고의 왼쪽에 여백 24px, “로그인” 버튼 오른쪽 여백 24px을 유지할 수 있도록 “판다마켓” 로고와 “로그인" 버튼의 간격이 가까워집니다.
  • Mobile 사이즈로 작아질 때 “판다마켓” 로고의 왼쪽에 여백 16px, “로그인” 버튼 오른쪽 여백 16px을 유지할 수 있도록 “판다마켓” 로고와 “로그인" 버튼의 간격이 가까워집니다.
  • 화면 영역이 줄어들면 “Privacy Policy”, “FAQ”, “codeit-2024”이 있는 영역과 SNS 아이콘들이 있는 영역의 간격이 줄어듭니다.
  • Tablet 사이즈에서 내부 디자인은 PC사이즈와 동일합니다.
  • Mobile 사이즈에서 좌우 여백 16px 제외하고 내부 요소들이 너비를 모두 차지합니다.
  • Mobile 사이즈에서 내부 요소들의 너비는 기기의 너비가 커지는 만큼 커지지만 400px을 넘지 않습니다.

심화

  • 페이스북, 카카오톡, 디스코드, 트위터 등 SNS에서 Linkbrary 랜딩 페이지(“/”) 공유 시 좌측 예시와 같은 미리보기를 볼 수 있도록 랜딩 페이지 메타 태그를 설정해 주세요.
  • 미리보기에서 제목은 “판다 마켓”, 설명은 “일상의 모든 물건을 거래해보세요”로 설정합니다.
    주소와 이미지는 자유롭게 설정하세요.

주요 변경사항

  • div에 height 높이만큼 여백을 주는 방식 -> margin을 통해 여백을 주는 방식으로 수정했습니다.
  • eyes에 px에 작성했던 소수점 제거 했습니다.
  • 화면이 커지면 이미지가 뭉개지는 현상 수정했습니다.

멘토에게

  • 공통 스타일관리를 어떻게 해야할지, 어떤 자료를 찾으면 저에게 도움이 될지 궁금합니다.

@kwonjin2 kwonjin2 self-assigned this Feb 3, 2025
@kwonjin2 kwonjin2 added the 매운맛🔥 뒤는 없습니다. 그냥 필터 없이 말해주세요. 책임은 제가 집니다. label Feb 3, 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.

권진님 이번 미션도 잘 마무리 해주셨네요 💯

요소 배치하실 때 조금 더 유연한 방법들을 고려해보셔도 좋을 거 같습니다 :) (flex, grid 등)

공통 스타일관리를 어떻게 해야할지, 어떤 자료를 찾으면 저에게 도움이 될지 궁금합니다.
-> 지금은 간단하게만 해보셔도 좋아요! 재사용되는 코드, 같은 목적을 가진 코드를 한 곳에 모아주면 좋습니다! 예를 들면 지금 login.css, signup.css, style.css 파일들에 색상 변수가 각각 정의되어 있는데, 여기에 색상이 추가되면 어떻게 될까요? 모든 css 파일을 찾아다니며 새로운 색상을 추가하시게 될 거에요! 페이지가 늘어나고 css 파일이 늘어날수록 관리가 더욱 힘들어지겠죠. 😢
이 때 공통으로 사용되는 스타일을 관리하는 css파일을 만들고 관리 하시면 유지보수가 훨씬 편해집니다 :)
예를 들면 아래처럼 common.css에는 공통 스타일이 있고 login.css에는 login 관련 스타일만 넣고 사용하는 거죠!

<link rel="stylesheet" href="./css/common.css" />
<link rel="stylesheet" href="./css/login.css" />

지금은 사용 중인 부분에서 공통만 뽑아내셔도 좋습니다! 이후에 reset css, css 유틸리티 클래스를 키워드로 조금 더 검색해 보시면 좋습니다 :)

rel="stylesheet"
/>
</head>
<!DOCTYPE html>
Copy link
Collaborator

Choose a reason for hiding this comment

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

html 태그가 잘못 들어온 거 같네요🤣

@@ -96,13 +110,14 @@ <h2>
<div class="landing-page-last">
<div class="landing-page-text">
<span class="large-text-last">
믿을 수 있는<br />판다마켓 중고 거래
<span class="reliable">믿을 수 있는</span><br />판다마켓 중고 거래
Copy link
Collaborator

Choose a reason for hiding this comment

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

가운데 정렬을 의도하신거라면 부모 태그에서 text-align을 사용하실 수 있어요 :)

margin-top: 231px;
}

#join {
Copy link
Collaborator

Choose a reason for hiding this comment

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

불필요한 코드 같네요! 🤔

margin-top: 80px;
}

.panda-logo-img {
Copy link
Collaborator

Choose a reason for hiding this comment

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

반응형 이미지로 picture를 사용해보셔도 좋아요!

max-width: 400px;
}

.eyes {
Copy link
Collaborator

Choose a reason for hiding this comment

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

image
어떤 이유로 %가 들어갔을까요!? 요구 사항과 다르게 간격이 너무 멀어지는 거 같습니다!

@media (max-width: 766px) {
.panda-logo-a {
display: block;
width: fit-content;
Copy link
Collaborator

Choose a reason for hiding this comment

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

fit-content와 block을 사용하신 이유는 무엇일까요!? 속성 하나하나 짚어가시면 좋습니다 :)

.landing-page-text {
width: auto;
height: 56px;
position: relative;
Copy link
Collaborator

Choose a reason for hiding this comment

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

relative가 자주 쓰이는 거 같은데, 정말 필요한지 생각해보시면 좋을 거 같아요! position을 통해 요소를 옮겨야하는 경우도 물론 있지만, 지금 같은 경우만 봐도 left 400px로 요소를 옮기는 것이 아니라 '가운데'로 옮기는 것이 맞겠죠 :)

image

@dongqui dongqui merged commit 086204f into codeit-bootcamp-frontend:Basic-최권진 Feb 4, 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