search

sleep0001 / cleaner-schedule

1 stars 0 forks source link

20240810 #26 #28

Closed RyoUchiyamaGenius closed 1 month ago

RyoUchiyamaGenius commented 1 month ago

@sleep0001 とりあえずPC画面ではOK（のはず、、）です。 レビューお願いします。

sleep0001 commented 1 month ago

@RyoUchiyamaGenius さん モードオフの時と2つ未満の時の挙動は確認できました。 モードオンかつ、2つ選択時に押下可能でアクションを持ったボタンに切り替える実装は含まれていますか？

RyoUchiyamaGenius commented 1 month ago

ボタンにdisabledつけただけなのでボタン押した時の処理は以前から何も変更してないです

sleep0001 commented 1 month ago

日付2つ選択されている時はabled それ以外はdisabledみたいな処理ってできますか？ 多分ボタンは2種差し替えになるかと思います

RyoUchiyamaGenius commented 1 month ago

ボタンのdisabled属性はbooleanで、trueの時は押せないボタン、falseの時は押せるボタンになります。 これに渡してる変数のdisabledは始めはtrueで、日付が2つ選択されるとfalseになる、日付が2つ未満だったり交換モードがOFFの時はtrueになるようにしてます。なので2種類のボタンを用意する必要はないです。

sleep0001 commented 1 month ago

image Modalとmessage抜けていませんか？ 多分コンパイルエラーになってます

RyoUchiyamaGenius commented 1 month ago

どうして消えるんですか？ もう嫌になったので新しくブランチ切ってそっちでプルリク投げます

sleep0001 commented 1 month ago

@RyoUchiyamaGenius さん ２つ選択した時にボタンがableになることが確認できました。 ただ、リクエスト送信後やモーダルキャンセル後にdisableに戻らないのが気になります

RyoUchiyamaGenius commented 1 month ago

コンフリクト解決してる時にマージ分がごっそり消えてたらしい、のでcloneしなおしました。

@sleep0001

リクエスト送信後やモーダルキャンセル後にdisableに戻らないのが気になります

これ完全に考慮漏れです、追加しました switchの左にあった「交換モード」の文言消しましたか？

sleep0001 commented 1 month ago

交換モードの文言は消しました なんか…ダサかったので… 必要なら別の文言にしたいのですが、とりあえず必要ないかと思いました。特にtooltipつけるなら。

RyoUchiyamaGenius commented 1 month ago

おけです

RyoUchiyamaGenius commented 1 month ago

正しくないのでクローズ