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

Merged

Conversation

gayeong0916
Copy link
Collaborator

@gayeong0916 gayeong0916 commented Apr 26, 2024

요구사항

UI 디자인 기초 토픽을 수강해 보세요.
피그마 디자인에 맞게 페이지를 만들어 주세요.
React와 같은 UI 라이브러리를 사용하지 않고 진행합니다.
HTML, CSS 파일을 Netlify로 배포해 주세요. (참고: https://www.codeit.kr/learn/5309)
PC사이즈만 고려해 주어진 디자인으로 구현합니다.

기본

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

심화

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

주요 변경사항

멘토에게

  • git 사용이 익숙치 않아 제대로 커밋이 된 건지 잘 모르겠습니다!

@gayeong0916 gayeong0916 requested a review from 13akstjq April 27, 2024 01:03
@gayeong0916 gayeong0916 added the 미완성🫠 죄송합니다.. label Apr 27, 2024
@@ -0,0 +1,82 @@
<!DOCTYPE html>
<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에는 문서의 언어를 알 수 있도록 lang 속성이 있답니다~! 확인후 적절한 lang 속성을 추가해주세요!

<body>
<header>
<a href="https://panda-market-sprint.netlify.app/">
<img class="logo" src="Group 19.png">
Copy link
Collaborator

Choose a reason for hiding this comment

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

img를 사용하실 때는 width, height 값을 추가해주어야 합니다.
브라우저 렌더링 시 레이아웃의 크기를 미리 계산할 수 있어서 성능에도 유리합니다.

<body>
<header>
<a href="https://panda-market-sprint.netlify.app/">
<img class="logo" src="Group 19.png">
Copy link
Collaborator

Choose a reason for hiding this comment

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

img 태그의 alt 속성에는 의미 있는 텍스트를 넣어주세요!
스크린 리더기에 읽히는 것을 고려하여 작성해주시면 네이밍하시는데 도움이 되실 거에요.

</head>
<body>
<header>
<a href="https://panda-market-sprint.netlify.app/">
Copy link
Collaborator

Choose a reason for hiding this comment

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

링크 클릭영역이 불필요하게 많이 잡히고 있네요! 로고부분만 클릭되도록 작업하면 좋을 것 같습니다.
image

</div>
</header>
<main>
<div class="main_section">
Copy link
Collaborator

Choose a reason for hiding this comment

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

vscode의 확장프로그램으로 prettier를 설치하셔서 포멧팅을 적용하시면 저장만 해도 구조를 정리해줘서 편리하답니다~!

.article2_reverse{
font-family: Pretendard;
font-size: 40px;
font-weight: 700;
Copy link
Collaborator

Choose a reason for hiding this comment

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

strong태그를 사용했다면 굳이 font-weight:700을 줄 필요가 없겠죠?!

color: #374151;
}
.article3_reverse{
font-family: Pretendard;
Copy link
Collaborator

Choose a reason for hiding this comment

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

Pretendard 같은 웹 폰트를 사용한다면 폴백용 폰트를 추가해주는 것이 좋습니다. 가장 기본적인 폰트가 sans-serif 체 이기 때문에 아래와 같이 Pretendard를 로드하지 못했을 때를 대비해서 작성해주시는 것 이 좋아요.

Suggested change
font-family: Pretendard;
font-family: Pretendard , sans-serif;

display:flex;
gap:30px;
}
#Privacy, #FAQ{
Copy link
Collaborator

Choose a reason for hiding this comment

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

스타일은 클래스를 사용해서만 작업해주세요~!
태그선택자와 id선택자는 사용을 지양해주세요.

}
.container{
width: 1200px;
height: 2160px;
Copy link
Collaborator

Choose a reason for hiding this comment

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

height를 고정값으로 주게 되면 콘텐츠가 가변적인 경우에 레이아우싱 깨지게 됩니다~! 높이는 콘텐츠 양에 따라 자연스럽게 변하도록 제거해주세요!

Copy link
Collaborator

Choose a reason for hiding this comment

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

각 섹션의 간격은 .section에 margin-top 같은 속성으로 간격을 지정하면 될 것 같아요.

<a href="https://www.facebook.com/" id="facebook" target="_balnk"><img src="ic_facebook.png"></a>
<a href="https://www.twitter.com/" id="twitter" target="_balnk"><img src="ic_twitter.png"></a>
<a href="https://www.youtube.com/" id="youtube" target="_balnk"><img src="ic_youtube.png"></a>
<a href="https://www.instagram.com/" id="insta" target="_balnk"><img src="ic_instagram.png"></a>
Copy link
Collaborator

Choose a reason for hiding this comment

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

멘토링 시간에 말씀드린 것처럼 alt 속성으로 어떤 아이콘인지 적혀있으면 좋을 것 같습니다.

@13akstjq
Copy link
Collaborator

가영님 미션 하시느라 고생하셨습니다.
마크업이 익숙치 않으셔서 구조가 복잡하게 작업된 부분들이 있는데, 멘토링 시간에 효율적으로 작업하는 방법에 대해서 이야기 나눠보도록 할게요!
중복되는 코멘트들은 한 곳에만 남겨두었기 때문에 코멘트 남겨드린 부분 외에도 비슷한 수정사항들은 한번 확인하셔서 반영해주세요!

그리고 브랜치는 주차별로 다른 브랜치를 만들어 작업하셔야하니 다음부터는 Basic-김가영-sprint2 같이 -sprint{주차} 를 붙혀서 브랜치를 만들어주세요~!

@13akstjq 13akstjq merged commit 8e15c29 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