-
Notifications
You must be signed in to change notification settings - Fork 309
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ショートカットキーを初期値に戻すボタンの追加 #472
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
プルリクエストありがとうございます!
素晴らしいです、今の実装に完璧に沿っている全く問題のない実装だと思います・・・!!!
1つだけ、誤操作やなんとなく押しちゃったときにいきなり初期値に戻ってしまうとちょっと混乱するユーザーもいるかもなので、「初期値に戻します。よろしいでしょうか?」といった内容のダイアログを出すのをお願いしたいです。
(こちらが参考になるかもしれません。
https://github.com/Hiroshiba/voicevox/blob/989659a1ca08e4901c95c542993af63d51c8c184/src/components/SettingDialog.vue#L414 )
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
とても細かいことですが、ホバー時にボタンの色が濃くなるとより一層わかりやすいかなと思いました!
(他の機能を実装中等でちょっと面倒でしたら言っていただけると!)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!!良い実装とUIだと感じました!
またのプルリクエストをお待ちしております・・・!
内容
ショートカットキーをデフォルトに戻すボタンを実装してみました。
行をマウスホバーでボタンが表示され、1秒間ボタンの上に置いておくとデフォルトに戻すというツールチップが出るようになっています。
勝手がわからず、ショートカットキーの初期値を毎回取ってくるようにしましたが、
配列を探索して初期値を取ってくる処理はレンダラー側ではなくメイン側で行ってもいいのかなと思いました。
また初期値に戻すような処理が見当たらなかったため初期値を保持しておくようにしましたが、他に何かよい方法があるようでしたら教えていただければ嬉しいです。
関連 Issue
close #446
スクリーンショット・動画など
その他