Conversation
seung-in-Yoo
left a comment
seung-in-Yoo
left a comment
There was a problem hiding this comment.
수고하셨습니다! 고민을 되게 많이 한 흔적이 보이는것 같아요
| align-items: flex-start; | ||
| min-height: 100vh; | ||
| margin: 0; | ||
| padding-top: 50px; |
There was a problem hiding this comment.
css를 구성할때 직접적인 px로 세부요소를 지정한다면 나중에 유지보수가 어렵고 통일된 디자인을 가져가기 힘들다고 생각해요. px 단위로 css를 구성하는것보다 rem/em 단위나 Tailwind의 spacing scale를 활용하는것도 한번 찾아보시면 좋을것 같아요!
| } | ||
|
|
||
| export function getStorage() { | ||
| const saved = localStorage.getItem('todos') |
There was a problem hiding this comment.
todos라는 키가 하드코딩 되어 있는것 같은데 다른 키를 쓸 경우 재사용 불가해서 개선 방안을 찾아보셔도 좋을것 같아요!
|
check한 값에 따라서 textDecoration을 적용하여 완료된 리스트를 시각적으로 구분한게 직관적이고 좋았습니다. |
|
로컬 스토리지에 기본으로 주는 데이터를 storage.js파일에 넣어서 상대적으로 useTodos.js가 깔끔한 것 같아요 |
|
디자인이 깔끔하고 보기 좋네요 수고하셨습니다 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
React 기반의 TodoApp 만들기
이번 주 팀 활동 목표는 React 기반의 TodoApp 만들기이다. 각자가 기능을 구현한 후, GitHub에 PR(Pull Request)을 생성하고 팀원들과 코드리뷰를 통해 서로의 코드를 검토하고 피드백하는 경험을 쌓는 것이 핵심이다.
협업 도구(GitHub, 노션, 슬랙)를 적극적으로 활용해 개발 → 리뷰 → 피드백 전 과정을 실습해보는것이 목표다.
시작화면
전체 기능 요약