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
Binary file added assets/icons/ic_facebook.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 assets/icons/ic_instagram.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 assets/icons/ic_twitter.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 assets/icons/ic_youtube.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 assets/images/Img_home_01.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 assets/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.

❗️ 수정요청
해당 이미지 파일의 크기를 보면 588× 444로 되어 있습니다.
저희는 반응형 사이트를 만들고 있고, 이런 이미지들은 모바일에서도 사용할 것이기 때문에 추출하실때, 1배수가 아닌 2배수로 추출해주시는 것이 화질면에서 좋습니다.
아래처럼 figma export 탭에서 2x 로 설정하고 추출해주세요~

스크린샷 2025-02-24 오전 9 58 17 스크린샷 2025-02-24 오전 9 59 10

Copy link
Collaborator

Choose a reason for hiding this comment

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

💊 제안
한 레포안에서는 동일한 룰을 따르는 것이 좋습니다.
따라서 다른 이미지 파일명처럼 소문자로 이루어지게 img_home_01.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 assets/images/Img_home_03.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 assets/images/Img_home_bottom.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 assets/images/Img_home_top.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 assets/logos/logo_1.png
Copy link
Collaborator

Choose a reason for hiding this comment

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

❗️ 수정요청
로고 이미지가 여러개가 필요한 이유가 있을까요?
하나의 이미지만 남기시는 것을 추천드립니다~

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 assets/logos/logo_2.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 assets/logos/logo_3.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
181 changes: 181 additions & 0 deletions css/pages/home.css
Copy link
Collaborator

Choose a reason for hiding this comment

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

💊 제안

브라우저는 기본적으로 자체적인 스타일 파일을 가지고 있습니다.
그렇지 않다면 html 파일은 그냥 문서처럼 보여질거에요. 이러한 스타일은 사실 브라우저마다 다르기 때문에
이러한 스타일을 없애 원하는 스타일을 적용하기 위한 밑작업이 필요합니다.
주로 이러한 역할을 하는 css 파일을 reset.css 혹은 normalize.css 라는 이름으로 만들어서 관리합니다.
지금은 필요한 경우 스타일을 overriding 하시면서 작업하셨지만 reset.css 같은 파일을 만드셔서
다른 태그들의 기본 스타일도 초기화하고 작업하시는 것이 예상치 못한 동작을 방지할 수 있어서 추천드려요!

혹시 어떤 내용들이 들어가는지 궁금하시다면 reset.css, normalize.css 를 구글에 검색해보시면 다양한 파일이 나오니
확인해보시고 마음에 드는 것을 참고하시면 좋을 것 같습니다.

Original file line number Diff line number Diff line change
@@ -0,0 +1,181 @@
body {
margin: 0 auto;
font-family: -apple-system, BlinkMacSystemFont, "Apple SD Gothic Neo", "Pretendard Variable", Pretendard, Roboto, "Noto Sans KR", "Segoe UI", "Malgun Gothic", "Apple Color Emoji", "Segoe UI Emoji", "Segoe UI Symbol", sans-serif;
}
Comment on lines +1 to +4
Copy link
Collaborator

Choose a reason for hiding this comment

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

❗️ 수정요청
폰트 패밀리 속성을 왜 이렇게 작성하셨을까요? mdn의 font-family 설명을 보면 해당 속성에 나열된 폰트명들은 순서대로 우선순위를 가집니다.
즉 위의 값에서는 apple의 시스템 폰트가 우선순위가 제일 높다는 의미로 저희가 사용하고자 했던 Pretendard는 4번째의 우선순위로 앞의 폰트들이 없을때 선택되어집니다.

아래처럼 사용하고자 하는 pretendard 폰트의 우선순위를 높여주시는 것을 추천드려요~

Suggested change
body {
margin: 0 auto;
font-family: -apple-system, BlinkMacSystemFont, "Apple SD Gothic Neo", "Pretendard Variable", Pretendard, Roboto, "Noto Sans KR", "Segoe UI", "Malgun Gothic", "Apple Color Emoji", "Segoe UI Emoji", "Segoe UI Symbol", sans-serif;
}
body {
margin: 0 auto;
font-family: "Pretendard Variable", Pretendard, -apple-system, BlinkMacSystemFont, system-ui, Roboto, "Helvetica Neue", "Segoe UI", "Apple SD Gothic Neo", "Noto Sans KR", "Malgun Gothic", "Apple Color Emoji", "Segoe UI Emoji", "Segoe UI Symbol", sans-serif;
}


header {
display: flex;
justify-content: space-between;
align-items: center;
width: 100%;
height: 70px;
}

h2 {
font-size: 40px;
font-weight: 700;
line-height: 56px;
color: #374151;
}

