Feat/home screen/buttons #18
Conversation
|
@KikyoNanakusa prettier でコードをフォーマットしてコミットしてほしいです。本当はpush時にCIでフォーマッタが走るべきなんだけど、整備してなくてすみません。 |
|
@h-takeyeah これで適用できてますかね |
|
@KikyoNanakusa できてます！ |
There was a problem hiding this comment.
- GameStartButtonなどのボタンは汎用的なコンポーネントじゃないのてmoleculesに分類すべき
この記事が非常に良かった
https://note.com/tabelog_frontend/n/n4b8bcb44294c
- loginページを作る必要は無いと思っていて、homeページで状態管理で学籍番号の入力フォームを表示するぐらいで良さそう
|
や、loginページの有無に関しては任せます。 |
|
HomeMenuContainerという名前からはMenuという単語があることから、私はmoreculeみを感じるんですけどorganismなんですかね。変更しろって意味ではなくてどうなんだろうかっていう。あとログインページは pagesで完結してるけどホームの方はorganismを呼び出していてpageだけでは完結してないという違いがあるけど、これは意図的？ |
HomeMenuContainerはボタンを組み合わせた一つのUIの単位として作ったつもりなのでorganismに置いたんですがどうなんでしょう。いかんせんAtomic Designに疎く.... どちらかというと命名の問題な気もします LoginPageは廃止してモーダルにしました。 |
|
なるほどー！Atomic Design 何も分からないので、これ以上はやめておきます！ |
チケットへのリンク
https://github.com/orgs/su-its/projects/1/views/1?pane=issue&itemId=57422910
やったこと
ホームページにボタンを配置
各ページへの遷移を実装
やらないこと
ログアウト時の処理の実装
できるようになること（ユーザ目線）
ホームページの表示
できなくなること（ユーザ目線）
なし
動作確認
ブラウザで確認
その他
フロント不慣れなので気になるところがあれば厳しめにお願いします
サーバーサイドコンポーネントだとAppRouter使えないって言われたので
use clientしちゃってるんですけどこれはいいんでしょうか