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

Conversation

sjiwon02
Copy link
Collaborator

@sjiwon02 sjiwon02 commented Apr 26, 2024

요구사항

기본

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

심화

  • [] palette에 있는 color값들을 css 변수로 등록하고 사용해 주세요.
  • [] 사용자의 브라우저가 크고 작아짐에 따라 페이지의 요소간 간격, 요소의 크기, font-size 등 모든 크기와 관련된 값이 크고 작아지도록 설정해 보세요.(설정값은 자유입니다)

주요 변경사항

스크린샷

image

멘토에게

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

@sjiwon02 sjiwon02 requested a review from jlstgt April 26, 2024 05:59
@sjiwon02 sjiwon02 added 매운맛🔥 뒤는 없습니다. 그냥 필터 없이 말해주세요. 책임은 제가 집니다. 미완성🫠 죄송합니다.. labels Apr 26, 2024
Copy link
Collaborator

@jlstgt jlstgt left a comment

Choose a reason for hiding this comment

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

처음 미션인데 잘 작성해주신 것 같습니다. 제가 리뷰에 중요 라고 남겨뒀던 부분을 특히 더 신경써서 봐주시면 좋을 것 같습니다. 고생 많으셨습니다! 👍

<body>
<header>
<a href="/"><img class="logo" src="images/logo.svg"></a>
<a href="/login" class="login" type="button" value="로그인">로그인</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><button></button></a>와 같이 사용하시는 게 더 좋습니다!

</head>
<body>
<header>
<a href="/"><img class="logo" src="images/logo.svg"></a>
Copy link
Collaborator

Choose a reason for hiding this comment

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

img 태그와 같이 닫는 태그가 별도로 없는 태그는 끝 부분을 <img class="logo" src="images/logo.svg" />와 같이 />로 처리해주셔야 합니다.

</div>
</nav>
<main>
<div class="mainfirst">
Copy link
Collaborator

Choose a reason for hiding this comment

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

정말 사소한 부분이지만, 클래스명에 두 단어가 있다면 -로 구분하는 게 좋습니다. ex) main-first

일상의 모든 물건을<br>
거래해 보세요
</h2>
<a href="/items" class="view" type="button" value="구경하러가기">구경하러 가기</a>
Copy link
Collaborator

Choose a reason for hiding this comment

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

여기도 <a><button></button></a>이 되어야 합니다.

<nav>
<div class="toppanda">
<h2>
일상의 모든 물건을<br>
Copy link
Collaborator

Choose a reason for hiding this comment

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

멘토링 시간에 말씀드린대로,

<p>12</p>
<p>34</p>

와 같은 방법을 사용해서 줄바꿈하시는 게 좋습니다.

}

.section-description {
position: absolute;
Copy link
Collaborator

Choose a reason for hiding this comment

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

이 부분이 absolute일 필요가 없습니다.
예를 들어, .mainfirst 요소를 가로로 두 개 나눠서(왼쪽, 오른쪽으로) 왼쪽에는 이미지, 오른쪽에는 텍스트를 작성하기만 하면 됩니다.

background-image: url("images/navi.png");
background-repeat: no-repeat;
background-size: 996px 447px;
background-position: 701px 93px;
Copy link
Collaborator

Choose a reason for hiding this comment

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

잘 하셨네요. 👍

}

footer {
width: 1920px;
Copy link
Collaborator

Choose a reason for hiding this comment

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

여기도 고정값일 필요가 없습니다.

}

.lastmain {
width: 1920px;
Copy link
Collaborator

Choose a reason for hiding this comment

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

여기도 마찬가지로 고정값일 필요가 없습니다.

background-repeat: no-repeat;
background-size: 996px 540px;
background-position: 701px;
top: 2298px;
Copy link
Collaborator

Choose a reason for hiding this comment

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

중요

여기 포함 모든 파트에서, top, left 등으로 여백을 조절하시면 안 됩니다.
예를 들어 아래처럼 단순히 여러가지 섹션을 두고,

<section></section>
<section></section>
<section></section>

각 섹션 안에 위 아래 좌 우 padding을 이용해 여백을 조절해야 합니다.

@jlstgt jlstgt merged commit d709629 into codeit-bootcamp-frontend:Basic-신지원 Apr 29, 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