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
19 changes: 14 additions & 5 deletions .github/pull_request_template.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,14 +2,23 @@

### 기본

- [x]
- []
- []
- [0] 아래로 스크롤 해도 상단 네비게이션 바(Global Navigation Bar)가 최상단에 고정됩니다.
Copy link
Collaborator

Choose a reason for hiding this comment

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

💊 제안
mdx 포멧에서 list 요소의 check 여부를 작성하실 때는 영문 o를 사용하시면 됩니다~

Suggested change
- [0] 아래로 스크롤 해도 상단 네비게이션 바(Global Navigation Bar)가 최상단에 고정됩니다.
- [o] 아래로 스크롤 해도 상단 네비게이션 바(Global Navigation Bar)가 최상단에 고정됩니다.

- [0] 판다마켓" 클릭 시 루트 페이지("/")로 이동합니다.(새로고침)
- [0] 로그인 페이지, 회원가입 페이지 모두 로고 위 상단 여백이 동일합니다.
- [0] SNS 아이콘들은 클릭시 각각 실제 서비스 홈페이지로 이동합니다.

### 로그인 페이지

- [0] "회원가입"버튼 클릭 시 "/signup" 페이지로 이동합니다.

### 회원가입 페이지

- [0] "로그인"버튼 클릭 시 "/login" 페이지로 이동합니다.

### 심화

- [x]
- []
- [x] palette에 있는 color값들을 css 변수로 등록하고 사용해 주세요.
- [0] 비밀번호 input 요소 위에 비밀번호를 확인할 수 있는 아이콘을 추가해 주세요.

## 주요 변경사항

Expand Down
103 changes: 103 additions & 0 deletions css/auth.css
Copy link
Collaborator

Choose a reason for hiding this comment

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

👍 칭찬
중복된 코드를 줄이고자 하나의 파일로 작성하신 점 좋습니다.

Original file line number Diff line number Diff line change
@@ -0,0 +1,103 @@
.auth-container {
max-width: 640px;
margin: 60px auto;
text-align: center;
}

.logo-container {
margin-bottom: 40px;
}

.auth-form {
width: 100%;
max-width: 640px;
margin: 0 auto;
}

.input-group {
margin-bottom: 24px;
text-align: left;
}

.input-group label {
display: block;
font-weight: 500;
margin-bottom: 16px;
font-size: 16px;
color: #1f2937;
}

.input-group input {
Copy link
Collaborator

Choose a reason for hiding this comment

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

❗️ 수정요청
해당 input이 디자인보다 높이가 작게 작업되어 있어요~
피그마에서 해당 요소의 높이를 확인해보시고 디자인과 동일하게 변경해보세요!

width: 100%;
max-width: 640px;
padding: 12px;
font-size: 16px;
background-color: #f5f5f5;
color: #1f2937;
border: none;
border-radius: 5px;
box-sizing: border-box;
outline: none;
}

.input-group input:focus {
background-color: #e0e0e0;
color: #1f2937;
}

.password-wrapper {
position: relative;
display: flex;
align-items: center;
}

.password-wrapper input {
width: 100%;
padding-right: 40px;
}

.toggle-password {
position: absolute;
right: 12px;
cursor: pointer;
}

.auth-button {
width: 100%;
max-width: 640px;
padding: 16px;
font-size: 18px;
font-weight: 600;
border-radius: 40px;
background-color: #3692ff;
color: white;
cursor: pointer;
border: none;
}

.auth-button:hover {
background-color: #1967d6;
}

.social-login {
background-color: #e6f2ff;
border-radius: 8px;
padding: 16px 23px;
margin: 24px auto;
display: flex;
align-items: center;
justify-content: space-between;
max-width: 640px;
}

.social-icons {
display: flex;
gap: 16px;
}

.auth-footer {
font-weight: 500;
font-size: 15px;
max-width: 640px;
margin: 0 auto;
}
32 changes: 32 additions & 0 deletions css/components.css
Original file line number Diff line number Diff line change
@@ -0,0 +1,32 @@
.button {
background-color: #3692ff;
color: #ffffff;
display: inline-flex;
align-items: center;
justify-content: center;
padding: 12px 24px;
border-radius: 8px;
font-size: 16px;
font-weight: 600;
}

