-
Notifications
You must be signed in to change notification settings - Fork 17
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
[유호은] sprint3 #33
Merged
dennis2311
merged 14 commits into
codeit-sprint-fullstack:basic-유호은
from
HoeunRyu:basic-유호은-sprint3
Jan 2, 2025
The head ref may contain hidden characters: "basic-\uC720\uD638\uC740-sprint3"
Merged
[유호은] sprint3 #33
dennis2311
merged 14 commits into
codeit-sprint-fullstack:basic-유호은
from
HoeunRyu:basic-유호은-sprint3
Jan 2, 2025
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
dennis2311
reviewed
Jan 2, 2025
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.
정말 잘 하셨습니다. 정말로 제가 더 건드릴 게 없네요.
- className의 네이밍 규칙을 통일한 것
- 직관적인 함수 이름을 사용한 것
- 로직의 주요 지점마다 주석을 적절히 사용한 것
- commit 을 적절하게 나누어 처리한 것
등의 지점에서 많이 공부하고 섬세하게 작업하신 게 돋보입니다.
두 가지만 팁을 드리면:
- JavaScript 에는 JSDoc 이라는 주석 체계가 존재합니다. /** {주석내용} */ 형태로 사용할 수 있는데, 함수나 변수 위에 사용하면 해당 다른 곳에서 해당 객체를 사용할 때 마우스 호버하여 해당 객체에 대한 힌트를 얻을 수 있습니다. 이것까지 활용하신다면 코드가 한 층 더 견고해 질 거예요.
- 깨알 UX지만, email, password 입력에 이벤트 리스너를 지정할 때 keydown 이벤트에 리스너를 설정하면 좀 더 자연스러운 UX가 됩니다. 지금은 모두 입력한 후 blur를 시켜야 검증이 반영되기 때문에 다소 어색한 감이 있습니다.
- 만약 이것이 키 입력 하나하나마다 서버 호출을 하는 것이 비효율적이라는 판단 하에 일부러 그렇게 하신 것이라면, 'debounce' 라는 개념을 찾아보시면 좋을 것 같습니다.
잘 하셨습니다.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
배포 링크
https://pandamarket-hoeun.netlify.app/
요구사항
기본
심화
랜딩 페이지
로그인, 회원가입 페이지
주요 변경사항
로그인, 회원가입 페이지 유효성 검증, UI컨트롤 JS 추가
랜딩 페이지 메타태그 추가, 모바일 사이즈 미디어쿼리 추가
그 외 네이밍 통일, 오타 수정, 공통 로직 모듈 분리 등 자잘한 디테일 작업
유효성 검증 조건은 일반적으로 많이 쓰이는 조건을 참고하였습니다.
로그인 페이지에서 요구하는 모달 메시지는 "비밀번호가 일치하지 않습니다" 인데, 이메일이 데이터베이스에 없는 경우에도 모달 메시지가 열리므로 "이메일 또는 비밀번호가 잘못 되었습니다”로 임의 수정하였습니다.
스크린샷
랜딩 페이지 태블릿 사이즈
랜딩 페이지 모바일 사이즈
메타태그
로그인 페이지
회원가입 페이지
멘토에게