Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
a6e66e5
fix: 로고 이미지 깨짐 현상 수정 및 파일명 변경 (logo_1.png -> logo.png)
two678 Mar 14, 2025
c9de9e2
rename: 로그인/회원가입 페이지에서 이미지 파일명 변경 및 코드 수정 (GoogleBg.png -> Google.png…
two678 Mar 14, 2025
08a33b6
feat: CSS 변수(auth.css -> variables.css) 분리 및 login, signup.html에 적용
two678 Mar 14, 2025
27fcacf
style: font-family 상속 구조 개선
two678 Mar 14, 2025
5ae67a7
fix: login.html, signup.html에 label 태그 추가하여 접근성 개선
two678 Mar 14, 2025
8a48ccc
fix: 비밀번호 보이기/숨기기 아이콘을 button 요소로 변경
two678 Mar 14, 2025
e0edb69
fix: signup.html에서 회원가입 링크를 로그인 링크로 변경
two678 Mar 14, 2025
c34cefe
fix: login.html, signup.html 에서 alt 태그 메시지 수정
two678 Mar 14, 2025
77617cd
style: index.html에서 Tablet, Mobile 사이즈에 맞게 로고와 로그인 버튼 여백 조정 (home.css)
two678 Mar 14, 2025
c9fadbe
style: 클래스 이름 하이픈 스타일로 변경 (line_break -> line-break 등)
two678 Mar 14, 2025
2780ba2
style: Tablet, Mobile 사이즈에서 메인 위쪽 페이지 글, 페이지 이동 태그,
two678 Mar 14, 2025
676d3ca
Table, Mobile 사이즈에서 아래쪽 이미지와 글 크기 조정
two678 Mar 14, 2025
81e7eda
style: Tablet, Mobile 사이즈에서 메인 컨테이너 안의 내용물(사진, 글) 사이즈 여백 조정
two678 Mar 14, 2025
d512865
feat: footer 영역 PC, Tablet, Mobile 사이즈에 맞게 레이아웃 및 스타일 조정
two678 Mar 17, 2025
3e1e609
feat: table사이즈, mobile 사이즈에서의 항상 화면 가운데에 내용이 있게
two678 Mar 17, 2025
faebdd0
feat: mobile 사이즈 내부 요소들의 너비 조정
two678 Mar 17, 2025
9c43698
feat: 랜딩페이지 메타태그 설정
two678 Mar 17, 2025
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
File renamed without changes
File renamed without changes
Binary file added assets/logo/logo.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.
Binary file removed assets/logo/logo_1.png
Binary file not shown.
117 changes: 79 additions & 38 deletions css/common/auth.css
Original file line number Diff line number Diff line change
@@ -1,40 +1,27 @@
:root {
--gray900 : #111827;
--gray800 : #1F2937;
--gray700: #374151;
--gray600 : #4B5563;
--gray500 : #6B7280;
--gray400: #9CA3AF;
--gray200 : #E5E7EB;
--gray100 : #F3F4F6;
--gray50: #F9FAFB;
--blue: #3692FF;
html, body {
display: flex;
flex-direction: column;
justify-content: center;
align-items: center;
}

.logo {
width: 396px;
width: auto;
height: 132px;
margin: 0 122px;
}

body {
display: flex;
flex-direction: column;
justify-content: center;
align-items: center;
}

.input_info {
.input-info {
margin-top: 40px;
}

.input_group {
.input-group {
font-size: 18px;
font-weight: 700;
line-height: 26px;
}

.input_group, .login_btn {
.input-group, .login-btn {
margin-bottom: 24px;
}

Expand All @@ -52,28 +39,39 @@ input {
padding: 16px 24px 14px;
}

#pwd {
padding: 16px 24px 16px;
input::placeholder {
color: var(--gray400);
font-weight: 400;
font-size: 16px;
line-height: 26px;
letter-spacing: 0%;
}

/* #pwd {
padding: 16px 24px 16px;
} */

Comment on lines +50 to +53
Copy link
Collaborator

Choose a reason for hiding this comment

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

💊 제안
작성하셨다가 테스트를 하시려고 주석처리하신 것 같아요~
주석이 남아있으면 눈길이 가니 불필요한 코드라면 삭제하시는 것을 추천드려요!


.password {
display: flex;
position: relative;
width: 640px;
width:100%;
}

.pwd_icon {
display: inline-flex;
.pwd-icon-btn {
position: absolute;
right: 16px;
top: 32px;
width: 24px;
height: 24px;
right: 16px;
top: 32px;
}

.pwd-icon {
width: 100%;
height: 100%;
}


.submit_btn button {
.submit-btn button {
color: var(--gray100);
background-color: var(--blue);
width: 640px;
Expand All @@ -86,18 +84,15 @@ input {
margin-bottom: 24px;
}

.sign_easy {
.sign-easy {
display: flex;

box-sizing: border-box;
justify-content: space-between;
align-items: center;
padding: 16px 23px;
gap: 10px;

width: 640px;
height: 74px;

background: #E6F2FF; /* 배경색 */
border-radius: 8px;

Expand All @@ -111,7 +106,7 @@ input {
text-align: center;
}

.sign_easy div {
.sign-easy div {
display: flex;
gap: 16px;
}
Expand All @@ -122,7 +117,7 @@ input {
}


.move_page {
.move-page {
margin-top: 24px;
display: flex;
justify-content: center;
Expand All @@ -133,7 +128,53 @@ input {
color: var(--gray800);
}

.move_page a {
.move-page a {
color: var(--blue);
text-decoration: underline;
}


@media (max-width: 1199px) and (min-width: 768px) {
body {
margin: 190px 52px 325px;
}


Comment on lines +141 to +142
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

}

@media (max-width: 767px) and (min-width: 375px) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

💊 제안
min-width가 조건에 걸려있어 375px 미만으로 화면이 작아지게 되면 PC 스타일이 적용되네요~
body에 min-width: 375px를 추가하셔서 해당 사이즈 미만으로 작아지게 되면 가로 스크롤이 생기게 해주시거나
아니라면 min-width를 조건문에서 빼주시는 것을 추천드려요~


.logo-a {
display: flex;
justify-content: center;
}

.logo {
width: 198px;
height: 66px;
}

main {
width: 100%;
max-width: 400px;
min-width: 343px;
margin: 0 auto;
}

input {
Copy link
Collaborator

Choose a reason for hiding this comment

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

💊 제안
태그 선택자는 해당 태그가 모든 페이지에 걸쳐 적용되어야 하는 스타일링 외에는 사용을 추천드리지 않습니다~
위의 경우도 프로젝트가 확장되다보면 변경될 수 있으니 가능한 class를 사용하시는 것을 추천드릴께요~

예를 들어, 1월에는 auth.css를 사용하는 페이지가 로그인, 회원가입만 존재하고 해당 페이지의 input 요소가 width 속성 100%를 가지고 있어 이렇게 작업을 했지만 2월에 비밀번호 찾기라는 auth 관련 새 페이지가 추가될 수 있습니다. 해당 페이지는 다른 페이지들처럼 디자인은 거의 비슷하지만 사용되는 input 요소의 width 속성이 90% 일 수 있습니다. 따라서 이렇게 디자인에 따라 다양하게 적용될 수 있는 스타일의 경우 classname을 통해 개별로 css 해주시는 것을 추천드려요.

물론 지금 저희가 작업하는 환경의 경우 위와 같은 일은 없겠지만 클래스를 이용한 스타일링을 하시는 습관을 들이시는 것이 좋습니다~

width: 100%;
}

.submit-btn button {
width: 100%;
}

.sign-easy {
width: 100%;
}

.pwd-icon-btn {
right: 16px;
}

}
6 changes: 5 additions & 1 deletion css/common/reset.css
Original file line number Diff line number Diff line change
Expand Up @@ -8,10 +8,14 @@
}

/* 글꼴 설정 (모든 요소에 대해) */
body, h1, h2, h3, h4, h5, h6, p, a, li, ul, ol {
body {
font-family: "Pretendard Variable", Pretendard, -apple-system, BlinkMacSystemFont, system-ui, Roboto, "Helvetica Neue", "Segoe UI", "Apple SD Gothic Neo", "Noto Sans KR", "Malgun Gothic", "Apple Color Emoji", "Segoe UI Emoji", "Segoe UI Symbol", sans-serif;
}

h1, h2, h3, h4, h5, h6, p, a, li, ul, ol {
font-family: inherit;
}

/* a 태그의 기본 스타일 제거 */
a {
text-decoration: none;
Expand Down
12 changes: 12 additions & 0 deletions css/common/variables.css
Copy link
Collaborator

Choose a reason for hiding this comment

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

💊 제안
css 변수들을 분리하신 점 좋습니다. 다만 figma에 정의된 색상 변수보다 수가 더 적은 것 같아요~

스크린샷 2025-03-13 오후 3 22 14 스크린샷 2025-03-13 오후 3 23 55

큰 이유가 없다면 figma와 동일하게 색상 변수를 선언하시는 것이 좋습니다.
그래야 figma에서 해당 색상 변수가 사용될때 확인하기 쉽고 색상값은 디자이너와 의사 소통할 일이 많은 값이라 소통 측면에서도 같은 이름을 쓰는 것이 좋기 때문입니다~

Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
:root {
--gray900 : #111827;
--gray800 : #1F2937;
--gray700: #374151;
--gray600 : #4B5563;
--gray500 : #6B7280;
--gray400: #9CA3AF;
--gray200 : #E5E7EB;
--gray100 : #F3F4F6;
--gray50: #F9FAFB;
--blue: #3692FF;
}
Loading
Loading