xtne6f / EDCB

BonDriver based multifunctional EPG software
Other
165 stars 37 forks source link

カスタム番組表の設定でキャンセルしても設定が変更されることがあるのを修正 #15

Closed tkntrec closed 7 years ago

tkntrec commented 7 years ago

更新お疲れ様です。

8 や #13 と同様の参照バグです。

気付いたのでご報告いたします。

番組表の場合は割り切って子ウィンドウ側の確定でもいいのかもしれませんが、 その一方で番組表から設定画面を開いた場合は、一時的な変更しか出来ない仕様だったりしますよね。

xtne6f commented 7 years ago

頂きます。ロジック的に意図しないものであるはずなのでこの修正が良いと思います。

モーダルな子ウィンドウでさらに「OK」ボタンがある設定画面って確定のタイミングが直感的でないときがありますね。この表示タブの設定だと子ウィンドウとして表示する場合いっそ「閉じる」ボタンだけにしちゃえば迷いは少ないかもしれないです(利便性は下がるか…)。

xtne6f commented 7 years ago

番組表から設定画面を開いた場合は、一時的な変更しか出来ない

本当だ…これについても手当要りますね。 ちゃんと設定保存するのが正攻法ですがこちらでは番組表のコンテキストメニューから撤去するかもしれません。

tkntrec commented 7 years ago

コンテキストメニューのタイトルから設定保存出来そうな雰囲気になっていますが、 元々一時変更用に追加されたものだったのかもしれませんね。 (自分も一時変更したいときは確かにある・・けれど、やはり設定保存したいときもあったり。)

xtne6f commented 7 years ago

コンテキストメニューの「表示設定」は確かに原作の意図的にも一時変更用っぽいですね。 自分はこういう使い方したことなかったですが、「ジャンルを少し絞り込みたい」的な用途に使えるので設定保存はやめておくことにしました。

tkntrec commented 7 years ago

コンテキストメニューの表記を少し変更すると判りやすくなる気もするのですが、 自分はそのへん決めかねたので、結局、一時変更と設定保存の両対応で誤魔化してます。