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
37 changes: 31 additions & 6 deletions .github/pull_request_template.md
Original file line number Diff line number Diff line change
@@ -1,15 +1,40 @@
## 요구사항

● UI 디자인 기초 토픽을 수강해 보세요.

● 피그마 디자인에 맞게 페이지를 만들어 주세요.

● React와 같은 UI 라이브러리를 사용하지 않고 진행합니다.

● HTML, CSS 파일을 Netlify로 배포해 주세요. (https://gentle-cactus-6c2f2d.netlify.app/)

● PC사이즈만 고려해 주어진 디자인으로 구현합니다.

### 기본

- [x]
- []
- []
- [x] 랜딩 페이지의 url path는 루트('/')로 설정합니다.

- [x] title은 "판다마켓"로 설정합니다.

- [x] 화면의 너비가 1920px 이상이면 하늘색 배경색은 너비를 꽉 채우도록 채워지고, 내부 요소들의 위치는 고정되고, 여백만 커지도록 합니다.

- [x] 화면의 너비가 1920px 보다 작아질 때, "판다마켓" 로고의 왼쪽 여백 200px"로그인" 버튼의 오른쪽 여백 200px이 유지되고, 화면의 너비가 작아질수록 두 요소간 거리가 가까워지도록 설정합니다.

- [x] 클릭으로 기능이 동작해야 하는 경우, 사용자가 클릭할 수 있는 요소임을 알 수 있도록 CSS 속성 cursor: pointer 로 설정합니다.

- [x] "판다마켓" 클릭 시 루트 페이지('/')로 이동합니다.

- [x] '로그인'버튼 클릭 시 로그인 페이지('/login')로 이동합니다 (빈 페이지)

- [x] "구경하러가기"버튼 클릭 시('/items')로 이동합니다.(빈 페이지)

- [x] "Privacy Policy", "FAQ"는 클릭 시 각각 Privacy 페이지('/privacy'), FAQ 페이지('/faq')로 이동합니다.(모두 빈 페이지)

- [x] 페이스북, 트위터, 유튜브, 인스타그램 아이콘을 클릭 시 각각의 홈페이지로 새로운 창이 열리면서 이동합니다.

### 심화

- [x]
- []
- [] 사용자의 브라우저가 크고 작아짐에 따라 페이지의 요소간 간격, 요소의 크기, font-size 등 모든 크기와 관련된 값이 크고 작아지도록 설정해 보세요.(설정값은 자유입니다)

## 주요 변경사항

Expand All @@ -18,7 +43,7 @@

## 스크린샷

![image](이미지url)
![image](/siteImg.png)

## 멘토에게

Expand Down
Binary file added images/bottomImg.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 images/card1.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 images/card2.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 images/card3.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 images/icons/ic_facebook.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 images/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 images/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 images/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 images/logo.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 images/topImg.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
117 changes: 117 additions & 0 deletions index.html
Original file line number Diff line number Diff line change
@@ -0,0 +1,117 @@
<!DOCTYPE html>
<html lang="en">
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라는 의미이니 ko로 변경해주세요!

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

<head>
<meta charset="UTF-8">
<meta name="viewport" content="width=device-width, initial-scale=1.0">
<title>판다마켓</title>
<link rel="stylesheet" href="style.css">
</head>
<body>
<!-- 헤더 -->
<header class="header">
<div class="header-content">
<a href="/">
<img class="logoImg" src="/images/logo.png" alt="logo">
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 라는 의미로, 이미지 파일을 다운로드하는 것에 실패해서 이미지 파일을 보여줄 수 없을 때
어떤 이미지인지 파악할 수 있게 대신 제공되거나, 스크린리더로 읽혀지는 문자를 의미합니다.
따라서 가능한 해당 이미지를 설명하는 내용을 넣어주면 좋습니다~
지금의 경우 해당 이미지가 안 보일때 제공되면 좋을 텍스트는 이미지에 들어간 문자인 "판다마켓"이 되겠네요~

Suggested change
<img class="logoImg" src="/images/logo.png" alt="logo">
<img class="logoImg" src="/images/logo.png" alt="판다마켓">

</a>

<a href="/pages/login.html" class="header-button button">로그인</a>
</div>
</header>
<!-- 상단랜딩 -->
<div class="topLanding">
<div class="top-content">
<div class="top-content-text">
<h2>일상의 모든 물건을<br>거래해 보세요</h2>
<a href="/pages/items.html" class="items-button button">구경하러 가기</a>

</div>

<img class="topLandingImg" src="/images/topImg.png" alt="topImg">

</div>

</div>
<!-- 섹션 -->
<section class="section">

<div class="section-content">
<div class="section-card">
<img class="sectionImg" src="/images/card1.png" alt="sectionImg1">
<div class="section-text card1">
<p class="section-text-top">Hot item</p>
<p class="section-text-middle">인기 상품을<br>확인해 보세요</p>
<p class="section-text-bottom">가장 HOT한 중고거래 물품을<br>판다 마켓에서 확인해 보세요</p>
</div>

</div>
</div>

<div class="section-content">
<div class="section-card">
<div class="section-text card2">
<p class="section-text-top">Search</p>
<p class="section-text-middle">구매를 원하는<br>상품을 검색하세요</p>
<p class="section-text-bottom">구매하고 싶은 물품은 검색해서<br>쉽게 찾아보세요</p>
</div>

<img class="sectionImg" src="/images/card2.png"alt="sectionImg2" >

</div>
</div>

<div class="section-content">
<div class="section-card card3">
<img class="sectionImg" src="/images/card3.png" alt="sectionImg3" >
<div class="section-text">
<p class="section-text-top">Register</p>
<p class="section-text-middle">판매를 원하는<br>상품을 등록하세요</p>
<p class="section-text-bottom">어떤 물건이든 판매하고싶은 상품을<br>쉽게 등록하세요</p>
Comment on lines +66 to +68
Copy link
Collaborator

Choose a reason for hiding this comment

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

💊 제안

classname을 지으실때 위치에 대한 이름은 추천드리지 않습니다.
나중에 디자인이 변해 태그의 순서가 변했을때, 혹은 반응형 사이트라 화면 크기에 따라 태그 위치가 변할 수 있기 때문에
해댱 요소를 잘 설명하지 못하기 때문입니다.
따라서 가능하면 해당 태그의 역할이나 내용을 설명하는 이름을 사용하는 것이 좋습니다.

Suggested change
<p class="section-text-top">Register</p>
<p class="section-text-middle">판매를 원하는<br>상품을 등록하세요</p>
<p class="section-text-bottom">어떤 물건이든 판매하고싶은 상품을<br>쉽게 등록하세요</p>
<p class="section-text-tag">Register</p>
<p class="section-text-title">판매를 원하는<br>상품을 등록하세요</p>
<p class="section-text-subTitle">어떤 물건이든 판매하고싶은 상품을<br>쉽게 등록하세요</p>

이렇게 클래스명을 작성하면 이름을 통해 태그의 역할을 추측할 수 있게되고 위치가 바뀌더라도 코드를 더 잘 이해할 수 있습니다.

</div>

</div>
</div>

</section>
<!-- 하단 랜딩 -->
<div class="bottomLanding">
<div class="bottom-content">
<div class="bottom-content-text">
<h2>믿을 수 있는<br>판다마켓 중고 거래</h2>

</div>

<img class="bottomLandingImg" src="/images/bottomImg.png" alt="bottomImg">

</div>
</div>
<!-- 푸터 -->
<footer class="footer">
<div class="footer-content">

<div class="footer-left-text">
<p>@codeit - 2024</p>
</div>
<div class="footer-middle-text">
<a href="/pages/privacy.html">Privacy Policy</a>
<a href="/pages/faq.html">FAQ</a>
</div>
<div class="footer-icons">
<a href="https://www.facebook.com/" target="_blank">
<img src="/images/icons/ic_facebook.png" alt="facebook">
</a>
<a href="https://x.com/" target="_blank">
<img src="/images/icons/ic_twitter.png" alt="twitter">
</a>
<a href="https://www.youtube.com/" target="_blank">
<img src="/images/icons/ic_youtube.png" alt="youtube">
</a>
<a href="https://www.instagram.com/" target="_blank">
<img src="/images/icons/ic_instagram.png" alt="instagram">
</a>
</div>
Comment on lines +98 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 관련 태그로 작성하시는 것을 추천드립니다.

Suggested change
<div class="footer-icons">
<a href="https://www.facebook.com/" target="_blank">
<img src="/images/icons/ic_facebook.png" alt="facebook">
</a>
<a href="https://x.com/" target="_blank">
<img src="/images/icons/ic_twitter.png" alt="twitter">
</a>
<a href="https://www.youtube.com/" target="_blank">
<img src="/images/icons/ic_youtube.png" alt="youtube">
</a>
<a href="https://www.instagram.com/" target="_blank">
<img src="/images/icons/ic_instagram.png" alt="instagram">
</a>
</div>
<ul class="footer-icons">
<li>
<a href="https://www.facebook.com/" target="_blank">
<img src="/images/icons/ic_facebook.png" alt="facebook">
</a>
</li>
</ul>

</div>

</footer>

</body>
</html>
11 changes: 11 additions & 0 deletions pages/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>
<h2>FAQ</h2>
</body>
</html>
11 changes: 11 additions & 0 deletions pages/items.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>ITEMS</title>
</head>
<body>
<h2>Items</h2>
</body>
</html>
11 changes: 11 additions & 0 deletions pages/login.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>Login</title>
</head>
<body>
<h2>Login</h2>
</body>
</html>
11 changes: 11 additions & 0 deletions pages/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</title>
</head>
<body>
<h2>Privacy</h2>
</body>
</html>
Binary file added siteImg.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Loading