Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file added 15-Sprint-Mission/images/home/feature1-image.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file added 15-Sprint-Mission/images/home/feature2-image.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file added 15-Sprint-Mission/images/home/feature3-image.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file added 15-Sprint-Mission/images/home/hero-image.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file added 15-Sprint-Mission/images/logo/favicon.ico
Copy link
Collaborator

Choose a reason for hiding this comment

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

👍 칭찬
파비콘 추가하신 것 좋아요~

Binary file not shown.
Copy link
Collaborator

Choose a reason for hiding this comment

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

💊 제안
해당 레포를 통해 panda-market이라는 것을 알고 있으니 logo.png로 하는게 더 명확한 것 같아요~

Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
3 changes: 3 additions & 0 deletions 15-Sprint-Mission/images/social/facebook-logo.svg
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
3 changes: 3 additions & 0 deletions 15-Sprint-Mission/images/social/instagram-logo.svg
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
3 changes: 3 additions & 0 deletions 15-Sprint-Mission/images/social/twitter-logo.svg
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
10 changes: 10 additions & 0 deletions 15-Sprint-Mission/images/social/youtube-logo.svg
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
114 changes: 114 additions & 0 deletions 15-Sprint-Mission/index.html
Original file line number Diff line number Diff line change
@@ -0,0 +1,114 @@
<!DOCTYPE html>
<html lang="ko">
<head>
<meta charset="UTF-8">
<meta name="viewport" content="width=device-width, initial-scale=1.0">
<title>판다마켓</title>
<link rel="icon" href="images/logo/favicon.ico">
<link rel="stylesheet" as="style" crossorigin="" href="https://cdn.jsdelivr.net/gh/orioncactus/[email protected]/dist/web/static/pretendard.min.css">

<link rel="stylesheet" href="style.css">
<link rel="stylesheet" href="home.css">
</head>
<body>
<header>

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

<a href="/">
<img src="images/logo/panda-market-logo.png" alt="판다마켓홈" width="153">
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 속성은 alternative 라는 의미로, 이미지 파일을 다운로드하는 것에 실패해서 이미지 파일을 보여줄 수 없을 때
어떤 이미지인지 파악할 수 있게 대신 제공되거나, 스크린리더로 읽혀지는 문자를 의미합니다.
따라서 가능한 해당 이미지를 잘 표현할 수 있는 문자열을 넣어주는 것이 좋습니다.
지금의 경우 "판다마켓"이 적절할 것 같아요~

</a>
<a href="login.html" id="loginLinkButton" class="button">로그인</a>
</header>
<main>
<section id="hero" class="banner">
<div class="wrapper">
<h1>
"일상의 모든 물건을"
<br>
"거래해 보세요"
</h1>
Comment on lines +24 to +28
Copy link
Collaborator

Choose a reason for hiding this comment

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

❗️ 수정요청
h1 태그는 heading 1이라는 의미의 태그입니다. 이러한 h1~h6까지의 태그들은 시멘틱한 작성에 중요한 역할을 합니다.
그중 h1 태그는 제목에 해당하는 친구입니다.
목차나 영화를 생각해보시면 제목은 늘 1개입니다. 그래서 h1태그는 한 페이지에 한번만 사용하는 것이 좋습니다.
다만 이는 접근성을 위한 권고 사항일 뿐 필수사항은 아니지만 지키시는 것이 좋습니다.

mdn

<a href="items.html" class="button pill-button">구경하러 가기</a>
</div>
</section>
<section id="features" class="wrapper">
<div class="feature">

<img src="images/home/feature1-image.png" alt="인기 상품" width="50%">
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 는 단위없는 정수여야합니다~ 위와 같이 작업하고 싶으시다면 css를 통해서 해주세요~
mdn

Suggested change
<img src="images/home/feature1-image.png" alt="인기 상품" width="50%">
<img src="images/home/feature1-image.png" alt="인기 상품">

<div class="feature-content">
<h2 class="feature-tag">Hot item</h2>
<h1>
"인기 상품을"
<br>
"확인해 보세요"
</h1>
<p class="feature-description">
"가장 HOT한 중고거래 물품을"
<br>
"판다마켓에서 확인해 보세요"
</p>
</div>
</div>
<div class="feature">
<div class="feature-content">
<h2 class="feature-tag">Search</h2>
<h1>
"구매를 원하는"
<br>
"상품을 검색하세요"
</h1>
<p class="feature-description">
"구매하고 싶은 물품은 검색해서"
<br>
"쉽게 찾아보세요"
</p>
</div>
<img src="images/home/feature2-image.png" alt="검색 기능" width="50%">
</div>
<div class="feature">
<img src="images/home/feature3-image.png" alt="판매 상품 등록" width="50%">
<div class="feature-content">

