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

💊 제안
한 레포안에서는 동일한 룰을 따르는 것이 좋습니다.
따라서 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 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 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 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 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 images/Property 1=sm.png
Copy link
Collaborator

Choose a reason for hiding this comment

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

❗️ 수정요청
중복되는 이미지가 크기별로 존재하네요. 사이별로 이미지가 존재해야할 필요가 없다면 이미지 하나로 사용하시면 좋을 것 같습니다.
한 이미지가 다양한 크기로 사용될 때, 예를 들어 160x200 / 80x100 사이즈로 로고가 사용된다면, 160x200으로 추출해서 작은 사이즈의 경우 css를 통해 크기를 조절해서 사용하시면 될 것 같습니다.

Copy link
Collaborator

Choose a reason for hiding this comment

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

💬 여담
figma에서 그때 그때 필요한 이미지만 추출하시는 것을 추천드려요~
아마 같이 사용하는 디자인 파일이라 export 탭에서 export 처리된 친구들을 다 추출하게 되시면
불필요한 친구들도 같이 딸려올 가능성이 큽니다!

Copy link
Collaborator

Choose a reason for hiding this comment

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

❗️ 수정요청
어떤 파일인지 알 수 있게 네이밍하시는 것이 좋아요.
또한 이미지명에 공백이 들어가 있는데 이는 예기치 않은 동작을 발생시킬 수 있습니다. 이러한 공백은 자동으로 공백을 의미하는 대체 문자열로 변환되기 때문입니다.
그래서 일반적으로 공백대신 _ -를 이용해 Property_1=sm, Property-1=sm 같은 식으로 공백을 표현합니다.
태식님의 다른 이미지 파일명으로 보면 _를 공백대신 사용하시면 되겠네요~

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/Property 1=sm@2x.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/Property 1=sm@3x.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/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 images/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/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/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.
127 changes: 127 additions & 0 deletions index.html
Original file line number Diff line number Diff line change
@@ -0,0 +1,127 @@
<!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="stylesheet" href="style.css">
</head>
<body>
<header>
<div class="gnb wrap">
<a href="/" class="logo">
<img src="./images/Property 1=sm.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 라는 의미로, 이미지 파일을 다운로드하는 것에 실패해서 이미지 파일을 보여줄 수 없을 때
어떤 이미지인지 파악할 수 있게 대신 제공되거나, 스크린리더로 읽혀지는 문자를 의미합니다.
따라서 가능한 해당 이미지를 잘 표현할 수 있는 문자열을 넣어주는 것이 좋습니다.

Suggested change
<img src="./images/Property 1=sm.png" alt="판다마켓 로고">
<img src="./images/Property 1=sm.png" alt="판다마켓">

</a>
<a href="/login" class="btn-login Text-lg semibold">로그인</a>
Copy link
Collaborator

Choose a reason for hiding this comment

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

❗️ 수정요청
배포 사이트에서 로그인 버튼을 누르면 당연히 파일이 없으니 에러가 납니다.
다만 파일이 있어도 다른 설정을 해주지 않으시면 같은 에러가 날 확률이 높습니다~

/login의 의미는 root 폴더의 login으로 접근하라는 의미라서, 만약 login.html 파일이 root 파일에 위치해있고
다른 설정을 안해주셨으면 아래처럼 작성해주셔야 정상동작합니다~

Suggested change
<a href="/login" class="btn-login Text-lg semibold">로그인</a>
<a href="/login.html" class="btn-login Text-lg semibold">로그인</a>

</div>
</header>

<main>
<section class="home-top-section">
<div class="home-top wrap">
<div class="home-top-content">
<p class="text-4xl bold">
Copy link
Collaborator

Choose a reason for hiding this comment

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

👍 칭찬
자주 사용되는 스타일들을 유틸 클래스로 선언해두시고 사용하시는 것 좋습니다~

일상의 모든 물건을 <br>
거래해 보세요
</p>
<a href="/items" class="btn-large text-xl semibold">구경하러 가기</a>
</div>
<img src="./images/Img_home_top.png" alt="판다 이미지">
</div>
</section>

