Daram0o0 / ToDoList

0 stars 0 forks source link

코드리뷰 #2

Open ptk5758 opened 1 year ago

ptk5758 commented 1 year ago

https://github.com/Daram0o0/ToDoList/blob/4c1fefa1e3f9478ebb27fb97b6abc55e23e5bb26/src/App.js#L25-L29

해당 변수들이 어떤 역할을 하는지 주석이 있으면 좋겠어요

ptk5758 commented 1 year ago

https://github.com/Daram0o0/ToDoList/blob/4c1fefa1e3f9478ebb27fb97b6abc55e23e5bb26/src/App.js#L31-L34

이 함수는 어떤함수인지 설명 해주실수 있을까요?

whisper74541 commented 1 year ago

https://github.com/Daram0o0/ToDoList/blob/4c1fefa1e3f9478ebb27fb97b6abc55e23e5bb26/src/App.js#L49-L62 혹시 여기서 true 가 어떻게 받아오는건가요??

CaKlan commented 1 year ago

https://github.com/Daram0o0/ToDoList/blob/4c1fefa1e3f9478ebb27fb97b6abc55e23e5bb26/src/App.css#L66-L77

버튼들에

cursor : pointer;

추가해서 버튼인지 인식하게 하면 좋을 것 같아요

Daram0o0 commented 1 year ago

https://github.com/Daram0o0/ToDoList/blob/83b7cc69c8de410ead90cb129645f79a7ade1733/src/App.js#L49-L51

반복되는 구문들 함수로 표현