.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;
}
21 changes: 21 additions & 0 deletions css/forms.css
Original file line number Diff line number Diff line change
@@ -0,0 +1,21 @@
input {
width: 100%;
padding: 12px;
border: 1px solid #ccc;
border-radius: 5px;
font-size: 16px;
}

input:focus {
border-color: #3692ff;
outline: none;
}

.form-container {
max-width: 400px;
margin: auto;
padding: 20px;
background: #f9f9f9;
border-radius: 8px;
box-shadow: 0px 4px 6px rgba(0, 0, 0, 0.1);
}
30 changes: 30 additions & 0 deletions css/global.css
Copy link
Collaborator

Choose a reason for hiding this comment

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

👍 칭찬
모든 페이지에 걸쳐 사용되는 속성을 global.css 로 분리해주신 것 좋습니다 👍

Original file line number Diff line number Diff line change
@@ -0,0 +1,30 @@
* {
margin: 0;
padding: 0;
box-sizing: border-box;
}

body {
color: #374151;
word-break: keep-all;
font-family: "Pretendard", sans-serif;
}

a {
text-decoration: none;
color: inherit;
}

button {
background: none;
border: none;
outline: none;
box-shadow: none;
cursor: pointer;
}

.wrapper {
max-width: 1200px;
margin: 0 auto;
width: 100%;
}
52 changes: 52 additions & 0 deletions css/home.css
Original file line number Diff line number Diff line change
@@ -0,0 +1,52 @@
.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("../images/home/Img_home_top.png");
}

#bottomBanner {
background-image: url("../images/home/Img_home_bottom.png");
}

#features {
padding-bottom: 138px;
}

.feature {
padding: 138px 0;
display: flex;
align-items: center;
gap: 5%;
}

.feature:nth-child(2) {
text-align: right;
}

.feature-content {
flex: 1;
}

.feature-tag {
color: #3692ff;
font-size: 18px;
line-height: 25px;
font-weight: 700;
margin-bottom: 12px;
}

.feature-description {
font-size: 24px;
font-weight: 500;
line-height: 120%;
letter-spacing: 0.08em;
margin-top: 24px;
}
40 changes: 40 additions & 0 deletions css/layout.css
Original file line number Diff line number Diff line change
@@ -0,0 +1,40 @@
header {
Copy link
Collaborator

Choose a reason for hiding this comment

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

💊 제안
현재 루트 페이지의 button 태그와 로그인, 회원가입 페이지의 button 태그가 서로 다른 디자인으로 존재하는 것 처럼 태그는 페이지마다 다르게 사용될 수 있습니다. 따라서 태그 선택자는 모든 페이지에서 동일한 스타일을 적용할 때만 사용하는 것이 좋습니다.

특정 페이지에서만 다르게 스타일링해야 하는 요소라면 태그 선택자보다는 class를 사용하는 것이 유지보수와 확장성 면에서 더 좋습니다. 프로젝트가 커질수록 각 페이지의 구조나 스타일이 변경될 가능성이 높으므로, 이를 미리 고려하여 class로 스타일을 지정하는 것을 추천드립니다.

position: fixed;
width: 100%;
height: 70px;
display: flex;
justify-content: space-between;
align-items: center;
padding: 0 200px;
background-color: #ffffff;
border-bottom: 1px solid #dfdfdf;
z-index: 1000;
}

#loginLinkButton {
font-size: 16px;
font-weight: 600;
border-radius: 8px;
padding: 11.5px 23px;
}

footer {
background-color: #111827;
color: #9ca3af;
display: flex;
justify-content: space-between;
align-items: center;
padding: 32px 200px 108px 200px;
font-size: 16px;
}

#footerMenu {
Copy link
Collaborator

Choose a reason for hiding this comment

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

💊 제안
아이디는 한 페이지에서 고유해야 하는 값입니다. 이러한 특성으로 클래스와 달리 재활용이 어렵고 우선순위가 높기 때문에 덮어씌우기가 어렵습니다. css 작성시에는 클래스 선택자 사용을 추천드립니다~
아이디 선택자는 css 작성시보다 JS, 혹은 특정 태그를 유일하게 식별해야할 때 사용하시는 것을 추천드려요!

display: flex;
gap: 30px;
color: #e5e7eb;
}

#socialMedia {
display: flex;
gap: 12px;
}
Loading
Loading