search

macmeals / todo_context_typescript

TodoリストのTypeScript化
0 stars 0 forks source link

emotionによるCSSやコンポーネントの定義箇所について #11

Closed mizushima1226 closed 2 years ago

mizushima1226 commented 2 years ago

https://github.com/macmeals/todo_context_typescript/blob/6c14d0727e31ac5aa52462bffd6349c21eb7dba0/src/component/pages/TodoList.tsx#L20-L61

↑のように、TodoListコンポーネント内でemotionでcssやコンポーネントの定義をしていますが、 Reactのステートに依存しない値はコンポーネント定義の外側に定義するのが良いです

特に、StyledListsはTodoList内のステートが変更されるたびに、StyledListsコンポーネントそのものの再計算が行われるので、思いもよらないバグを引き起こす原因となります（現状、問題なく動作していると思います）

macmeals commented 2 years ago

マージしましたので、こちらクローズいたします！