Conversation
kotaeho
requested review from
withyj-codeit,
codeit-sprinter and
hoody-jellybean
and removed request for
withyj-codeit and
codeit-sprinter
There was a problem hiding this comment.
.github 디렉토리 내의 파일은 Github에서 사용할 템플릿, 워크플로우 등을 정의합니다.
템플릿은 그대로 두시고, PR에서 템플릿 양식만 가져와서 수정해주세요~!
There was a problem hiding this comment.
백터 이미지를 사용할 수 있는 아이콘 요소들은 svg 파일로 사용하시는 걸 추천합니다!
There was a problem hiding this comment.
각 화면 요소에 적절한 시멘틱 태그를 잘 사용해 주셨습니다. 👍
| <div> | ||
| <input id="password" type="password" placeholder="비밀번호를 입력해주세요"> | ||
| </div> | ||
| <input id="login_btn"type="submit" value="로그인"> |
There was a problem hiding this comment.
폼 제출 버튼 요소는 <button type='submit'/>을 사용하셔도 됩니다.
| /* | ||
| 전체 선택자(*) 및 a, button, img 태그에 대한 CSS는 새로운 웹 개발 프로젝트를 시작할 때 전역 스타일에 기본적으로 넣어주면 좋은 내용들입니다. | ||
| 브라우저 간의 스타일 차이를 최소화하고 예측 가능한 레이아웃을 위해 브라우저 기본 세팅을 일부 리셋해 준다고 생각하면 됩니다. | ||
| */ | ||
|
|
||
| * { | ||
| margin: 0; | ||
| padding: 0; | ||
| box-sizing: border-box; | ||
| } | ||
|
|
||
| /* 링크가 상위 요소 또는 설정된 기본 텍스트 색상과 동일하게 표현되길 원한다면 a 태그에 명시적으로 color: inherit을 지정해 줘야 합니다. */ | ||
| a { | ||
| text-decoration: none; | ||
| color: inherit; | ||
| } | ||
|
|
||
| button { | ||
| background: none; | ||
| border: none; | ||
| outline: none; | ||
| box-shadow: none; | ||
| cursor: pointer; | ||
| } |
There was a problem hiding this comment.
브라우저의 각 요소 별 스타일 초기화 속성을 잘 처리하셨네요.
직접 작성하셔도 되지만, 크로스 브라우징까지 대응하려면 resetCSS 또는 normalizeCSS를 적용하시는걸 추천드립니다.
There was a problem hiding this comment.
네 전에 멘토님 말씀해주신거 기억하고 있습니다. 말씀들으니 resetCSS 또는 normalizeCSS는 필수라고 생각합니다. 감사합니다
| font-size: 16px; | ||
| } | ||
|
|
||
| #footerMenu { |
There was a problem hiding this comment.
클래스 대신 ID를 사용하시는 이유가 있을까요?
| background-color: #ffffff; | ||
| border-bottom: 1px solid #dfdfdf; |
There was a problem hiding this comment.
색상은 CSS Custom property로 선언하셔서 사용하시면 더 좋을 것 같습니다.
위클리 과제에서 디자인 시스템 색상 파운데이션을 사용하기 좋도록 잘 정리해 놓으신걸로 알고 있습니다~!
|
PR 리뷰 요청 주신 내용하고, 배포된 사이트하고 차이가 좀 있는 것 같은데 한 번 확인 부탁드립니다~! |
요구사항
기본
심화
주요 변경사항
스크린샷
https://66b6c74acc7773b04ed9bcb6--weekly-kotaeho.netlify.app/
멘토에게