-
Notifications
You must be signed in to change notification settings - Fork 39
[하재호]sprint1 #120
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
The head ref may contain hidden characters: "Basic-\uD558\uC7AC\uD638-sprint1"
[하재호]sprint1 #120
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| 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" /> | ||
| <title>판다마켓</title> | ||
| <link rel="icon" href="icon.png" /> | ||
| <link rel="stylesheet" href="mission1.css" /> | ||
| </head> | ||
|
|
||
| <body> | ||
|
|
||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 불필요한 공백은 제거해볼까요? |
||
| <header> | ||
|
|
||
| <a href="index.html" | ||
| ><img | ||
| src="top-logo.png" | ||
| alt="판다마켓 홈" | ||
| width="153" | ||
| /> | ||
|
|
||
| <a href="login.html" id="loginLinkButton" class="button">로그인</a> | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 스크린리더 사용자들을 위한 접근성 향상을 위해 aria-label을 사용해볼까요? <a
href="/login"
class="login"
aria-label="로그인 페이지로 이동"
>
로그인
</a>접근성 개념에 대해 더 알고싶으시다면 아래 아티클 참고해보세요! |
||
| </header> | ||
| <main> | ||
| <section id="hero" class="banner"> | ||
| <div class="wrapper"> | ||
| <h1> | ||
| 일상의 모든 물건을<br /> | ||
| 거래해 보세요 | ||
| </h1> | ||
| <a href="items.html" class="button pill-button">구경하러 가기</a> | ||
| </div> | ||
| </section> | ||
|
|
||
| <section id="bodt" class="wrapper"> | ||
| <div class="body"> | ||
|
|
||
| <img | ||
| src="body1.png" | ||
| alt="인기 상품" | ||
| width="50%" | ||
| /> | ||
| <div class="body-content"> | ||
| <h2 class="body-tag">Hot item</h2> | ||
| <h1>인기 상품을<br />확인해 보세요</h1> | ||
| <p class="body-description"> | ||
| 가장 HOT한 중고거래 물품을<br />판다마켓에서 확인해 보세요 | ||
| </p> | ||
| </div> | ||
| </div> | ||
| <div class="body"> | ||
| <div class="body-content"> | ||
| <h2 class="body-tag">Search</h2> | ||
| <h1>구매를 원하는<br />상품을 검색하세요</h1> | ||
| <p class="body-description"> | ||
| 구매하고 싶은 물품은 검색해서 | ||
| <br />쉽게 찾아보세요 | ||
| </p> | ||
| </div> | ||
| <img | ||
| src="body2.png" | ||
| alt="검색 기능" | ||
| width="50%" | ||
| /> | ||
| </div> | ||
| <div class="body"> | ||
| <img | ||
| src="body3.png" | ||
| alt="판매 상품 등록" | ||
| width="50%" | ||
| /> | ||
| <div class="body-content"> | ||
| <h2 class="body-tag">Register</h2> | ||
| <h1>판매를 원하는<br />상품을 등록하세요</h1> | ||
| <p class="body-description"> | ||
| 어떤 물건이든 판매하고 싶은 상품을 | ||
| <br />쉽게 등록하세요 | ||
| </p> | ||
| </div> | ||
| </div> | ||
| </section> | ||
|
|
||
| <section id="bottomBanner" class="banner"> | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 클래스이름을 지을때는 UI의 색상, 위치, 순서등이 포함되어있으면 디자인 변경 시 대응하기도 어려워지고 유지보수에 좋지 않아요. 이런 정보들을 포함하기보다는 항상 엘리먼트의 역할을 명확하게 나타낼수있는 네이밍을 써볼까요? :) |
||
| <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="facebook-logo.png" alt="페이스북" width="20" | ||
| /></a> | ||
| <a href="https://twitter.com/" target="_blank" rel="noopener noreferrer" | ||
| ><img src="twitter-logo.png" alt="트위터" width="20" | ||
| /></a> | ||
| <a href="https://www.youtube.com/" | ||
| target="_blank" | ||
| rel="noopener noreferrer" | ||
| ><img src="youtube-logo.png" alt="유튜브" width="20" | ||
| /></a> | ||
| <a href="https://www.instagram.com/" | ||
| target="_blank" | ||
| rel="noopener noreferrer" | ||
| ><img | ||
| src="instagram-logo.png" | ||
| alt="인스타그램" | ||
| width="20" | ||
| /></a> | ||
| </div> | ||
| </footer> | ||
| </body> | ||
| </html> | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,159 @@ | ||
| * { | ||
| margin: 0; | ||
| padding: 0; | ||
| box-sizing: border-box; | ||
| } | ||
|
|
||
| a { | ||
| text-decoration: none; | ||
| color: inherit; | ||
| } | ||
|
|
||
| button { | ||
| background: none; | ||
| border: none; | ||
| outline: none; | ||
| box-shadow: none; | ||
| cursor: pointer; | ||
| } | ||
|
Comment on lines
+1
to
+18
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 용도에 따라 CSS를 나눠서 관리해볼까요?
이렇게 용도에 따라 파일을 나눠 관리하는건 어떤 이점을 가져다줄수있는지 한번 더 생각해볼수있어요.
|
||
|
|
||
| img { | ||
| vertical-align: bottom; | ||
| } | ||
|
|
||
|
|
||
| body { | ||
| color: #374151; | ||
| word-break: keep-all; | ||
| font-family: "Pretendard", sans-serif; | ||
| } | ||
|
|
||
| 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; | ||
| } | ||
|
|
||
| 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; | ||
| } | ||
|
|
||
| #socialMedia { | ||
| display: flex; | ||
| gap: 12px; | ||
| } | ||
|
|
||
| .wrapper { | ||
| max-width: 1200px; | ||
| margin: 0 auto; | ||
| width: 100%; | ||
| } | ||
|
|
||
| h1 { | ||
| font-size: 40px; | ||
| font-weight: 700; | ||
| line-height: 56px; | ||
| letter-spacing: 0.02em; | ||
| } | ||
|
|
||
| .button { | ||
| background-color: #3692ff; | ||
| color: #ffffff; | ||
| display: inline-flex; | ||
| align-items: center; | ||
| justify-content: center; | ||
| } | ||
|
|
||
| .button:hover { | ||
| background-color: #1967d6; | ||
| } | ||
|
|
||
| .button:focus { | ||
| background-color: #1251aa; | ||
| } | ||
|
|
||
| .button:disabled { | ||
| background-color: #9ca3af; | ||
| cursor: default; | ||
| pointer-events: none; | ||
| } | ||
|
|
||
| .pill-button { | ||
| font-size: 20px; | ||
| font-weight: 700; | ||
| border-radius: 999px; | ||
| padding: 16px 124px; | ||
| } | ||
| .banner { | ||
| background-color: #cfe5ff; | ||
| height: 540px; | ||
| display: flex; | ||
| align-items: center; | ||
| background-repeat: no-repeat; | ||
| background-position: 80% bottom; | ||
| background-size: 55%; | ||
| } | ||
|
|
||
| #hero { | ||
| background-image: url("top.png"); | ||
| } | ||
|
|
||
| #body { | ||
| padding-bottom: 138px; | ||
| } | ||
|
|
||
| #bottomBanner { | ||
| background-image: url("bottom.png"); | ||
| } | ||
|
|
||
| #loginLinkButton { | ||
| font-size: 16px; | ||
| font-weight: 600; | ||
| border-radius: 8px; | ||
| padding: 11.5px 23px; | ||
| } | ||
|
|
||
| .banner .pill-button { | ||
| margin-top: 32px; | ||
| } | ||
|
|
||
| .body { | ||
| padding: 138px 0; | ||
| display: flex; | ||
| align-items: center; | ||
| gap: 5%; | ||
| } | ||
|
|
||
|
|
||
| .body-tag { | ||
| color: #3692ff; | ||
| font-size: 18px; | ||
| line-height: 25px; | ||
| font-weight: 700; | ||
| margin-bottom: 12px; | ||
| } | ||
|
|
||
| .body-description { | ||
| font-size: 24px; | ||
| font-weight: 500; | ||
| line-height: 120%; | ||
| letter-spacing: 0.08em; | ||
| margin-top: 24px; | ||
| } | ||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
투명 배경을 지원할 필요가 없으면 png보다는 jpg 파일 사용을 권장드립니다.
추천 드리자면:
컨텐츠의 특성 혹은 몇가지 조건에 따라 올바른 파일 포맷을 고르는것도 최적화에 포함됩니다.
제가 추천드린것말고도 여러가지 참고해보시고 적용해보세요 :)