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

Merged

Conversation

ddol9
Copy link
Collaborator

@ddol9 ddol9 commented Apr 26, 2024

요구사항

기본

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

심화

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

멘토에게

@ddol9 ddol9 requested a review from kiJu2 April 26, 2024 08:22
@ddol9 ddol9 self-assigned this Apr 26, 2024
@ddol9 ddol9 added the 매운맛🔥 뒤는 없습니다. 그냥 필터 없이 말해주세요. 책임은 제가 집니다. label Apr 26, 2024
@KingNono1030
Copy link

커밋 단위를 짧게 해서, 리뷰하기 편하네요. 커밋 메시지도 직관적입니다.

@kiJu2 kiJu2 changed the base branch from main to Basic-최예원 April 29, 2024 16:28
@kiJu2
Copy link
Collaborator

kiJu2 commented Apr 29, 2024

커밋 단위를 잘게 나누셨군요 ! 👍

Fix외에는 커밋 타입을 작성하지 않으신 것 같아요 ! 다음 아티클을 읽어보세요 😊:

또한 깃 커밋 메시지 컨벤션도 함께 읽어보세요:

tl;dr:

커밋 메시지 형식

type: Subject

body

footer

기본적으로 3가지 영역(제목, 본문, 꼬리말)으로 나누어졌다.

메시지 type은 아래와 같이 분류된다. 아래와 같이 소문자로 작성한다.

feat : 새로운 기능 추가
fix : 버그 수정
docs : 문서 내용 변경
style : 포맷팅, 세미콜론 누락, 코드 변경이 없는 경우 등
refactor : 코드 리팩토링
test : 테스트 코드 작성
chore : 빌드 수정, 패키지 매니저 설정, 운영 코드 변경이 없는 경우 등

원문보기

