-
Notifications
You must be signed in to change notification settings - Fork 15
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
Basic 정진호 sprint3 #33
The head ref may contain hidden characters: "basic-\uC815\uC9C4\uD638-sprint3"
Basic 정진호 sprint3 #33
Conversation
} | ||
} | ||
|
||
function togglePasswordVisibility_c() { |
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.
함수의 이름이 길어져도 되니 _c 보다는 confirm 이라는 네이밍으로 명시해주면 좋겠습니다.
추가로 네이밍 전략은 camelCase 로 정하셨다면 전부 camel 로 작성해주시는걸 권장드립니다.
camelCase, 카멜 케이스, 낙타 표기법
PascalCase, 파스칼 케이스, 파스칼 표기법
snake_case, 스네이크 케이스, 뱀 표기법
kebab-case, 케밥 케이스, 케밥 표기법
UPPER_SNAKE_CASE, 어퍼 스네이크 케이스, 대문자 뱀 표기법
UPPER-KEBAB-CASE, 어퍼 케밥 케이스, 대문자 케밥 표기법
|
||
|
||
function togglePasswordVisibility() { | ||
const passwordInput = document.getElementById("password"); |
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.
상단에 password input dom 선택자를 미리 선언해 두셔서 그걸 사용해셔도 됩니다!
|
||
window.addEventListener("DOMContentLoaded", validateLogin); | ||
|
||
function showEmailError() { |
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.
이메일 예외를 체크하는 로직이 login 과 동일하므로
하나의 js 에서 같이 사용하도록 처리하면 좋겠습니다!
참고해보세요!
다른 공통으로 사용할만한 것들이 있으면 같이 확인해 보시고 고민해보세요!
이번 스프린트도 고생 많으셨습니다 진호님! 전반적으로 잘 작업 하셨고, 피드백 드린 부분들만 한번 고민해 보시고 적용해 보시면 좋겠습니다! 다음 스프린트도 화이팅입니다! |
기본 요구사항
공통
로그인, 회원가입 페이지 공통
로그인 페이지
회원가입
심화 요구사항
공통
랜딩 페이지
로그인, 회원가입 페이지 공통
스크린샷