-
Notifications
You must be signed in to change notification settings - Fork 39
[명지우] Sprint3 #94
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-\uBA85\uC9C0\uC6B0-sprint3"
[명지우] Sprint3 #94
Changes from all commits
2c5f23a
d49b0e4
105a3ee
7fea470
f695777
db16c36
16cad62
d01ddc3
b9cfaee
001fbfd
2253397
043b3cf
2ad788b
c66fc92
418bb2d
df96f9e
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 |
|---|---|---|
| @@ -1,43 +1,50 @@ | ||
| ## 판다마켓 2 | ||
| ## 판다마켓 3 | ||
|
|
||
| **🌐 배포 주소 : https://myungjiwoo-pandamarket.netlify.app/** | ||
|
|
||
| ### 기본 요구사항 | ||
|
|
||
| - [x] Github의 PR을 만들어서 미션을 제출한다. | ||
| - [x] Netlify에 파일 배포가 아닌 포크한 Github 레포지토리로 연결한다. | ||
| - [x] 피그마 디자인에 맞게 페이지를 만든다. | ||
| - [x] UI 라이브러리를 사용하지 않고 진행한다. | ||
|
|
||
| ### 체크 리스트 (기본) | ||
|
|
||
| **공통** | ||
|
|
||
| - [x] 아래로 스크롤 해도 상단 navbar가 최상단에 고정된다. | ||
| - [x] "판다마켓" 클릭 시 루트 페이지("/")로 이동한다. (새로고침) | ||
| - [x] 로그인 페이지, 회원가입 페이지 모두 로고 위 상단 여백이 동일하다. | ||
| - [x] SNS 아이콘들은 클릭시 각각 실제 서비스 홈페이지로 이동한다. | ||
| - [x] 브라우저에 현재 보이는 화면의 영역 너비를 기준으로 분기되는 반응형 디자인을 적용한다. | ||
| - PC: 1200px ~ | ||
| - Tablet: 768px ~ 1199px | ||
| - Mobile: 375px ~ 767px | ||
| - 375px 미만 사이즈의 디자인은 고려하지 않는다. | ||
|
|
||
| **로그인 페이지** | ||
| **랜딩 페이지** | ||
|
|
||
| - [x] "회원가입" 버튼 클릭 시 "/signup" 페이지로 이동한다. | ||
| - [x] Tablet 사이즈로 작아질 때 "판다마켓" 로고의 왼쪽에 여백 "24px", "로그인" 버튼 오른쪽 여백 "24px"을 유지할 수 있도록 "판다마켓" 로고와 "로그인" 버튼의 간격이 가까워진다. | ||
| - [x] Mobile 사이즈로 작아질 때 "판다마켓" 로고의 왼쪽에 여백 "16px", "로그인" 버튼 오른쪽 여백 "16px"을 유지할 수 있도록 "판다마켓" 로고와 "로그인" 버튼의 간격이 가까워진다. | ||
| - [x] 화면 영역이 줄어들면 "Privacy Policy", "FAQ", "codeit-2024"이 있는 영역과 SNS 아이콘들이 있는 영역의 간격이 줄어든다. | ||
|
|
||
| **회원가입 페이지** | ||
| **로그인, 회원가입 페이지 공통** | ||
|
|
||
| - [x] "로그인" 버튼 클릭 시 "/login" 페이지로 이동한다. | ||
| - [x] Tablet 사이즈에서 내부 디자인은 PC 사이즈와 동일하다. | ||
| - [x] Mobile 사이즈에서 좌우 여백 16px을 제외하고 내부 요소들이 너비를 모두 차지한다. | ||
| - [x] Mobile 사이즈에서 내부 요소들의 너비는 기기의 너비가 커지는 만큼 커지지만 400px을 넘지 않는다. | ||
|
|
||
| ### 체크 리스트 (심화) | ||
|
|
||
| - [x] palette에 있는 color 값들을 css 변수로 등록하고 사용한다. | ||
| - [x] 비밀번호 input 요소 위에 비밀번호를 확인할 수 있는 아이콘을 추가한다. | ||
| - [x] 페이스북, 카카오톡, 디스코드, 트위터 등 SNS에서 Linkbrary 랜딩 페이지("/") 공유 시 미리보기를 볼 수 있는 랜딩 페이지 메타 태그를 설정한다. | ||
| - [x] 미리보기에서 제목은 "판다 마켓", 설명은 "일상의 모든 물건을 거래해보세요"로 설정한다. | ||
| - [x] 주소와 이미지는 자유롭게 설정한다. | ||
|
|
||
| ### 지난 코드 리뷰 | ||
|
|
||
| - [x] notoSans 폰트 삭제. Pretendard 폰트만 사용. | ||
| - [x] 페이지에 대응하는 css 파일을 만들고 내부적으로 해당 페이지 구현을 위해 필요한 css 들을 import하는 방식으로 수정 | ||
| ➡️ 만약 css를 분리할거라면, 분리된 스타일 파일에 media query 문도 추가하기. | ||
| - [x] alt 속성을 이미지 내용이 드러나도록 수정. | ||
| - [x] sns 아이콘들을 list로 마크업. | ||
| - [x] 이미지 파일명 일관성있게 수정. | ||
| - [x] 디자인 구현에 불필요한 값들은 삭제. | ||
| - [x] css에서 소수점 사용 지양. | ||
| - [x] alt 속성명 수정하기 (가능한 해당 이미지를 잘 표현할 수 있는 문자열) | ||
| - [x] favicon 배경 삭제 | ||
| - [x] 여백 없이 로고 이미지 추출 | ||
| - [x] 이미지, 아이콘 파일 네이밍 수정 | ||
| - [x] 태그 간 계층 구조를 명확하게 개행하기 | ||
| - [x] 작성한 비밀번호 확인하는 버튼의 이미지 태그를 button 요소로 변경 | ||
| - [x] 불필요한 주석 삭제하기 | ||
| - [x] 태그 선택자 대신에 class 사용하기 | ||
| - [x] 네이밍 규칙 통일하기 (카멜 케이스 vs 파스칼 케이스) | ||
| - [x] 불필요한 import 삭제하기 (reset.css) |
|
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. 👍 칭찬 |
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -11,42 +11,56 @@ | |||||
| /> | ||||||
| <link rel="stylesheet" href="styles/account/account.css" /> | ||||||
| <title>판다마켓 - 회원가입</title> | ||||||
| <link rel="icon" href="/images/icon_favicon.ico" /> | ||||||
| <link rel="icon" href="/images/icon_favicon.png" /> | ||||||
| </head> | ||||||
|
|
||||||
| <body> | ||||||
| <div class="loginContainer"> | ||||||
| <div class="login-container"> | ||||||
| <header> | ||||||
| <a href="/"> | ||||||
| <img | ||||||
| src="images/img_logo.png" | ||||||
| alt="판다마켓 로고" | ||||||
| class="image-logo" | ||||||
| /> | ||||||
| <img src="images/logo.png" alt="판다마켓" class="image-logo" /> | ||||||
| </a> | ||||||
| </header> | ||||||
|
|
||||||
| <form> | ||||||
| <label>이메일 <input type="email" /></label> | ||||||
| <label>닉네임 <input type="text" /></label> | ||||||
| <label | ||||||
| >비밀번호 <input type="password" /> | ||||||
| <img | ||||||
| src="/images/btn_visibility_on.png" | ||||||
| alt="비밀번호 보기" | ||||||
| class="passwordVisibilityOn" | ||||||
| <form class="form-account"> | ||||||
| <label class="label-account"> | ||||||
| 이메일 | ||||||
| <input | ||||||
| type="email" | ||||||
| placeholder="이메일을 입력해주세요" | ||||||
| class="input-account" | ||||||
| /> | ||||||
| </label> | ||||||
| <label class="label-account"> | ||||||
| 닉네임 | ||||||
| <input | ||||||
| type="text" | ||||||
| placeholder="닉네임을 입력해주세요" | ||||||
| class="input-account" | ||||||
| /> | ||||||
| </label> | ||||||
| <label class="label-account"> | ||||||
| 비밀번호 | ||||||
| <input | ||||||
| type="password" | ||||||
| placeholder="비밀번호를 입력해주세요" | ||||||
| class="input-account password" | ||||||
| /> | ||||||
| <button class="btn-visibility-on"></button> | ||||||
|
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. ❗️ 수정요청
Suggested change
|
||||||
| </label> | ||||||
| <label | ||||||
| >비밀번호 확인 <input type="password" /><img | ||||||
| src="/images/btn_visibility_on.png" | ||||||
| alt="비밀번호 보기" | ||||||
| class="passwordVisibilityOn" | ||||||
| /></label> | ||||||
| <button type="submit">회원가입</button> | ||||||
| <label class="label-account"> | ||||||
| 비밀번호 확인 | ||||||
| <input | ||||||
| type="password" | ||||||
| placeholder="비밀번호를 다시 입력해주세요" | ||||||
| class="input-account password" | ||||||
| /> | ||||||
| <button class="btn-visibility-on"></button> | ||||||
| </label> | ||||||
| <button class="btn-submit" type="submit">회원가입</button> | ||||||
| </form> | ||||||
|
|
||||||
| <div class="SociaLoginContainer"> | ||||||
| <div class="social-login-container"> | ||||||
| <p>간편 로그인하기</p> | ||||||
| <ul> | ||||||
| <li> | ||||||
|
|
@@ -64,5 +78,12 @@ | |||||
|
|
||||||
| <p class="footer">이미 회원이신가요? <a href="/login.html">로그인</a></p> | ||||||
| </div> | ||||||
|
|
||||||
| <div class="unsupported-screen"> | ||||||
| <p> | ||||||
| 지원하지 않는 화면 크기입니다.<br /> | ||||||
| 브라우저의 크기를 넓혀주세요. | ||||||
| </p> | ||||||
| </div> | ||||||
| </body> | ||||||
| </html> | ||||||
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.
💊 제안
이미지는 여백 없이 관리하시는 것이 디자인 구현할때 더 편리합니다.
이미지 자체가 여백을 가지고 있으면 헷갈릴 수 있으니 여백없이 추출하시는 것을 추천드려요!