検索結果からの防具除外を変更 #69
検索結果からの防具除外を変更 #69
Conversation
1.検索結果の各装備にEボタンを表示(Excludeボタン)
A.`装備なし`の時は、Eボタンのスタイルに`visibility: hidden`を設定します。
`装備なし`以外の部位と段差が出来ないようにします。
2.Eボタンを押下すると防具設定モーダルが表示され、初期値としてEボタンを押下した装備のグループ名をテキストボックスに入力してフィルタ処理を実施
A.グループ名からはEX, α, β, γなどを取り除いて、下位、上位、マスターの区別なくリストに表示する。
シミュレータ使用者は現状作成できない装備や作成が難しい装備を除外したいと思うので、下位、上位、マスターを表示する。
B.Eボタンが押下された装備が分かるように、グループ名、部位名、チェックボックスのセル背景を変更する。
チェックボックスの背景を変更しようと思ったが難しそうなので諦めました。
C.テキストボックスに入力されたグループ名を変更すれば、通常の防具設定と同じように使用できます。
グループ名を変更してもグループ名、部位名、チェックボックスのセル背景は色が付いたままになります。
3.検索結果の各装備リンクをクリックするとでるモーダルにある除外チェックボックスは残してあります。
|
プルリクエストありがとうございます。 プルリクエストとして頂いていますが、一旦、機能要望として受け取らせていただきます。
というのも、頂いた説明が実装に関してばかりで上記の内容があまり書かれておらず、次の判断ができません。
なお、上記の判断（ディスカッションの内容含む）によっては、このプルリクエストをcloseしたり、ソースコードの修正をお願いしたりする可能性がありますので、ご了承ください。 お手数ですが、何卒よろしくおねがいします。 |
|
メッセージありがとうございます １. 欲しい機能 ２. なぜその機能が欲しいのか 以上、よろしくおねがいします。 |
|
確かに、攻略中ですと使いづらそうですね…
この2つの課題を、yousuke-fさんは「Eボタンの追加」と「防具設定ポップアップの拡張」で対応したのだと思います。
スクショした画像を重ねているだけで、まだ未作成ですが、こんな感じです。 上記を実装した場合、この対応方針で解決できそうでしょうか。 |
|
メッセージありがとうございます。
今回の修正とは別に考えていたことなのですが、ArmorTable.tsxの操作に関する修正を考えていました。
「1, 2, 3」については、たとえば装備が１つだけ除外されている時にどのような表示にするのかと処理として「表示中装備を全て除外」するのか「表示中装備を全て非除外」にするのかという問題があります。 私が添付した画像の |
|
上記に記載忘れました。
以上、よろしくおねがいします。 |
|
承知しました。 |
初めてプルリクエストを送ってみました。
手違いが無いようにしたいと思います。
1.検索結果の各装備にEボタンを表示(Excludeボタン)
A.
装備なしの時は、Eボタンのスタイルにvisibility: hiddenを設定します。装備なし以外の部位と段差が出来ないようにします。2.Eボタンを押下すると防具設定モーダルが表示され、初期値としてEボタンを押下した装備のグループ名をテキストボックスに入力してフィルタ処理を実施
A.グループ名からはEX, α, β, γなどを取り除いて、下位、上位、マスターの区別なくリストに表示する。
シミュレータ使用者は現状作成できない装備や作成が難しい装備を除外したいと思うので、下位、上位、マスターを表示する。
B.Eボタンが押下された装備が分かるように、グループ名、部位名、チェックボックスのセル背景を変更する。
チェックボックスの背景を変更しようと思ったが難しそうなので諦めました。
C.テキストボックスに入力されたグループ名を変更すれば、通常の防具設定と同じように使用できます。
グループ名を変更してもグループ名、部位名、チェックボックスのセル背景は色が付いたままになります。
3.検索結果の各装備リンクをクリックするとでるモーダルにある除外チェックボックスは残してあります。
以上、宜しくお願いします。