img {
max-width: 100%;
height: auto;
}

.logo {
margin-left: 200px;
}

.logo img {
width: auto;
height: 50px;
}

.login_btn {
background-color: #3692FF;
border: none;
border-radius: 8px;
color: #F3F4F6;
cursor: pointer;
font-size: 16px;
font-weight: 600;
width: 128px;
height: 48px;
margin-right: 200px;
}

.background {
background-color: #CFE5FF;
display: flex;
justify-content: center;
align-items: end;
height: 540px;
}

.background.top {
gap: 7px;
}

.background.bottom {
gap: 69px;
}

.bg_description {
justify-content: center;
align-items: center;
padding-bottom: 60px;
}

.bg_description.top {
margin: 40px 0 40px;
}

.bg_description.bottom {
margin: 112.5px 0 112.5px;
}

.bg_description h2 {
margin: 0 0 32px 0;
}

.items_btn {
background-color: #3692FF;
color: #F3F4F6;
border-radius: 40px;
border: none;
font-size: 20px;
font-weight: 600;
padding: 16px 124px 16px 124px;
cursor: pointer;
}

.main_container {
display: flex;
flex-direction: column;
justify-content: center;
align-items: center;
}


.container {
margin: 138px 0;
width: 988px;
background-color: #FCFCFC;
display: flex;
justify-content: center;
align-items: center;
gap: 64px;
}

.description.first {
width: 298px;
}

.description.second {
text-align: right
}

.description.third {
width: 359px;
}


.description h4 {
color: #3692FF;
font-size: 18px;
font-weight: 700;
line-height: 26px;
margin: 0 0 12px;
}

.description h2 {
margin: 0 0 24px;
}

.description h3 {
font-size: 24px;
font-weight: 500;
line-height: 32px;
color: #374151;
}

footer {
background-color: #111827;
justify-content: center;
height: 160px;
}

.footer_container {
color: #9CA3AF;
display: flex;
justify-content: space-between;
align-items: center;
width: 1120px;
padding: 32px;
margin: 0 auto;
}

.privacy_faq {
display: flex;
gap: 30px;
}

.privacy_faq a{
text-decoration: none;
color: #E5E7EB;
}

a:hover {
cursor: pointer;
}
Comment on lines +169 to +171
Copy link
Collaborator

Choose a reason for hiding this comment

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

❗️ 수정요청
동작 측면에서 차이는 없을 것 같습니다만 css에서 cursor 속성을 pointer 로 해주는 것은 해당 속성이 클릭 가능하다는 것을 의미하기 때문이라, :hover 상태가 아닌 a 자체에 추가하시는 게 더 이해하기 쉬울 것 같습니다.
다만 대부분의 브라우저들은 자체적으로 a 태그에 cursor pointer 속성을 가지고 있어서 동작은 같을 것 같습니다~

Suggested change
a:hover {
cursor: pointer;
}
a {
cursor: pointer;
}
스크린샷 2025-02-26 오후 5 20 14


.icons {
display: flex;
gap: 12px;
}

.icons img {
width: 20px;
height: 20px;
}
11 changes: 11 additions & 0 deletions faq.html
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
<!DOCTYPE html>
<html lang="en">
<head>
<meta charset="UTF-8">
<meta name="viewport" content="width=device-width, initial-scale=1.0">
<title>FAQ</title>
</head>
<body>

</body>
</html>
82 changes: 82 additions & 0 deletions index.html
Original file line number Diff line number Diff line change
@@ -0,0 +1,82 @@
<!DOCTYPE html>
<html lang="kr">
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 속성은 스크린리더, 검색 엔진등에게 해당 사이트가 어떤 언어 기반으로 작성되었는지 설명하는 속성입니다.
해당 속성에는 이미 특정 언어들을 의미하는 예약어가 들어가야 합니다. 예를들어 en은 english 즉 영어를 의미합니다.
kr은 신기하게도 Kanuri 어를 의미합니다. 한국어는 ko를 써야합니다~

https://developer.mozilla.org/ko/docs/Web/HTML/Global_attributes/lang
(해당 페이지 sidebar LockUp 에서 kr과 ko를 검색해보세요~)
https://r12a.github.io/app-subtags/

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

<head>
<meta charset="UTF-8">
<meta name="viewport" content="width=device-width, initial-scale=1.0">
<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="/css/pages/home.css">
<title>판다마켓</title>
</head>
<body>
<header>
<div class="logo">
<img src="/assets/logos/logo_1.png" alt="판다마켓 로고">
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 라는 의미로, 이미지 파일을 다운로드하는 것에 실패해서 이미지 파일을 보여줄 수 없을 때
어떤 이미지인지 파악할 수 있게 대신 제공되거나, 스크린리더로 읽혀지는 문자를 의미합니다.
따라서 가능한 이미지를 설명할 수 있는 대체 텍스트를 입력해주신 점 좋습니다~