<section class="home-section">
<div class="home">
<div class="home-content">
<img src="./images/Img_home_01.png" alt="인기 상품">
<div class="home-content-1">
<p class="keyword text-2lg bold">
Hot item
</p>
<p class="text-4xl bold">
인기 상품을<br>
확인해 보세요
</p>
<p class="home-describe text-2xl medium">
가장 HOT한 중고거래 물품을<br>
판다 마켓에서 확인해 보세요
</p>
</div>
</div>
</div>
<div class="home">
<div class="home-content">
<div class="home-content-2">
<p class="keyword text-2lg bold">
Search
</p>
<p class="text-4xl bold">
구매를 원하는<br>
상품을 검색하세요
</p>
<p class="home-describe text-2xl medium">
구매하고 싶은 물품은 검색해서<br>
쉽게 찾아보세요
</p>
</div>
<img src="./images/Img_home_02.png" alt="상품 검색">
</div>
</div>
<div class="home">
<div class="home-content">
<img src="./images/Img_home_03.png" alt="상품 등록">
<div class="home-content-3">
<p class="keyword text-2lg bold">
Register
</p>
<p class="text-4xl bold">
판매를 원하는<br>
상품을 등록하세요
</p>
<p class="home-describe text-2xl medium">
어떤 물건이든 판매하고 싶은 상품을<br>
쉽게 등록하세요
</p>
</div>
</div>
</div>
</section>

<section class="home-bottom-section">
<div class="home-bottom wrap">
<div class="home-bottom-content">
<p class="text-4xl bold">
믿을 수 있는 <br>
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
믿을 수 있는 <br>
믿을 수 있는<br>

판다마켓 중고 거래
</p>
</div>
<img src="./images/Img_home_bottom.png" alt="판다 이미지">
</div>
</section>
</main>

<footer>
<div class="footer-content wrap">
<p class="copyright">@codeit - 2024</p>
<div class="faq">
<a href="/privacy">Privacy Policy</a>
<a href="/faq">FAQ</a>
</div>
<div class="sns">
<a href="https://facebook.com" target="_blank">
<img src="./images/ic_facebook.png" alt="페이스북">
</a>
<a href="https://twitter.com" target="_blank">
<img src="./images/ic_twitter.png" alt="트위터">
</a>
<a href="https://youtube.com" target="_blank">
<img src="./images/ic_youtube.png" alt="유튜브">
</a>
<a href="https://instagram.com" target="_blank">
<img src="./images/ic_instagram.png" alt="인스타그램">
</a>
</div>
Comment on lines +110 to +123
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="sns">
        <li>
          <a href="https://facebook.com" target="_blank">
            <img src="./images/ic_facebook.png" alt="페이스북">
          </a>
        </li>
      </ul>

</div>
</footer>
</body>
</html>
235 changes: 235 additions & 0 deletions style.css
Original file line number Diff line number Diff line change
@@ -0,0 +1,235 @@
* {
box-sizing: border-box;
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.

💊 제안

font-family는 상속되는 속성이므로, 모든 태그에 개별적으로 지정하기보다 html이나 body 같은 상위 태그에 한 번만 적용하는 것이 더 효율적입니다. 이렇게 하면 하위 태그들이 자연스럽게 상속받아 일관된 스타일을 유지할 수 있어요.
비슷한 속성인 font-size도 같은 방식으로 적용하는 것을 추천드려요~

color: #374151;
font-size: 16px;
}

a {
text-decoration: none;
cursor: pointer;
}

p {
margin: 0;
}
Comment on lines +1 to +15
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 속성들이라 따로 분리하지 않으시고 한 파일에 작성하셨나봐요~
하지만 필요시 특정 태그의 스타일을 초기화하는건 번거로운 일이고, 이를 염두에 두고 css 작성을 해야하니 가능하면 처음부터 브라우저 스타일을 초기화하시고 reset.css로 분리하시고 작업하시는 것을 추천드립니다!


body {
margin: 0;
background-color: #FCFCFC;
}

