-
Notifications
You must be signed in to change notification settings - Fork 17
[이서정] Sprint22 스프린트 미션 2 #26
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
base: Basic-이서정
Are you sure you want to change the base?
[이서정] Sprint22 스프린트 미션 2 #26
The head ref may contain hidden characters: "Basic-\uC774\uC11C\uC815-sprint2"
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.
안녕하세요, 서정님!
이번 미션 역시, 구현을 정말 잘 해주셨습니다.
크게 피드백 드릴만한 요소는 없는 것 같아요.
코드 포맷을 수정한 부분이 꽤 보이네요. 추후 이런 수정으로 인한 커밋이 늘면 안 되겠죠.
prettier를 활용해서 자동 포맷팅이 되도록 만들면 좋을 것 같아요. 그럼 더 이상 코드 포맷을 신경쓰지 않고, 일관된 코드 스타일을 유지할 수 있어요. 🎨 (Tab Size는 지금처럼 2로 설정하시는 것을 추천드려요.)
자잘한 피드백은 코멘트를 참고해 주세요. 😊
변수명을 축약해서 쓰는 것과 풀어서 쓰는 것 중 어떤 것이 더 좋을지 궁금합니다.(ex. btn vs button/ desc vs description)
: 예로 들어주신 변수명처럼 충분히 자주 사용되거나, 맥락을 보고 알아볼 수 있다면 괜찮습니다. 중요한 건 btn으로 작성했다면, 일관되게 작성하는 것이 중요해요. 뭐가 더 나은지는 개인 프로젝트에서는 크게 상관이 없고, 팀 프로젝트 시 축약보다는 전체적으로 명확하게 작성하는 쪽을 더 선호한다면 풀어서 작성하는 것이 좋은거죠. 🙂
이번 미션도 정말 고생하셨습니다. 👏 미션 3까지 빠르게 가시죠! 🚀
| @@ -0,0 +1,159 @@ | |||
| body { | |||
| font-family: 'Pretendard', sans-serif; | |||
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.
가변 글꼴의 font-family 속성값은 'Pretendard Variable'이에요.
| body { | ||
| font-family: 'Pretendard', sans-serif; | ||
| } | ||
| .auth-container{ | ||
| max-width: 640px; | ||
| margin: 0 auto; | ||
| } | ||
| .auth-login-layout { |
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.
전체 구현은 너무나 잘 해주셨기 때문에, 코드 작성 포맷에 더 신경써주시면 좋아요. 줄바꿈 및 공백을 수정해주세요!
| body { | |
| font-family: 'Pretendard', sans-serif; | |
| } | |
| .auth-container{ | |
| max-width: 640px; | |
| margin: 0 auto; | |
| } | |
| .auth-login-layout { | |
| body { | |
| font-family: 'Pretendard', sans-serif; | |
| } | |
| .auth-container { | |
| max-width: 640px; | |
| margin: 0 auto; | |
| } | |
| .auth-login-layout { |
| </div> | ||
|
|
||
| <div class="auth-field"> |
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 for="password_check" class="auth-label">비밀번호 확인</label> | ||
| <div class="auth-password-container"> | ||
| <input | ||
| id="password_check" |
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.
id 속성값의 네이밍은 snake_case로 구분 하셨군요. 좋습니다! 👍
id의 경우 보통 JS로 제어하는 경우가 많아, camelCase로 작성하기도 합니다. 🤓
배포 링크
https://snazzy-baklava-3e5b7c.netlify.app/
진행한 미션 버전
미션 요구사항
공통
심화
주요 변경사항 및 학습 포인트
pages,css폴더로 재구성메인 페이지 이미지 포맷 변경 (PNG → SVG)
Pretendard 가변 다이나믹 서브셋 적용
주강사에게
집중적으로 봐줬으면 하는 부분
해결하지 못한 문제 / 궁금한 점