</div>
<button class="login_btn" onclick="location.href='login.html'">로그인</button>
Copy link
Collaborator

Choose a reason for hiding this comment

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

💊 제안

해당 디자인이 버튼처럼 보이지만 기능상 페이지 이동이기에 a 태그 사용을 추천드립니다.
(디자인대로 태그를 작성하시기보다 역할을 보시고 태그를 선택하시는 것을 추천드려요.)
a 태그를 사용하시면 링크이기때문에 웹 접근성에도 좋으며 alt 를 누르고 링크 클릭시 새창으로 열 수도 있어 사용성면에서도 더 좋다고 생각합니당

Suggested change
<button class="login_btn" onclick="location.href='login.html'">로그인</button>
<a class="login_btn" href="/login.html">로그인</a>

</header>
<main>
<section class="background top">
<div class="bg_description top">
<h2>일상의 모든 물건을<br> 거래해보세요</h2>
<button class="items_btn" onclick="location.href='items.html'">구경하러 가기</button>
</div>
<img src="/assets/images/Img_home_top.png" alt="메인 페이지 위쪽 이미지">
</section>
<section class="main_container">
<article class="container">
<img src="/assets/images/Img_home_01.png" alt="인기 상품 확인 이미지">
<div class="description first">
<h4>Hot item</h4>
<h2>인기 상품을<br> 확인 해보세요</h2>
<h3>가장 HOT한 중고거래 물품을<br> 판다마켓에서 확인해보세요</h3>
</div>
</article>
<article class="container">
<div class="description asecond">
<h4>Search</h4>
<h2>구매를 원하는<br> 상품을 검색하세요</h2>
<h3>구매하고 싶은 물품은 검색해서<br> 쉽게 찾아보세요</h3>
</div>
<img src="/assets/images/Img_home_02.png" alt="물품 검색 이미지">
</article>
<article class="container">
<img src="/assets/images/Img_home_03.png" alt="물품 판매 이미지">
<div class="description third">
<h4>Register</h4>
<h2>판매를 원하는<br> 상품을 등록하세요</h2>
<h3>어떤 물건이든 판매하고 싶은 상품을<br> 쉽게 등록하세요</h3>
</div>
</article>
</section>
<section class="background bottom">
<div class="bg_description bottom">
<h2>믿을 수 있는<br> 판다마켓 중고거래</h2>
</div>
<img src="/assets/images/Img_home_bottom.png" alt="메인 페이지 아래쪽 이미지">
</section>
</main>
<footer>
<div class="footer_container">
<h5>@codeit-2024</h5>
<div class="privacy_faq">
<a href="privacy.html">Privacy Policy</a>
<a href="faq.html">FAQ</a>
</div>
<div class="icons">
<a href="https://www.facebook.com/">
<img src="/assets/icons/ic_facebook.png" alt="페이스북 아이콘">
</a>
<a href="https://x.com/">
<img src="/assets/icons/ic_twitter.png" alt="트위터 아이콘">
</a>
<a href="https://www.youtube.com/">
<img src="/assets/icons/ic_youtube.png" alt="유튜브 아이콘">
</a>
<a href="https://www.instagram.com/">
<img src="/assets/icons/ic_instagram.png" alt="인스타 아이콘">
</a>
</div>
Comment on lines +65 to +78
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 class="icons">
                <li>
                    <a href="https://www.facebook.com/">
                        <img src="/assets/icons/ic_facebook.png" alt="페이스북 아이콘">
                    </a>
                </li>
            </ul>

</div>
</footer>
</body>
</html>
11 changes: 11 additions & 0 deletions items.html
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
<!DOCTYPE html>
<html lang="kr">
<head>
<meta charset="UTF-8">
<meta name="viewport" content="width=device-width, initial-scale=1.0">
<title>Items Page</title>
</head>
<body>

</body>
</html>
11 changes: 11 additions & 0 deletions login.html
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
<!DOCTYPE html>
<html lang="kr">
<head>
<meta charset="UTF-8">
<meta name="viewport" content="width=device-width, initial-scale=1.0">
<title>Login Page</title>
</head>
<body>

</body>
</html>
11 changes: 11 additions & 0 deletions privacy.html
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
<!DOCTYPE html>
<html lang="en">
<head>
<meta charset="UTF-8">
<meta name="viewport" content="width=device-width, initial-scale=1.0">
<title>Privacy Policy</title>
</head>
<body>

</body>
</html>
Loading