header {
background-color: #FFFFFF;
border-bottom: 1px solid #DDE0E3
}

footer {
background-color: #111827;
height: 10rem;
}

.gnb {
padding: 9px 400px;
display: flex;
justify-content: space-between;
align-items: center;
}

.gnb img {
max-width: 10rem;
}

.wrap {
max-width: 1920px;
width: 100%;
margin: 0 auto;
}

.home-top-section img, .home-bottom-section img {
max-width: 45rem;
}

.home-top-section {
background-color: #CFE5FF;
height: 34rem;
}

.home-top {
height: 100%;
display: flex;
justify-content: center;
align-items: flex-end;
}

.home-top-content{
display: flex;
flex-direction: column;
gap: 2rem;
padding-bottom: 4%;
}

.home-section img {
max-width: 36rem;
}

.home {
background-color: #FFFFFF;
height: 45rem;
display: flex;
justify-content: center;
align-items: center;
}

.home-content{
background-color: #FCFCFC;
display: flex;
justify-content: center;
gap: 4rem;
border-radius: 12px;
width: 65rem;
}

.home-content-1, .home-content-2, .home-content-3 {
display: flex;
flex-direction: column;
justify-content: center;
margin-right: 0.75rem;
}

.home-content-2 {
text-align: end;
margin-right: 0;
margin-left: 0.75rem;
}

.home-describe {
margin-top: 1.5rem;
}

.home-bottom-section {
background-color: #CFE5FF;
height: 33.75rem;
}

.home-bottom {
height: 100%;
display: flex;
justify-content: center;
align-items: flex-end;
gap: 4rem;
}

.home-bottom-content {
padding-bottom: 10%;
}

.footer-content {
display: flex;
justify-content: space-between;
max-width: 70rem;
padding-top: 1.5%;
}

.faq, .sns {
display: flex;
}

.faq {
gap: 2rem;
}

.sns {
gap: 0.75rem;
}

.btn-large, .btn-login {
background-color: #3692FF;
}

.btn-large:hover, .btn-login:hover {
background-color: #1967D6;
}

.btn-large:active, .btn-login:active {
background-color: #1251AA;
}

.btn-login {
color: #F3F4F6;
width: 8rem;
height: 3rem;
padding: 0.75rem 1.5rem;
border-radius: 8px;
text-align: center;
}

.btn-large {
color: #F9FAFB;
padding: 1rem 7.75rem;
border-radius: 40px;
}

.keyword {
color: #3692FF;
margin-bottom: 0.75rem;
}

.copyright, .faq {
font-size: 1rem;
font-weight: 400;
}

.copyright {
color: #9CA3AF;
}

.faq a {
color: #E5E7EB;
}

.bold {
font-weight: 700;
}

.semibold {
font-weight: 600;
}

.medium {
font-weight: 500;
}

.text-4xl {
font-size: 2.5rem;
line-height: 3.5rem;
letter-spacing: -0.125rem;
}

.text-2xl {
font-size: 1.5rem;
line-height: 2rem;
letter-spacing: -0.0625rem;
}

.text-xl {
font-size: 1.25rem;
line-height: 2rem;
}

.text-2lg {
font-size: 1.125rem;
line-height: 1.625rem;
}
Comment on lines +191 to +223
Copy link
Collaborator

Choose a reason for hiding this comment

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

💊 제안
이런 유틸 클래스들은 따로 파일을 빼시는 것을 추천드려요~
관심사 분리 원칙에 따라 관련있는 친구들만 모아두시는 것이 가독성 및 유지보수에 용이하답니다~
지금 태식님 구조에서 제가 추천드리는 파일 분리 구조는 아래와 같아요~

  1. reset.css : 브라우저 기본 스타일을 초기화하는 코드
  2. utility.css/common.css : 자주사용하는 스타일을 위한 클래스들이 모여있는 곳
  3. [page명].css : index.css, login.css 와 같이 페이지별 스타일이 모여있는 곳


@media (max-width:1920px) {
.gnb {
padding: 9px 200px
}
}

@media (max-width:1280px) {
* {
font-size: 12px;
}
}
Loading