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

Conversation

duckjae12
Copy link
Collaborator

요구사항

기본

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

심화

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

배포주소

https://endearing-stroopwafel-0f2047.netlify.app

스크린샷

![Group19](images/Property 1=sm.png)
img_home_top
Img_home_01
Img_home_02
img_home_03
img_home_bottom

멘토에게

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

@duckjae12 duckjae12 marked this pull request as draft August 10, 2024 14:35
@JaeSang1998 JaeSang1998 marked this pull request as ready for review August 11, 2024 02:46
@JaeSang1998 JaeSang1998 self-requested a review August 11, 2024 02:47
Copy link
Collaborator

@JaeSang1998 JaeSang1998 left a comment

Choose a reason for hiding this comment

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

혁재님 너무 고생하셨습니다!

align-content: center;
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.

한글보다는 영어를 사용해주세요~!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

네 영어로 바꿔보겠습니다

height: 540px;
text-align: center;
}
.Frame2610528 {
Copy link
Collaborator

Choose a reason for hiding this comment

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

css 도 좀 더 의미있는 이름으로 지어주세요~

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

네 이름도 좀 의미있게 지어볼게요

<a href="/login"><button class="btn_small_48">로그인</button></a>
</div>
</div>
<div class="큰틀">
Copy link
Collaborator

Choose a reason for hiding this comment

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

각각 div 로 잘 레이아웃을 분리해주셨군요.
다음번엔 div 뿐만 아니라 section 등의 Semantic Tag 를 사용해 보시면 어떨까요?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

한번 section도 사용해보겠습니다

<p id="상품검색txt">구매를 원하는<br>상품을 검색하세요</p>
<p id="물품검색txt">구매하고 싶은 물품은 검색해서<br>쉽게 찾아보세요</p>
</div>
<img id="Img_home_02" src="images/Img_home_02.png">
Copy link
Collaborator

Choose a reason for hiding this comment

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

절대경로로 이미지를 잘 넣어주셨습니다!

./images 처럼 상대경로로 표현하는 방법도 있으니, 절대 경로, 상대경로의 차이에 대해서 한 번 알아보셔도 좋을 것 같아요!

position: absolute;
top: 50%;
left: 50%;
margin-left: -343px;
Copy link
Collaborator

Choose a reason for hiding this comment

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

margin 을 minus 를 주신 이유가 있을까요?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

figma에서 이미지 위치가 가운데가 아니라 가운데보다 좀 옆에 있길래 넣어봤습니다..

display: flex;
box-sizing: border-box;
}
#text5 {
Copy link
Collaborator

Choose a reason for hiding this comment

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

전체적으로 동일한 피드백인데 text5 보다는 ~~_text 처럼 어떤 텍스트인지 좀 더 이름을 잘 지어주시면 좋습니다 :)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

넵! 의미있는 이름..!

@@ -0,0 +1,279 @@
html, body {
color: #374151;
Copy link
Collaborator

Choose a reason for hiding this comment

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

color 를 variable 로 설정하는 방법도 한 번 알아보시면 좋을 것 같습니다!

--color-var: #374151

이런식으로 등록하면

var(--color-var) 요렇게 사용할 수 있습니다!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

참고해보겠습니다

@JaeSang1998 JaeSang1998 merged commit ea31c5b into codeit-bootcamp-frontend:Basic-오혁재 Aug 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants