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
35 changes: 35 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,35 @@
## 요구사항

- [] 피그마 디자인에 맞게 페이지를 만들어주세요.
- [] React와 같은 UI 라이브러리를 사용하지 않고 진행합니다.
- [] HTML, CSS 파일을 Netlify로 배포해 주세요. => https://leech-spfe15-panda-market.netlify.app/
- [] PC사이즈만 고려해 주어진 디자인으로 구현합니다.

### 기본

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

### 심화

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

## 주요 변경사항

-

## 스크린샷

![image](![alt text](image.png))

## 멘토에게

-
Empty file added faq.html
Empty file.
Binary file added image.png
Copy link
Collaborator

Choose a reason for hiding this comment

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

❗️ 수정 요청
스크린샷을 코드에 추가하신 이유가 PR에 첨부하려는 목적 때문인 것 같아요~
PR에 이미지를 첨부하려면 해당 repo에 올려서 사용하실 수도 있지만, 이는 오래 유지되어야 하는 이미지에 적합한 방식입니다.
PR에 올리는 이미지는 일시적인 참고용이므로, 굳이 코드에 추가하지 않고 drag & drop으로 첨부하는 것을 추천드립니다!

Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
126 changes: 126 additions & 0 deletions index.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 을 이해하기 쉽게 잘 작성하셨어요!

Original file line number Diff line number Diff line change
@@ -0,0 +1,126 @@
<!DOCTYPE html>
<html lang="ko">
<head>
<meta charset="UTF-8" />
<meta name="viewport" content="width=device-width, initial-scale=1.0" />
<link rel="stylesheet" href="style.css" />
<link
rel="stylesheet"
as="style"
crossorigin
href="https://cdn.jsdelivr.net/gh/orioncactus/[email protected]/dist/web/static/pretendard.min.css"
/>
<title>판다마켓</title>
</head>
<body>
<!-- 헤더 -->
<header class="header">
<div class="header-container">
<a href="/" class="logo">
<img src="public/logos/Property 1=lg.png" alt="판다로고" />
</a>

<a href="login.html" class="login-btn">로그인</a>
</div>
</header>
<!-- 메인 -->
<main>
Comment on lines +26 to +27
Copy link
Collaborator

Choose a reason for hiding this comment

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

💊 제안
주석을 다는 것은 좋은 습관이지만, 가장 좋은 코드는 주석 없이도 이해할 수 있는 코드입니다.
현재 코드 구조상 주석 없이도 충분히 이해할 수 있어 보이는데, 오히려 주석이 추가적인 정보를 전달하지 못하고 가독성을 해치고 있는 것 같아요.
필요한 부분에만 주석을 추가하면 더 좋은 코드가 될 것 같습니다!

<!-- 섹션1 -->
<section class="main-banner">
<div class="main-banner-container">
<div class="main-banner-text">
<h1>일상의 모든 물건을<br />거래해 보세요</h1>
<a href="items.html" class="action-btn">구경하러 가기</a>
</div>
<div class="main-banner-image">
<img src="public/images/Img_home_top.png" alt="메인배너" />
</div>
</div>
</section>

<!-- 섹션2 -->
<section class="hot-item">
<div class="hot-item-container">
<div class="hot-item-image">
<img src="public/images/Img_home_01.png" alt="핫아이템" />
</div>
<div class="hot-item-text">
<span class="item-tag">Hot Item</span>
<h2>인기 상품을<br />확인해 보세요</h2>
<p>
가장 Hot한 중고거래 물품을<br />
판다 마켓에서 확인해 보세요
</p>
</div>
</div>
</section>

<!-- 섹션3 -->
<section class="search">
<div class="search-container">
<div class="search-text">
<span class="search-tag">Search</span>
<h2>구매를 원하는<br />상품을 검색하세요</h2>
<p>구매하고 싶은 물품은 검색해서<br />쉽게 찾아보세요</p>
</div>
<div class="search-image">
<img src="public/images/Img_home_02.png" alt="서치" />
</div>
</div>
</section>

