[조수빈] Sprint2#80
Hidden character warning
Conversation
|
실수로 close를 눌러서 실패 메세지가 뜨는 것 같습니다. |
baeggmin
left a comment
baeggmin
left a comment
There was a problem hiding this comment.
코드는 전반적으로 깔끔하게 잘 작성해주셨습니다. 👍
다만 현재 ⛔ node_modules 가 같이 올라왔어요.
node_modules/ 디렉토리는 수천 개의 라이브러리 파일이 포함된 의존성 디렉토리입니다. 해당 디렉토리는 용량도 크고, 프로젝트마다 환경도 달라서 절대 Git에 올려선 안 됩니다.
반드시 아래와 같이 .gitignore 파일을 만들어 추가해주세요.
# .gitignore 파일 최상단에 아래 내용 추가
node_modules/
또한 이미 올라간 node_modules 는 아래와 같은 절차로 제거 부탁드립니다.
# Git 캐시에서 삭제
git rm -r --cached node_modules
git commit -m "Remove node_modules from repo"
참고: node_modules는 package.json 및 package-lock.json만 있으면 npm install로 누구나 동일하게 재설치 가능합니다. 향후 커밋 시 주의 부탁드립니다!
| background-color: #ffffff; | ||
| } | ||
|
|
||
| :root { |
| </a> | ||
| </header> | ||
| <main> | ||
| <form class="auth-form"> |
There was a problem hiding this comment.
form 구현에 필요한 시맨틱 태그 (form, label, input 등) 를 잘 활용해주셨네요! 👏
| <i class="fa-solid fa-eye eye-icon"></i> | ||
| </div> | ||
|
|
||
| <button type="submit" class="auth-submit">로그인</button> |
There was a problem hiding this comment.
type="submit" 도 꼼꼼히 추가해주신 점 좋습니다! 👍
| <div class="simple-login-container"> | ||
| <p class="simple-login-desc">간편 로그인하기</p> | ||
| <a href="https://www.google.com/"> | ||
| <img src="public/images/google_icon.png" class="simple_login_img" alt="간편 로그인 이미지"> |
There was a problem hiding this comment.
alt="간편 로그인 이미지" 가 반복되고 있는데요, 접근성 측면에서 더 구체적인 alt를 설정해주시는게 좋습니다.
예시
<img src=".../google_icon.png" alt="Google 계정으로 로그인">
<img src=".../kakao_icon.png" alt="카카오 계정으로 로그인">
There was a problem hiding this comment.
reset.css 로 파일 분리를 해주신 것은 좋지만, 더 많은 초기화 규칙 포함되면 좋겠습니다. (예: ul, a, img, button)
There was a problem hiding this comment.
이미지는 꼭 직관적인 이름으로 수정해주세요! 그래야 추후 협업 시 헷갈리지 않고 사용할 수 있습니다.
|
.gitignore 파일 추가 후 다시 push 했습니다. |
2 participants
기본 요구사항
로그인 페이지
회원가입 페이지
심화 요구사항
멘토에게