Conversation
Deploying utcode-learn with
|
| Latest commit: |
823e766
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://01850df6.utcode-learn.pages.dev |
| Branch Preview URL: | https://react-session-declarative-ui.utcode-learn.pages.dev |
There was a problem hiding this comment.
let state → const stateの方が適切かと思います。
There was a problem hiding this comment.
コードのフォーマットがPrettierに準拠していないため、Prettierのフォーマットに準拠したフォーマットとすると良いかと思います。
There was a problem hiding this comment.
細かいですが、特に理由がなければ、removeよりもaddを使用した方がわかりやすいかと思います。というのも、基本的にremoveよりもaddの方が実装しやすくより単純だからです。
There was a problem hiding this comment.
Todo List → ToDoリスト
などにした方が良いかと思います。（TodoではなくToDoの方がよく使われる表記であり、さらに日本語にした方がわかりやすい）
There was a problem hiding this comment.
ToDoリストのUIの図のレイアウトが少し崩れているので、グリッドを使用して揃えるとよいかと思います。
There was a problem hiding this comment.
全体的に不必要に文字サイズが小さい（あるいは図の横幅が不必要に大きい）ところがあるので、適切な大きさにすると良いかと思います。また、同じ意味論を持つテキストに対して別の文字サイズが使われているため、それぞれのテキストの意味に応じた文字サイズにすると良いかと思います。（タイトルは〇〇px, 地の文は〇〇pxなどのように揃えるイメージです。）
There was a problem hiding this comment.
状態からUIへ向かう矢印の大きさと位置が左右で異なるので、揃えるとよいかと思います（グリッドを用いて合わせるとよいかと思います）。また、矢印の始点と終点の位置も見やすいように調整するかと良いと思います。
There was a problem hiding this comment.
removeを表す矢印が左側の箱（すなわち、状態とUIの両方を含んだもの）から右側の箱に向かっており、左側の状態 + UIから右側の状態 + UIに向かう矢印となってしまっています。状態から状態へと向かう矢印という意味を表したいはずなので、食い違ってしまっているように思います。
There was a problem hiding this comment.
テキストでの説明に合わせて、
render() → render関数
のようにするとよいかと思います。
2 participants
変更点
画像 v1
画像 v2