-
Notifications
You must be signed in to change notification settings - Fork 35
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
[엄세환] sprint4 #137
The head ref may contain hidden characters: "Basic-\uC5C4\uC138\uD658-sprint4"
[엄세환] sprint4 #137
Conversation
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.
세환님 너무 잘하셨어요!!
다음번이 기대가 됩니다 :)
<meta name="viewport" content="width=device-width, initial-scale=1.0" /> | ||
<meta property="og:title" content="판다마켓" /> | ||
<meta property="og:url" content="https://panda-market-sample.netlify.app" /> | ||
<meta property="og:type" content="website" /> |
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.
각 페이지 마다 메타 tag 를 잘 넣어주셨군요 너무 잘하셨습니다
--gray-50: #f7f7f8; | ||
|
||
/* Primary color */ | ||
--blue: #3692ff; |
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.
global css 에 색상 변수를 정의해서 쓰는거는 매우매우 좋은습관입니다.
|
||
<body> | ||
<div class="auth-container"> | ||
<a href="/" class="logo-home-button" |
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.
여기 a 태그가 잘못 닫혀있네요.
앞 태그가 / 를 포함하고 있다면 닫아주지 않아도 됩니다!
passwordInput.parentNode.parentNode.appendChild(passwordErrorMessage); | ||
|
||
// 입력 필드 유효성 검사를 수행하는 함수입니다. | ||
function validateInputs(showErrors = false) { |
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.
특정 동작을 함수로 분리하는 습관은 매우 좋은 습관입니다!
|
||
// 이메일 유효성 검사 | ||
isValid = | ||
validateEmail(emailValue, showErrors, emailInput, emailErrorMessage) && |
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.
이렇게 할당을 반복하는 경우 배열에 담아서 every 메서드를 활용하는 방법도 있으니 참고해주시면 좋습니다!
validateEmail, | ||
validatePassword, | ||
togglePasswordVisibility, | ||
}; |
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.
벌써 모듈을 사용하고 계시군요! 👍
// 이메일 유효성 검사를 수행하는 함수입니다. | ||
function validateEmail(emailValue, showErrors, emailInput, emailErrorMessage) { | ||
const emailRegex = /^[^\s@]+@[^\s@]+\.[^\s@]+$/; | ||
let isValid = true; |
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.
함수 맨 위에 어떤 값을 반환할지 선언해주는 습관 👍
단순히 result 라는 이름을 써줘도 큰 문제 없습니다!
함수를 마지막에 꼭 안전하게 return 이 되게끔하는 생각과 습관을 유지해주셔야해요!
return isValid; | ||
} | ||
|
||
// 비밀번호 입력 필드의 표시/숨김을 토글하는 함수입니다. |
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.
단순하게 함수의 동작만을 설명하는 주석은 크게 필요성이 높지 않습니다. 오히려 업데이트되지 않은 주석이 헷갈리게 만들지도 몰라요!
주석은 보다 복잡한 함수에 추가적인 정보를 담거나, docs를 위한 정보를 나열하거나 나의 의도가 필요할때 담아주면 좋습니다!
요구사항
기본
netlify배포링크입니다.
로그인
회원가입
심화
-[x] 눈 모양 아이콘 클릭시 비밀번호의 문자열이 보이기도 하고, 가려지기도 합니다.
-[x] 비밀번호의 문자열이 가려질 때는 눈 모양 아이콘에는 사선이 그어져있고, 비밀번호의 문자열이 보일 때는 사선이 없는 눈 모양 아이콘이 보이도록 합니다.
주요 변경사항
스크린샷
멘토에게