<h2 class="feature-tag">Register</h2>
<h1>
"판매를 원하는"
<br>
"상품을 등록하세요"
</h1>
<P class="feature-description">
"어떤 물건이든 판매하고 싶은 상품을"
<br>
"쉽게 등록하세요"
</P>
</div>
</div>
</section>
<section id="bottomBanner" class="banner">
<div class="wrapper">
<h1>
"믿을 수 있는"
<br>
"판다마켓 중고거래"
</h1>
</div>
</section>
</main>
<footer>
<div>©codeit - 2024</div>
<div id="footerMenu">

<a href="privacy.html">Privacy Policy</a>
<a href="faq.html">FAQ</a>
</div>
<div id="socialMedia">

<a href="https://www.facebook.com/" target="_blank" rel="noopener noreferrer"><img src="images/social/facebook-logo.svg" alt="페이스북" width="20">
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
<a href="https://www.facebook.com/" target="_blank" rel="noopener noreferrer"><img src="images/social/facebook-logo.svg" alt="페이스북" width="20">
<a href="https://www.facebook.com/" target="_blank" rel="noopener noreferrer"><img src="images/social/facebook-logo.svg" alt="페이스북" width="20" /></a>


<a href="https://twitter.com/" target="_blank" rel="noopener noreferrer"><img src="images/social/twitter-logo.svg" alt="트위터" width="20">

<a href="https://www.youtube.com/" target="_blank" rel="noopener noreferrer"><img src="images/social/youtube-logo.svg" alt="유튜브" width="20">

<a href="https://www.instagram.com/" target="_blank" rel="noopener noreferrer"><img src="images/social/instagram-logo.svg" alt="인스타그램" width="20">

</div>
Comment on lines +101 to +111
Copy link
Collaborator

Choose a reason for hiding this comment

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

💊 제안
이 영역은 로고들의 모음이라고 할 수 있습니다. 관련된 요소들끼리 묶인 목록이니 의미록적인 마크업을 작성한다면 list 관련 태그로 작성하시는 것을 추천드립니다.

<ul id="socialMedia">
  <li>
    <a href="https://www.facebook.com/" target="_blank" rel="noopener noreferrer"><img src="images/social/facebook-logo.svg" alt="페이스북" width="20"></a>
  </li>
</ul>

</footer>
</body>
</html>
162 changes: 162 additions & 0 deletions 15-Sprint-Mission/style.css
Copy link
Collaborator

Choose a reason for hiding this comment

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

💊 제안
브라우저에서는 진형님이 css를 작성해주신 것처럼 자체적으로 style을 가지고 있습니다.
이는 검사 창에서 사용자 에이전트 스타일시트라는 항목으로 확인하실 수 있습니다.
이러한 스타일은 브라우저 별로 다를 수 있기 때문에 밑바탕을 만들듯 이를 초기화하는 과정이 필요합니다.

스크린샷 2025-02-24 오후 1 49 59

목적에 따라 구분해 두는 것이 나중에 찾기 편하기 때문에 css 또한 구분해서 파일을 관리하는 것을 추천드립니다.
예를 들어 위의 브라우저 스타일 시트 초기화 관련 css들은 reset.css 혹은 normalize.css 라는 파일로 관리를 합니다.
구글에 reset.css, normalize.css로 검색해보시면 미리 작성된 css 파일이 있으니 확인해보시고 추가해보세요~

이렇게 하시면 브라우저 기본 스타일 초기화를 위해 반복적으로 css를 작성하실 필요가 없어집니다~

Original file line number Diff line number Diff line change
@@ -0,0 +1,162 @@
header {
width: 100%;
height: 70px;
display: flex;
justify-content: space-between;
align-items: center;
padding: 0 200px;
background-color: #ffffff;
border-bottom: 1px solid #dfdfdf;
}

#loginLinkButton {
Copy link
Collaborator

Choose a reason for hiding this comment

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

💊 제안
css selector로 id 사용은 지양하시는 것이 좋습니다. id 선택자의 경우 높은 우선순위를 가지고 있어
추후 스타일을 덮어쓰기 어렵게 만듭니다~
그와 반대로 태그는 쉽게 스타일이 덮어 씌워집니다.
따라서 overriding 되기 쉽게 하고 싶으시면 태그 선택자를 그 외에는 클래스명을 이용해주세요.
id는 꼭 필요한 경우 사용하시면 됩니다~

