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

Basic 김가희 #54

Conversation

stella-418
Copy link
Collaborator

@stella-418 stella-418 commented Aug 24, 2024

체크리스트 [기본]

공통

  • 브라우저에 현재 보이는 화면의 영역(viewport) 너비를 기준으로 분기되는 반응형 디자인을 적용합니다.
    • PC: 1200px 이상
    • Tablet: 768px 이상 ~ 1199px 이하 / 24px
    • Mobile: 375px 이상 ~ 767px 이하
    • 375px 미만 사이즈의 디자인은 고려하지 않습니다

랜딩 페이지

  • Tablet 사이즈로 작아질 때 “판다마켓” 로고의 왼쪽에 여백 24px, “로그인” 버튼 오른쪽 여백 24px을 유지할 수 있도록 “판다마켓” 로고와 “로그인" 버튼의 간격이 가까워집니다.
  • Mobile 사이즈로 작아질 때 “판다마켓” 로고의 왼쪽에 여백 16px, “로그인” 버튼 오른쪽 여백 16px을 유지할 수 있도록 “판다마켓” 로고와 “로그인" 버튼의 간격이 가까워집니다.
  • 화면 영역이 줄어들면 “Privacy Policy”, “FAQ”, “codeit-2024”이 있는 영역과 SNS 아이콘들이 있는 영역의 간격이 줄어듭니다.

로그인, 회원가입 페이지 공통

  • Tablet 사이즈에서 내부 디자인은 PC사이즈와 동일합니다.
  • Mobile 사이즈에서 좌우 여백 16px 제외하고 내부 요소들이 너비를 모두 차지합니다.
  • Mobile 사이즈에서 내부 요소들의 너비는 기기의 너비가 커지는 만큼 커지지만 400px을 넘지 않습니다.

체크리스트 [심화]

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

주요 변경사항

스크린샷

image

멘토에게

  • 셀프 코드 리뷰를 통해 질문 이어가겠습니다.

@stella-418 stella-418 self-assigned this Aug 24, 2024
@stella-418 stella-418 added the 미완성🫠 죄송합니다.. label Aug 24, 2024
rel="stylesheet"
href="https://cdn.jsdelivr.net/npm/[email protected]/reset.min.css"
/>
<link rel="stylesheet" href="reset.css" />
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 ="/">
<img src="image/logo.svg" class="logotxt"></img>
<img src="image/logo.svg" class="logotxt" alt='판다마켓 글자'></img>
Copy link
Collaborator

Choose a reason for hiding this comment

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

대체 텍스트는 이미지를 보지 못하는 사용자에게 노출되는 텍스트입니다!
실제 사용자 입장에서 의미 있는 텍스트를 넣어주시면 좋을 것 같습니다.

<div class="top-box">
<div class="txtbtn">
<div class="txt-align-box">
<span class="top-text">일상의 모든 물건을&nbsp</span>
Copy link
Collaborator

@hoody-jellybean hoody-jellybean Aug 28, 2024

Choose a reason for hiding this comment

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

&nbsp; 로 넣으셔야 될 것 같은데 혹시 오타인가요?

Comment on lines +66 to +69
<a href="https://www.youtube.com/?hl=ko&gl=KR&app=desktop" target="_blank">
<img src="image/youtube.png">
</a>
<a href="https://www.instagram.com/sem/campaign/emailsignup/?campaign_id=13530338586&extra_1=s%7Cc%7C547419126947%7Ce%7Cinstagram%20c%7C&placement=&creative=547419126947&keyword=instagram%20c&partner_id=googlesem&extra_2=campaignid%3D13530338586%26adgroupid%3D126262419014%26matchtype%3De%26network%3Dg%26source%3Dnotmobile%26search_or_content%3Ds%26device%3Dc%26devicemodel%3D%26adposition%3D%26target%3D%26targetid%3Dkwd-1321618852491%26loc_physical_ms%3D9197620%26loc_interest_ms%3D%26feeditemid%3D%26param1%3D%26param2%3D&gad_source=1&gclid=EAIaIQobChMIzebolvaKiAMV0l4PAh0gcSwWEAAYASAAEgJlLvD_BwE" target="_blank">
Copy link
Collaborator

@hoody-jellybean hoody-jellybean Aug 28, 2024

Choose a reason for hiding this comment

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

유튜브, 인스타그램의 도메인만 넣어주셔도 될 것 같습니다.

사용자의 상태(ex. 사용 언어 등)에 따른 상세 URL 처리는 해당 사이트 내부에서 처리해줄거에요~!

font-size: 100%;
vertical-align: baseline;
}
/* 출처: https://keembloo.tistory.com/5 [keembloo:티스토리] */
Copy link
Collaborator

Choose a reason for hiding this comment

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

티스토리 등 블로그 내용은 참고만 해주시면 좋을 것 같습니다!
이런 자료는 커뮤니티에서 다수의 사용자가 추천한 자료를 사용하시는 걸 더 추천드려요~
(사실 이 블로그도 커뮤니티 내에서 많은 사용자들이 이용하는 reset css를 첨부해놓고 있긴 합니다.)

align-items: center;
}

span {
Copy link
Collaborator

Choose a reason for hiding this comment

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

단순 태그 선택으로 스타일 적용은 지양해 주세요!

body {
width: 100%;
font-family: 'Pretendard Variable';
white-space: nowrap;
Copy link
Collaborator

Choose a reason for hiding this comment

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

white-space: nowrap은 실제 사용이 필요한 곳에서 적용하시는게 좋을 것 같습니다!
줄바꿈이 필요한 영역에서 제대로 적용되지 않을 수 있어요. :)

display: flex;
justify-content: center;
}
.secondimg {
Copy link
Collaborator

@hoody-jellybean hoody-jellybean Aug 28, 2024

Choose a reason for hiding this comment

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

배포된 사이트가 없어 확인이 어려운데
해당 요소의 마진을 지나치게 큰 고정 픽셀 값으로 넣으면 반응형 대응이 안될 것 같습니다!

Copy link
Collaborator

Choose a reason for hiding this comment

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

너비, 높이도 디바이스 너비에 따라 제대로 출력되는지 확인 부탁드려요~

@hoody-jellybean
Copy link
Collaborator

첫 파트 고생 많으셨습니다. 남은 파트도 잘 마무리하시고 화이팅입니다! 👍

@hoody-jellybean hoody-jellybean merged commit 4d5dde6 into codeit-bootcamp-frontend:Basic-김가희 Aug 28, 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