Skip to content
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

[최성락] sprint1 #47

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
1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
img/
Binary file added img/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 img/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 img/Property 1=md.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 img/Vector-1.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 img/Vector-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 img/Vector-3.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 img/Vector.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 img/desktop_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.

이부분은 왼쪽 영역만 이미지로 추출하고 오른쪽 영역은 직접 html과 css로 만드셨어야 합니다!

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 img/desktop_03.png
Copy link
Collaborator

Choose a reason for hiding this comment

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

이 영역도 위와 마찬가지에요!
"구매를 원하는 상품을 검색하세요" 쪽 영역은 html로 만드셔야합니다1

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 img/desktop_04.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.
91 changes: 91 additions & 0 deletions index.html
Original file line number Diff line number Diff line change
@@ -0,0 +1,91 @@
<!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" />
<title>판다마켓</title>
</head>

<body>
<header>
<a href="/" class="logo-img"
><img src="img/Property 1=md.png" alt="로고"
/></a>
<a href="/login" class="login">로그인</a>
</header>
<div class="container">
Copy link
Collaborator

Choose a reason for hiding this comment

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

태그 안에서 , 는 잘 나누어 주셨는데 이부분이 조금 아쉽네요 `` 이렇게 하고 위 div태그는 없어지는 편이 좀 더 좋을것 같습니다 ㅎ

<main>
<section class="banner">
<div class="banner-section">
Comment on lines +19 to +20
Copy link
Collaborator

Choose a reason for hiding this comment

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

여기도요 ㅎ 굳이

 <section class="banner">
          <div class="banner-section">
...

과 같은 중첩 구조가 필요했을까 싶긴 하네요 ㅎ 그냥

태그 하나로는 처리하는게 좀 더 좋아보입니다 ㅎ

<div>
<span>일상의 모든 물건을<br />거래해 보세요</span>
<a href="/items" class="item-link">구경하러 가기</a>
</div>
<div>
<img
src="img/Img_home_top.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 속성 달아주시는건 너무 좋습니다 ㅎㅎ

class="banner img"
Copy link
Collaborator

Choose a reason for hiding this comment

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

class도 여러개를 조합해서 사용하는거 너무 좋아요!

/>
</div>
</div>
</section>

<section class="main-content">
<img src="img/desktop_02.png" alt="메인 이미지 1" />
</section>

<section class="main-content">
<img src="img/desktop_03.png" alt="메인 이미지 2" />
</section>

<section class="main-content">
<img src="img/desktop_04.png" alt="메인 이미지 3" />
</section>

<section class="banner">
<div class="banner-section">
<div>
<span>믿을 수 있는<br />판다마켓 중고 거래</span>
</div>

<div>
<img
src="img/Img_home_bottom.png"
alt="하단 이미지"
class="banner img"
/>
</div>
</div>
</section>
</main>

<footer>
<div class="footer-content">
<div class="footer-copyright">
<span>©codeit - 2024</span>
</div>
<div class="footer-info">
<span><a href="/privacy">Privacy Policy</a></span>
<span><a href="/faq">FAQ</a></span>
</div>
<div class="footer-social">
<a href="https://facebook.com" target="_blank"
><img src="img/Vector.png" alt="페이스북" class="social-link"
/></a>
<a href="https://x.com" target="_blank"
><img src="img/Vector-1.png" alt="트위터" class="social-link"
/></a>
<a href="https://youtube.com" target="_blank"
><img src="img/Vector-2.png" alt="유튜브" class="social-link"
/></a>
<a href="https://instagram.com" target="_blank"
><img src="img/Vector-3.png" alt="인스타그램" class="social-link"
/></a>
</div>
</div>
</footer>
</div>
</body>
</html>
155 changes: 155 additions & 0 deletions style.css
Original file line number Diff line number Diff line change
@@ -0,0 +1,155 @@
* {
box-sizing: border-box;
}

body {
font-family: Arial, sans-serif;
margin: 0;
padding: 0;
height: 100vh;
Copy link
Collaborator

Choose a reason for hiding this comment

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

여기 height는 굳이 없어도 될것 같다는 생각이 드네요 ㅎ

}

a {
cursor: pointer;
}

.container {
width: 100%;
margin: 0 auto;
}

header {
width: 100%;
height: 70px;
display: flex;
justify-content: space-between;
align-items: center;
padding: 9px 200px;
margin: 0 auto;
}

.logo-img {
display: flex;
align-items: center;
}

.login {
background-color: #3692FF ;
color: #F3F4F6;
text-decoration: none;
line-height: 26px;
font-weight: 600;
width: 128px;
height: 48px;
border-radius: 8px;
padding: 12px 23px;
text-align: center;
}

.button.login:hover {
background-color: #1967D6 ;
}

.banner {
width: 100%;
height: 540px;
position: relative;
background-color: #CFE5FF ;
display: flex;
justify-content: center;
align-items: flex-end;
margin: 0 auto;
}

.banner.img {
width: 746px;
height: 340px;
}

.banner span {
display: block;
font-size: 40px;
font-weight: 700;
color: #374151;
position: relative;
line-height: 56px;
top: 30px;
margin-bottom: 60px;
}

.banner-section{
display: flex;
justify-content: space-between;
width: 1110px;
Copy link
Collaborator

Choose a reason for hiding this comment

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

이런 부분은 width 보다는 max-width를 사용하는게 좀더 유연한 레이아웃이 될 것 같습니다 ㅎ

만약 viewport가 1110px 미만이라면 해당 코드로 인해 가로 스크롤이 생기고 말거에요 ㅎ

height: 340px;
}

.banner-section div:first-child {
flex: 1;
}

.banner-section div:last-child {
flex: 1;
flex-shrink: 2;
}

.item-link {
background-color: #3692FF;
padding: 16px 124px;
border-radius: 40px;
font-size: 20px;
line-height: 24px;
font-weight: 600;
color: #ffffff;
text-decoration: none;
}

.item-link:hover,
.login:hover {
background-color: #1967D6;
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

호버 처리하는것 좋습니다 !@


.main-content {
height: 720px;
width: 100%;
display: flex;
justify-content: center;
align-items: center;
margin: 0 auto;
}

footer {
width: 100%;
height: 160px;
background-color: #111827;
}

.footer-content {
display: flex;
justify-content: space-between;
width: 1120px;
margin: 0 auto;
padding: 40px 0;
}

.footer-copyright {
color: #9CA3AF;
}

.footer-info {
display: flex;
gap: 15px;
}

.footer-info span a {
color: #E5E7EB;
text-decoration: none;
}

.footer-social {
display: flex;
gap: 15px;
align-items: center;
}


Loading