<!-- 섹션 4 -->
<section class="register">
<div class="register-container">
<div class="register-image">
Comment on lines +73 to +75
Copy link
Collaborator

Choose a reason for hiding this comment

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

💊 제안
섹션 2 ~ 4까지를 각각의 classname을 이용해 구분하신 점 좋습니다.
다만 섹션 2 ~ 4 까지가 반복되는 요소가 많은 영역이니 반복을 줄일 수 있는 방법을 고민해보시면 좋을 것 같아요~

아래처럼 공통으로 사용하는 class를 정의해두시고 구분을 위한 class들을 추가해주셔도 좋을 것 같아요!

// section 태그별 공통되는 스타일이 정의된 main-list-container 추가로 반복되는 css 사용을 줄이고 유지보수하기 쉽게 바꿈
<section class="main-list-container hot-item" /> // section2
<section class="main-list-container search" /> // section3
<section class="main-list-container register" /> // section4

<img src="public/images/Img_home_03.png" alt="레지스터" />
</div>
<div class="register-text">
<span class="register-tag">Register</span>
<h2>판매를 원하는<br />상품을 등록하세요</h2>
<p>
어떤 물건이든 판매하고 싶은 상품을<br />
쉽게 등록하세요
</p>
</div>
</div>
</section>

<!-- 섹션 5 -->
<section class="bottom-banner">
<div class="bottom-container">
<h2>믿을 수 있는<br />판다마켓 중고 거래</h2>
<div class="bottom-image">
<img src="public/images/Img_home_bottom.png" alt="하단베너" />
</div>
</div>
</section>
</main>
<!-- 푸터 -->
<footer class="footer">
<div class="footer-container">
<div class="footer-info">
<span>©codeit - 2024</span>
</div>
<div class="footer-links">
<a href="privacy.html">Privacy Policy</a>
<a href="faq.html">FAQ</a>
</div>
<div class="social-links">
<a href="https://facebook.com" target="_blank" class="social-icon">
<img src="public/icons/ic_facebook.png" alt="페이스북" />
</a>
<a href="https://x.com" target="_blank" class="social-icon">
<img src="public/icons/ic_twitter.png" alt="트위터" />
</a>
<a href="https://youtube.com" target="_blank" class="social-icon">
<img src="public/icons/ic_youtube.png" alt="유튜브" />
</a>
<a href="https://instagram.com" target="_blank" class="social-icon">
<img src="public/icons/ic_instagram.png" alt="인스타" />
</a>
</div>
Comment on lines +109 to +122
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="social-links">
  <li>
    <a href="https://facebook.com" target="_blank" class="social-icon">
      <img src="public/icons/ic_facebook.png" alt="페이스북" />
    </a>
  </li>
</ul>

</div>
</footer>
</body>
</html>
Empty file added items.html
Empty file.
Empty file added login.html
Empty file.
Empty file added privacy.html
Empty file.
Binary file added public/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 public/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 public/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 public/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 public/images/Img_home_01.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 public/images/Img_home_02.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 public/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 public/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 public/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 public/logos/Property 1=lg.png
Copy link
Collaborator

Choose a reason for hiding this comment

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

❗️ 수정요청
해당 파일명에 공백이 들어가 있는데 이는 예기치 않은 동작을 발생시킬 수 있습니다. 이러한 공백은 자동으로 공백을 의미하는 대체 문자열로 변환되기 때문입니다.
그래서 일반적으로 공백대신 _ -를 이용해 icon_img, icon-img 같은 식으로 공백을 표현합니다.
해당 이미지의 경우 logo이니 간단하게 logo.png로 변경하시는 것을 추천드려요

https://developer.mozilla.org/ko/docs/Glossary/Percent-encoding?utm_source=chatgpt.com

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 public/logos/Property [email protected]
Copy link
Collaborator

Choose a reason for hiding this comment

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

❗️ 수정요청
지금 이미지 파일이 크기별로 존재하고 사용하시는 파일은 public/logos/Property 1=lg.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 public/logos/Property [email protected]
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Loading