Conversation
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
There was a problem hiding this comment.
컴포넌트를 나누어보는 것도 좋은 시도입니다. 다만 현재 Card1~Card4는 구조가 거의 같고 내용만 달라서, 이후에는 TodoCard 하나만 만들고 재사용 가능하게 만들어보면 좋겠습니다. 2주차 읽을거리에 컴포넌트에 Props를 전달하는 방법에 대해 다루니 그걸 읽고 수정해보면 좋을 것 같아요!
There was a problem hiding this comment.
하드코딩된 정적 데이터를 별도의 상수로 관리하는건 매우 좋은 접근입니다!
다만 현재 TodoList.tsx는 JSX를 렌더링하는 컴포넌트라기보다 할 일 데이터를 담고 있는 파일에 가까워서, 이름과 역할이 조금 다르게 느껴질 수 있습니다. JSX를 사용하지 않기 때문에 확장자를 .tsx가 아닌 .ts를 사용해서 todoItems.ts정도로 하면 좋을 것 같아요. 그리고 상수명을 TODO_ITEMS정도로 바꾸면 깔끔해질 것 같아요
| <> | ||
| <div className="container"> | ||
| <TodoHeader /> | ||
| <div className='toDoList'> |
There was a problem hiding this comment.
현재 할 일 목록은 의미상 리스트이기 때문에, 이후에는 div 대신 ul / li 구조로 작성해보셔도 좋을 것 같아요.
| export function TodoHeader() { | ||
| return ( | ||
| <> | ||
| <div className="title"> |
There was a problem hiding this comment.
현재 헤더 영역을 모두 div로 작성해주셨는데, 제목 역할을 하는 부분인 만큼 이후에는 div보다는 h1이나 h2 같은 시맨틱 태그를 사용해보셔도 좋을 것 같아요. 구조의 의미가 더 분명해지고 마크업도 조금 더 자연스러워집니다.
2 participants
✅ 제출 정보
✅ 체크리스트 (필수)
main이 아니라 내 GitHub 핸들 브랜치(<handle>)다<handle>-week-xx형식이다🧩 구현 내용 요약