Skip to content

[김가희] sprint1 #1

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

Conversation

stella-418
Copy link
Collaborator

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

요구사항

기본

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

심화

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

주요 변경사항

스크린샷

image

멘토에게

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

@stella-418 stella-418 changed the title Basic 김가희 [김가희] Sprinter1 Aug 10, 2024
@stella-418 stella-418 changed the title [김가희] Sprinter1 [김가희] sprinte1 Aug 10, 2024
@stella-418 stella-418 changed the title [김가희] sprinte1 [김가희] sprint1 Aug 10, 2024
@stella-418 stella-418 self-assigned this Aug 10, 2024
@stella-418 stella-418 added 매운맛🔥 뒤는 없습니다. 그냥 필터 없이 말해주세요. 책임은 제가 집니다. 미완성🫠 죄송합니다.. and removed 매운맛🔥 뒤는 없습니다. 그냥 필터 없이 말해주세요. 책임은 제가 집니다. labels Aug 10, 2024
Copy link
Collaborator

Choose a reason for hiding this comment

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

멘토링 시간에도 말씀 드렸던 것처럼 시멘틱 태그를 적극적으로 사용해보시면 좋을 것 같네요.

페이지의 푸터 영역에 footer 태그를 사용하신 것처럼 헤더, 메인 콘텐츠, 네비게이션 영역에 각각 적합한 태그를 사용해주시면 더 좋을 것 같습니다! 👍

</div>
<div>
<button class="navbtn">
<a href="login.html">로그인</a>
Copy link
Collaborator

Choose a reason for hiding this comment

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

<button/>, <a/> 태그 콘텐츠는 상호작용 가능한 요소는 들어갈 수 없습니다!
다른 페이지로 링크를 따라 이동하시도록 만드는게 목적이면 <a/> 태그만 사용하셔도 됩니다.

자세한 내용은 Stackoverflow의 다음 답변을 참고해 주세요.

Comment on lines +55 to +58
<img src="facebook.png">
<img src="twitter.png">
<img src="youtube.png">
<img src="instagram.png">
Copy link
Collaborator

Choose a reason for hiding this comment

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

푸터의 SNS 아이콘들은 링크 기능이 있는 요소로 보이는데, <a/> 태그 사용 후 <img/>에 alt 속성으로 대체 텍스트까지 제공해주시면 더 좋을 것 같네요. :)

Comment on lines +21 to +22
margin-left: 400px;
margin-right: 400px;
Copy link
Collaborator

Choose a reason for hiding this comment

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

위클리 과제는 반응형으로 작업하는 것으로 알고 있는데요. 화면이 1000px 정도만 되어도 네비게이션 콘텐츠가 적절하게 렌더링되지 않을 것 같습니다!

border: 1px;
border-color: #DFDFDF;
border-radius: 8px;
font-family: 'Pretendard Variable';
Copy link
Collaborator

@hoody-jellybean hoody-jellybean Aug 11, 2024

Choose a reason for hiding this comment

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

글꼴은 공통으로 사용하는 경우 :root(=html) 또는 body에서 제공해주셔도 됩니다.
CSS(Cascading Style Sheets)이라는 이름에 맞게 하위 요소로 상속이 되거든요. :)

Copy link
Collaborator

Choose a reason for hiding this comment

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

반응형 작업 시에 필요한 미디어 쿼리 쉽게 보실 수 있는 아티클이 있어서 공유 드립니다. 참고 부탁드려요~!!

Comment on lines +146 to +165
.secondimg,
.thirdimg,
.fourthimg {
height: 444px;
}

.secondimg {
margin: 138px 605px 138px 360px;
width: 955px;
}

.thirdimg {
margin: 138px 360px 138px 591px;
width: 969px;
}

.fourthimg {
margin: 138px 573px 138px 360px;
width: 987px;
}
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 f975bcf into codeit-bootcamp-frontend:Basic-김가희 Aug 11, 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