Conversation
|
#F5F5F5 배경색깔이 일부 pixel에 적용이 안 되는 문제를 어떻게 해결할 수 있는지 여쭤보고 싶습니다! |
bk-git-hub
left a comment
bk-git-hub
left a comment
There was a problem hiding this comment.
기성님 수고많으셨습니다!
리뷰가 늦어저셔 죄송해요
전반적으로 잘 해주셨습니다. 특히 시멘틱 태그를 사용하려고 고민하신 흔적이 보여서 좋았어요
파일별로 다듬어보면 좋을 것 같은 부분들 코멘트로 남겨두었습니다
이건 다음 과제 제출때부터 연습해보시면 좋을 것 같아요
https://www.notion.so/Git-Commit-Guide-327793efb80180bcb8f9c357c580a48d
| @@ -0,0 +1,17 @@ | |||
| // src/components/TodoCard.tsx | |||
| import React from 'react'; | |||
There was a problem hiding this comment.
현재 React import는 이 파일에서 사용중이지 않는것 같아요
| @@ -0,0 +1,21 @@ | |||
| // src/components/TodoList.tsx | |||
| import React from 'react'; | |||
| const todo1 = "리액트 공식문서 읽기"; | ||
| const todo2 = "알고리즘 문제 풀기"; | ||
| const todo3 = "운동 30분 하기"; | ||
| const todo4 = "프로젝트 회의 준비";<q></q> |
There was a problem hiding this comment.
빈 <q></q> 태그가 남아 있는데, 의도하지 않은 코드라면 제거해주시는 것이 좋겠습니다
| font-size: 24px; | ||
| font-weight: 700; | ||
| color: #1F2937; | ||
| font-family: Pretendard; |
There was a problem hiding this comment.
CSS에서 Pretendard 폰트를 사용하고 있는데, 현재 index.html에는 폰트를 실제로 불러오는 설정이 보이지 않습니다. 폰트 임포트를 확인해보시고 실제 폰트가 잘 적용되고 있는지 확인해보시면 좋을 것 같아요
There was a problem hiding this comment.
src/assets/App.css 파일이 비어 있는 상태로 남아 있는데, 실제로 사용하지 않는 파일이라면 정리해두셔도 좋을 것 같습니다.
|
|
||
| return ( | ||
| <main className="todo-list"> | ||
| <TodoCard content={todo1} /> |
There was a problem hiding this comment.
현재 할 일 목록은 의미상 리스트이기 때문에, div 구조 대신 ul / li 구조로 작성하면 조금 더 시맨틱한 마크업이 될 것 같습니다.
| @@ -0,0 +1,14 @@ | |||
| // src/components/TodoHeader.tsx | |||
| import React from 'react'; | |||
There was a problem hiding this comment.
사용하지 않는 React import 들은 제거해주셔야합니다.
2 participants
✅ 제출 정보
✅ 체크리스트 (필수)
main이 아니라 내 GitHub 핸들 브랜치(<handle>)다<handle>-week-xx형식이다🧩 구현 내용 요약
week1
❓ 궁금한 점
감사합니다.
