Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

7회차 과제 - 김진현 #11

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

JinhyunMSQ
Copy link
Member

@JinhyunMSQ JinhyunMSQ commented May 14, 2024

수업 때 배운 내용의 코드에서 5가지 변경사항을 추가 하였습니다.

  1. 할 일 아이템의 상태를 확장하기
    먼저, 각 할 일 아이템에 완료 여부를 나타낼 수 있는 상태를 추가해야 하므로 App.js에서 각 할 일 객체에 completed 속성을 추가하였습니다.
    (기본값은 false로 설정함)

  2. 완료 처리 함수 추가
    App.js에 할 일 완료 상태를 토글하는 함수를 추가했습니다.

  3. TodoListItem 컴포넌트 수정
    완료 버튼(botton onClick)을 추가하고, 완료된 아이템에 스타일을 적용했습니다.

  4. TodoList 컴포넌트 수정
    TodoList 컴포넌트에서 toggleComplete 함수를 TodoListItem에 전달했습니다. (export)

  5. App.js에서 toggleComplete 전달
    App.js에서 TodoList 컴포넌트로 toggleComplete 함수를 전달했습니다.

Copy link

@petitmatinz petitmatinz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

toggleComplete 함수로 각 항목의 완료 여부를 토글하여 상태를 업테이트를 잘 구현하셨고,
todolist.map 을 사용하여 각각의 할일 항목을 렌더링 하여 깔끔하게 잘 작성하신것 같습니다.
전체적으로 코드가 깔끔하고 잘 구성되어있고, 코드를 이해하기 쉽게 해주신것 같습니당.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants