Conversation
bk-git-hub
left a comment
bk-git-hub
left a comment
There was a problem hiding this comment.
벌써 제출해주시다니 엄청 빠르시군요~! 워낙 잘하시는 만큼 리뷰 드릴 부분이 많진 않은 것 같습니다.
There was a problem hiding this comment.
지금 보이는 바로는 App.css가 따로 사용되는 곳이 없는 것 같습니다! 불필요한 css파일은 지우셔도 될 것 같아요
There was a problem hiding this comment.
Tailwind CSS는 현재 공식 문서 기준으로 v4 사용을 기본으로 안내하고 있습니다. 저희 과제에서는 3버전을 쓰셔도 무방하긴 한데 4버전이 초기 설정 및 커스텀 테마 선언 같은게 더 단순해져서 한번 알아보시면 좋을 것 같아요!
조금 v4를 홍보를 하자면 v4부턴 tailwind.config.js를 다루지 않습니다. 또한 css파일에서 임포트 하는 방법도
@import "tailwindcss"; 한줄로 끝나게 됩니다
There was a problem hiding this comment.
전에 프로젝트에서 4버전을 사용했었을 때 오류가 많았어서 3을 계속 쓰던 습관이 남아있었던 것 같네요. 이번엔 그럼 v4를 사용해보도록 하겠습니다..!! 👍🏻
| @@ -0,0 +1,31 @@ | |||
| const cardStyle = { | |||
There was a problem hiding this comment.
Tailwind를 사용하시면서 inline style도 함께 쓰신 이유가 있는지 궁금합니다!
There was a problem hiding this comment.
JSX + Curly Braces를 의식하다보니 스타일들을 다 inline으로 했던 것 같네요..ㅎ 리팩토링할땐 tailwind로 다시 바꿀 예정입니다!
|
|
||
| export default function TodoList() { | ||
| return ( | ||
| <div className="flex flex-col items-start gap-[16px] self-stretch"> |
There was a problem hiding this comment.
리스트를 매핑하는 내용은 다음 주차에 다룰 예정이긴 하지만, 현재 TodoList컴포넌트를
- 로 감싸고 TodoCard는
- 로 작성해서 조금 더 시맨틱한 구조로 표현해볼 수 있을 것 같아요.
2 participants
✅ 제출 정보
✅ 체크리스트 (필수)
main이 아니라 내 GitHub 핸들 브랜치(<handle>)다<handle>-week-xx형식이다🧩 구현 내용 요약
❓ 궁금한 점