search

kuma0328 / todolist

0 stars 0 forks source link

コメント #1

Open mahiro72 opened 2 years ago

mahiro72 commented 2 years ago

タイトルとかはコンポーネント化すると、より見やすいかと思います！ https://github.com/kuma0328/todolist/blob/60b590587a208e20c232538211a0ea072b12961a/src/App.js#L6

mahiro72 commented 2 years ago

自分の知識不足ですみません。 こちらのファイルはどういった目的で使ってますか？👀 https://github.com/kuma0328/todolist/blob/60b590587a208e20c232538211a0ea072b12961a/src/components/index.js#L1

mahiro72 commented 2 years ago

buttonをすべて共通のコンポーネントにすると拡張性も高くなるし、統一性もでるので良いかと思います！

https://github.com/kuma0328/todolist/blob/60b590587a208e20c232538211a0ea072b12961a/src/components/Todolists.jsx#L36

mahiro72 commented 2 years ago

変数の書き方は統一した方が見やすくなるかもです！ xxx_yyyみたいなスネークケースか xxxYyyyのようなキャメルケースなど

https://github.com/kuma0328/todolist/blob/60b590587a208e20c232538211a0ea072b12961a/src/components/Todolists.jsx#L8

mahiro72 commented 2 years ago

細かいですが... https://github.com/kuma0328/todolist/blob/60b590587a208e20c232538211a0ea072b12961a/src/components/Todolists.jsx#L1

Todolistの最後のsはなくてもよいかもです！

mahiro72 commented 2 years ago

Tailwind 導入しているので、デザインもいろいろ触ってみるとよいかもです👀

https://tailwindcomponents.com/component/todo-list-app

この辺とか参考になると思います！