-
Notifications
You must be signed in to change notification settings - Fork 31
[김태욱] sprint2 #127
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
[김태욱] sprint2 #127
The head ref may contain hidden characters: "Basic-\uAE40\uD0DC\uC6B1"
Conversation
dongqui
left a comment
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.
태욱님! 기본적인 구현은 잘 해주셨습니다 :) 시맨틱, 접근성에 조금만 더 신경 써주시면 더욱 좋을 거 같습니다! 👍
| <link rel="icon" href="images/logo/favicon.ico" /> | ||
| <link | ||
| rel="stylesheet" | ||
| as="style" |
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.
as는 preload를 지정했을 때만 사용됩니다 :) 다시 한 번 확인해주세요!
https://developer.mozilla.org/ko/docs/Web/HTML/Element/link#%ED%8A%B9%EC%84%B1
| </a> | ||
| <section class="loginsection"> | ||
| <div id="emailbox"> | ||
| <label class="loginlabel"> 이메일 </label> |
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.
label로 input을 감싸주거나 id, for을 활용하시면 기능적으로도 활용할 수 있습니다 :)
https://developer.mozilla.org/ko/docs/Web/HTML/Reference/Elements/label#%ED%8A%B9%EC%84%B1
| type="password" | ||
| alt="passward" | ||
| /> | ||
| <img |
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.
기능을 가진 이미지라면 button으로 감싸거나 aria label 등을 활용해 주시는 것이 접근성에 좋습니다~!
| id="mainlogo" | ||
| /> | ||
| </a> | ||
| <section class="loginsection"> |
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.
form 태그로 감싸주시는 것이 접근성, 기능적으로 더 적절합니다!
| font-family: "Pretendard", sans-serif; | ||
| } | ||
|
|
||
| header { |
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.
header, footer 등은 불필요하게 login 페이지까지 들어오게 됩니다 :) global은 정말 공통의 스타일만 넣어주시는 것이 좋아요
| @@ -0,0 +1,143 @@ | |||
| /* 회원가입 페이지 전체 스타일 */ | |||
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.
로그인 페이지와 많은 부분이 겹칩니다! 이 경우 하나의 파일로 합치거나, 공통 스타일을 뽑아내는 것이 유지보수에 좋습니다~!
| <link rel="stylesheet" href="styles/signup&loginpage/signup.css" /> | ||
| </head> | ||
| <body> | ||
| <div class="signuppage"> |
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.
띄어쓰기 구분이 없으면 가독성이 떨어집니다!
훗날의 다른 팀원들을 위해 케이스 하나로 통일해 작성하는 습관을 들여주세요 🤣
(html css 보통 케밥 케이스를 많이 씁니다!) signup-page
요구사항
기본
###공통
아래로 스크롤 해도 상단 네비게이션 바(Global Navigation Bar)가 최상단에 고정됩니다.
"판다마켓" 클릭 시 루트 페이지("/")로 이동합니다.(새로고침)
로그인 페이지, 회원가입 페이지 모두 로고 위 상단 여백이 동일합니다.
SNS 아이콘들은 클릭시 각각 실제 서비스 홈페이지로 이동합니다.
###로그인 페이지
###회원가입 페이지
심화
주요 변경사항
스크린샷
멘토에게
-스프린트 미션 1을 했었을때 부터 좀 많은 시간이 지나가지고 스프린트 미션2를 하기전에 스프린트 미션1 템플릿 코드를 사용하여 스프린트 미션 2를 수행했습니다.