font-size: 16px;
font-weight: 600;
border-radius: 8px;
padding: 11.5px 23px;
Copy link
Collaborator

Choose a reason for hiding this comment

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

💬 여담
css에서의 소수점 사용에 대해 다양한 의견이 있습니다.
예전에는 이러한 소수점이 정수로 변환된다는 것이 정론에 가까웠으나, 다양한 해상도가 나오면서 이러한 소수점도 반영된다는 의견이 있습니다.
정확한 명세로 나온것은 없지만, 저는 소수점 사용을 지양하는 편입니다.
만약 같은 화면 크기임에도 해상도에 따라 소수점이 반영되는 것이 갈린다면, 디자인의 일관성이 떨어지는 것이고 소수점 사용은 가독성에 좋지 못하다고 생각하기 때문입니다~
이와 관련해 다양한 논의들이 있으니 참고해보세요!

https://www.reddit.com/r/css/comments/148y3xm/is_it_ok_to_give_text_the_size_in_decimals_like/

}

.button:hover {
Copy link
Collaborator

Choose a reason for hiding this comment

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

❗️ 수정요청
변수나 클래스명, 이미지명과 같은 이름은 해당 파일이 어떤 역할을 하는 친구인지 나타내는 큰 단서가 되고 코드파악 및 유지보수에 중요한 역할을 합니다. 가능하면 유의미한 이름을 사용해주세요~
버튼은 너무 일반 명사라 어떤 요소인지 알기가 어렵습니다.

background-color: #1967d6;
}

* {
margin: 0;
padding: 0;
box-sizing: border-box;
}


#hero {
background-image: url(../images/home/hero-image.png);
}

.banner {
background-color: #cfe5ff;
height: 540px;
display: flex;
align-items: center;
background-repeat: no-repeat;
background-position: 80% bottom;
background-size: 55%;
}

.wrapper {
max-width: 1200px;
margin: 0 auto;
width: 100%;
}

div {
display: block;
}

h1 {
font-size: 40px;
font-weight: 700;
line-height: 56px;
letter-spacing: 0.02em;
}

.banner .pill-button {
margin-top: 32px;
}

.pill-button {
font-size: 20px;
font-weight: 700;
border-radius: 999px;
padding: 16px 124px;
}

.button {
background-color: #3692ff;
color: #ffffff;
display: inline-flex;
align-items: center;
justify-content: center;
}

a. {
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
a. {
a {

text-decoration: none;
}

#features {
padding-bottom: 138px;
}

.feature {
padding: 138px 0;
display: flex;
align-items: center;
gap: 5%;
}

.feature-content {
flex: 1;
}

.feature-tag {
color: #3692ff;
font-size: 18px;
line-height: 25px;
font-weight: 700;
margin-bottom: 12px;
}

h1 {
font-size: 40px;
font-weight: 700;
line-height: 56px;
letter-spacing: 0.02em;
}

.feature-description {
font-size: 24px;
font-weight: 500;
line-height: 120%;
letter-spacing: 0.08em;
margin-top: 24px;
}

.feature:nth-child(2) {
text-align: right;
}

#bottomBanner {
background-image: url(
https://fe-sprint-mission-1.vercel.app/images/home/bottom-banner-image.png
);
}

footer {
background-color: #111827;
color: #9ca3af;
display: flex;
justify-content: space-between;
align-items: center;
padding: 32px 200px 108px 200px;
font-size: 16px;
}

#footerMenu {
display: flex;
gap: 30px;
color: #e5e7eb;
}

a {
text-decoration: none;
color: inherit;
}

#socialMedia {
display: flex;
gap: 12px;
}

body {
color: #374151;
word-break: keep-all;
font-family: "Pretendard", sans-serif;
}
1 change: 1 addition & 0 deletions 15-Sprint-Mission/test.txt
Copy link
Collaborator

Choose a reason for hiding this comment

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

💊 제안
귀여운 텍스트네요~ 다만 repo에는 불필요한 파일이 없는 것이 관리하기 좋아요~ 해당 파일은 삭제하시는 것을 추천드려요!
추후 commit 하실때도 필요한 파일만 포함하시는 것을 추천드릴께요~

Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
미션 수행중 입니다.
Loading