Comment on lines +1 to +3
{
"liveServer.settings.port": 5501
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

혹시 해당 setting은 의도된 것일까요? 👀

Copy link
Collaborator

Choose a reason for hiding this comment

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

.gitignore를 작성해보시는게 어떨까요?

macos의 압축 과정에서 흔히 .DS_Store 파일이 들어가는 경우가 있습니다.
해당 파일은 프로젝트와 무관하므로 .gitignore에 다음과 같이 작성하시면 git의 추적을 무시할 수 있습니다.

.DS_Store

@@ -0,0 +1,96 @@
<!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.

lang="ko" 속성을 사용해보는건 어떨까요?

Suggested change
<html>
<html lang="ko">

lang 속성은 Search bot이 탐색할 때 해당 문서가 어떠한 언어로 작성되어있는지 파악할 때 사용될 수 있습니다.
따라서 적절한 lang을 설정하시는걸 권장드립니다. 😊

lang의 기본 값은 '알 수 없음'입니다. 따라서 적절한 값의 lang 속성을 꼭 꼭 ! 설정해주시는걸 권장합니다 !
lang에 대해서 자세히 보기

SEO 마케팅 관련 기사

tl;dr

lang 속성은 다음을 포함하여 다양한 목적으로 사용됩니다.

  • 검색 엔진은 lang 속성을 사용하여 올바른 언어로 웹 페이지를 색인화합니다. 이를 통해 특정 언어로 검색하는 사용자에게 더 관련성이 높은 결과를 반환할 수 있습니다.
  • 화면 판독기는 lang 속성을 사용하여 언어 프로필을 전환하여 올바른 악센트와 발음을 제공합니다. 이는 시각 장애가 있는 사용자가 선호하는 언어로 웹 콘텐츠에 액세스하는 데 도움이 됩니다.
  • lang 속성을 사용하여 번역해야 하는 텍스트를 식별할 수 있습니다. 이는 다국어 웹사이트를 만들거나 사용자 생성 콘텐츠를 번역하는 데 유용할 수 있습니다.

<html> 요소 자체를 포함하여 텍스트 콘텐츠를 포함하는 모든 HTML 요소에 lang 속성을 지정해야 한다는 점에 유의하는 것이 중요합니다. 이렇게 하면 전체 웹페이지의 언어가 올바르게 식별됩니다.
원문 보기

Comment on lines +2 to +7
<html>
<head>
<meta charset="UTF-8">
<title>판다마켓</title>
<link rel="stylesheet" href="style.css">
<script src="https://kit.fontawesome.com/2ea727bd4f.js" crossorigin="2ea727bd4f"></script>
Copy link
Collaborator

Choose a reason for hiding this comment

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

프리티어를 사용해보시는거 어떨까요 ?

Suggested change
<html>
<head>
<meta charset="UTF-8">
<title>판다마켓</title>
<link rel="stylesheet" href="style.css">
<script src="https://kit.fontawesome.com/2ea727bd4f.js" crossorigin="2ea727bd4f"></script>
<html>
<head>
<meta charset="UTF-8">
<title>판다마켓</title>
<link rel="stylesheet" href="style.css">
<script src="https://kit.fontawesome.com/2ea727bd4f.js" crossorigin="2ea727bd4f"></script>

사소한 줄바꿈, 띄어쓰기 등 코드를 작성하시다보면 자연스럽게 불규칙해지는 경우가 많아요.
이를 돕기 위해서 개발 커뮤니티에서는 개발 경험 향상을 위한 여러가지 도구들이 있는데요.

보편적으로 많이 사용되는 툴은 prettier입니다 !

Prettier: https://prettier.io/

prettier는 vscode plugin에서 install하고 실행할 수 있습니다 ! 🤗

macos 경우 option + shift + f
windows의 경우 alt + shift + f

<img src="images/Img_home_01.png" alt="판다마켓 메인1">
<div class="content-box">
<h2 class="content-name">Hot item</h2>
<h1>인기 상품을<br/>확인해 보세요</h1>
Copy link
Collaborator

Choose a reason for hiding this comment

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

<h1> 태그는 문서에 하나만 넣는게 어떨까요?

페이지 당 하나의 <h1>만 사용하세요. 여러 개를 써도 오류는 나지 않겠지만, 단일 <h1>이 모범 사례로 꼽힙니다. 논리적으로 생각했을 때도, <h1>은 가장 중요한 제목이므로 전체 페이지의 목적을 설명해야 할 것입니다. 두 개의 제목을 가진 책이나, 여러 개의 이름을 가진 영화는 볼 수 없죠! 또한 스크린 리더 사용자와 SEO에도 더 적합합니다.

h1에 대해서: MDN

Comment on lines +40 to +60
<div id="right" class="content">
<div class="content-box">
<h2 class="content-name">Search</h2>
<h1>구매를 원하는<br/>상품을 검색하세요</h1>
<p class="content-description">
구매하고 싶은 물품은 검색해서<br/>쉽게 찾아보세요
</p>
</div>
<img src="images/Img_home_02.png" alt="판다마켓 메인2">
</div>

<div class="content">
<img src="images/Img_home_03.png" alt="판다마켓 메인3">
<div class="content-box">
<h2 class="content-name">Register</h2>
<h1>판매를 원하는<br/>상품을 등록하세요</h1>
<p class="content-description">
어떤 물건이든 판매하고 싶은 상품을<br/>쉽게 등록하세요
</p>
</div>
</div>
Copy link
Collaborator

Choose a reason for hiding this comment

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

section을 사용해보는게 어떨까요?

Suggested change
<div id="right" class="content">
<div class="content-box">
<h2 class="content-name">Search</h2>
<h1>구매를 원하는<br/>상품을 검색하세요</h1>
<p class="content-description">
구매하고 싶은 물품은 검색해서<br/>쉽게 찾아보세요
</p>
</div>
<img src="images/Img_home_02.png" alt="판다마켓 메인2">
</div>
<div class="content">
<img src="images/Img_home_03.png" alt="판다마켓 메인3">
<div class="content-box">
<h2 class="content-name">Register</h2>
<h1>판매를 원하는<br/>상품을 등록하세요</h1>
<p class="content-description">
어떤 물건이든 판매하고 싶은 상품을<br/>쉽게 등록하세요
</p>
</div>
</div>
<section id="right" class="content">
<div class="content-box">
<h2 class="content-name">Search</h2>
<h1>구매를 원하는<br/>상품을 검색하세요</h1>
<p class="content-description">
구매하고 싶은 물품은 검색해서<br/>쉽게 찾아보세요
</p>
</div>
<img src="images/Img_home_02.png" alt="판다마켓 메인2">
</section>
<section class="content">
<img src="images/Img_home_03.png" alt="판다마켓 메인3">
<div class="content-box">
<h2 class="content-name">Register</h2>
<h1>판매를 원하는<br/>상품을 등록하세요</h1>
<p class="content-description">
어떤 물건이든 판매하고 싶은 상품을<br/>쉽게 등록하세요
</p>
</div>
</section>

다음은 MDN<section>에 대한 설명 중 첫 문장입니다.

The <section> HTML element represents a generic standalone section of a document, which doesn't have a more specific semantic element to represent it. Sections should always have a heading, with very few exceptions.

<section> HTML 요소는 문서의 일반적인 독립형 섹션을 나타내며 이를 나타내는 더 구체적인 의미 요소가 없습니다. 섹션에는 거의 예외를 제외하고 항상 제목이 있어야 합니다.

<section /> 태그 내에 제목에 해당하는 부분이 있다면 <h> 시리즈로 작성해보세요 😊

Comment on lines +1 to +9
* {
margin: 0;
padding: 0;
box-sizing: border-box;
}

body {
font-family: "pretendard", sans-serif;
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

해당 코드들은 global.css를 만드신 후 옮기셔도 될 것 같습니다 😊

Comment on lines +24 to +26
a {
text-decoration: none;
}
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로 옮기셔도 될 듯 하네요 !:

그대로 사용하셔도 좋고, 참고해도 좋은 Eric meyer의 reset.css는 다음과 같습니다:

/* http://meyerweb.com/eric/tools/css/reset/ 
   v2.0 | 20110126
   License: none (public domain)
*/

html, body, div, span, applet, object, iframe,
h1, h2, h3, h4, h5, h6, p, blockquote, pre,
a, abbr, acronym, address, big, cite, code,
del, dfn, em, img, ins, kbd, q, s, samp,
small, strike, strong, sub, sup, tt, var,
b, u, i, center,
dl, dt, dd, ol, ul, li,
fieldset, form, label, legend,
table, caption, tbody, tfoot, thead, tr, th, td,
article, aside, canvas, details, embed, 
figure, figcaption, footer, header, hgroup, 
menu, nav, output, ruby, section, summary,
time, mark, audio, video {
	margin: 0;
	padding: 0;
	border: 0;
	font-size: 100%;
	font: inherit;
	vertical-align: baseline;
}
/* HTML5 display-role reset for older browsers */
article, aside, details, figcaption, figure, 
footer, header, hgroup, menu, nav, section {
	display: block;
}
body {
	line-height: 1;
}
ol, ul {
	list-style: none;
}
blockquote, q {
	quotes: none;
}
blockquote:before, blockquote:after,
q:before, q:after {
	content: '';
	content: none;
}
table {
	border-collapse: collapse;
	border-spacing: 0;
}

원문 보기

}

.btn {
background-color: #3692ff;
Copy link
Collaborator

Choose a reason for hiding this comment

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

컬러 팔레트를 사용해보시는건 어떨까요?

컬러는 따로 선언해두시고 재사용 해보시는건 어떨까요?
컬러 팔레트는 color.css 혹은 global.css에서 정의하는게 일반적입니다 !
컬러 팔레트는 다음과 같이 정의할 수 있습니다:

* {
  --primary: #6D6AFE;
  --red: #FF5B56;
  --black: #111322;
  --white: #FFFFFF;
  --gray-100: #3E3E43;
  --gray-200: #9FA6B2;
  --gray-300: #CCD5E3;
  --gray-400: #E7EFFB;
  --gray-500: #F0F6FF;
}

@kiJu2
Copy link
Collaborator

kiJu2 commented Apr 29, 2024

굳굳 ! 좋습니다. 예원님.
위클리 미션 수행하시느라 수고 많으셨습니다 ! 다음 코드도 기대가 되는군요 🤗🤗

@kiJu2 kiJu2 merged commit a8b44e0 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.

3 participants