選択中のキャラが一番上に表示されるようにした

[tunamaguro]

内容

キャラ選択ダイアログを開いた時、現在のキャラがリストの一番上に来るようにしました。
これにより、ダイアログ下側のキャラをスクロールせずにスタイル変更できるようになります。

関連 Issue

ref #897

スクリーンショット・動画など

• 例1
  

• 例2
  

その他

実装やクラス名が非常に残念な感じになってしまいました。お伝えいただければ修正いたします。

クラスを出し分けしているところは下にあったものをコピペしたので変数にまとめたほうが良いかもしれません。

[y-chan]

LGTM！

なるほど、CSSだけで制御できるのですね、これは初耳でした...!
TS側でキャラクターリストを操作することなく実装できていて、とても綺麗だと思いました！

[Hiroshiba]

ほぼLGTMです！！
命名とロジックに関してコメントしてみました！！

以下ちょっと蛇足です。

CSSで解決するのはとても良いアイデアだなと思いました！
でもflexを足してorderで制御するとなると、quasarの仕様が変わったときに気づかないうちにバグっていそうだなと思いました。（こういうときにUIフレームワークは不便ですね。。）

まあそれでもquasarの仕様を捻じ曲げている部分は他にいくらでもあるし、別に良いのかなと判断しました。
もし将来quasarのバージョンを上げたとき、この仕様がなくなってないかを確認したいですね･･･！

[tunamaguro]

レビューありがとうございます!

上でご指摘いただいたように流用可能にするため、クラス出しわけで使っていたロジックを関数にまとめました。
私の実力だとこれが限界でした💧。

冷静に考えると、選択中のキャラが一番上に来たとしても下のような場合には結局遠いままだということに気づきました😇。

何かしらしたほうが良いとはとは思いますが、何もアイデアがでてきません。

sevenc-nanashi さん、issueをクローズするように設定してくださりありがとうございます!

[Hiroshiba]

選択中のキャラが一番上に来たとしても下のような場合には結局遠いまま

これはもう仕方ない感じしますね。。。 😇
キャラ一覧の縦幅を縮めるか、はたまたフィルタリングや可能にするか･･･。

とりあえず頂いたプルリクは今の形が良さそうで、取り組むにしても別タスクかなと思いました！

[Hiroshiba]

LGTM！！

混乱させる事を言ってしまってすみませんでした！！
PRありあとうございました、またぜひお待ちしています･･･････！！！