-
Notifications
You must be signed in to change notification settings - Fork 33
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 #51
The head ref may contain hidden characters: "Basic-\uAE40\uC601\uD638-sprint4"
[김영호] sprint4 #51
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.
영호님, 함수도 잘 나눠주시고 변수나 함수명도 대체로 잘 지어주셔서 보기가 편했습니다. :) 고생 많으셨습니다~!
유호성 검사들은 focusout보다 input이 더 나은 서택지인가요?
-> 개인적이 느낌으론 input이 좀 더 많이 쓰이는 거 같기는 합니다. 다만, 정해진 것은 없습니다. ㅎ 요구 사항이나 서비스 성격에 따라 달라질 수도 있을 거 같아요!
} | ||
|
||
button.disabled = !isFormValid; | ||
button.classList.toggle("valid", isFormValid); |
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.
이 코드는 제거해도 되겠네요! 🤣
요구사항
기본
로그인
회원가입
심화
주요 변경사항
스크린샷
멘토에게