ohshimatani / groupA_AcceLance

2 stars 0 forks source link

コレクション画面の漢字を学年ごとに表示する #162

Closed Parasuatene closed 2 years ago

Parasuatene commented 2 years ago

issue #160

Parasuatene commented 2 years ago

画像の切り替えをスクリプトで行うように変更。 ひとまず漢字ボタンを押すと、学年ボタンの画像が変わる問題を解決した。

TODO:スクリプトファイルの命名規則

今回ボタンに取り付けるスクリプトとして、GradeSelectButton、ボタンの選択状態を管理するスクリプトとしてGradeSelectButtonControllerを作成したが、他の個所ではButtonに取り付けるスクリプトであっても全てControllerとなっている。今一度命名規則を考え直してもいいかもしれない。

image

Parasuatene commented 2 years ago

次にやること

学年ごとの漢字を表示する前に、まずコレクション画面に表示する漢字セルに関するプログラムのリファクタリングを行う。

Parasuatene commented 2 years ago

実装完了

学年を切り替えるボタンを押すと、表示する漢字を変更できるようにした。

「すべて」を選択しているとき

image

「2年生」を選択しているとき

image

Parasuatene commented 2 years ago

ひとまず実装完了したので、月曜日にコードレビューをお願いし、問題がなければマージする予定です。

Parasuatene commented 2 years ago

レビューしてからマージする予定であったが、次のタスクに進みづらいので今回はマージすることにする。 変更内容については口頭で改めて共有する。