-
Notifications
You must be signed in to change notification settings - Fork 304
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
【読み方&アクセント辞書 単語一覧】登録単語の編集・削除ボタンの常時表示をやめてOOUI的に #2072
【読み方&アクセント辞書 単語一覧】登録単語の編集・削除ボタンの常時表示をやめてOOUI的に #2072
Conversation
a66be3d
to
6db8b7b
Compare
e2e テストの存在を忘れてた… 時間のあるときに対応します |
修正してみました! |
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.
とても良いですね!!!
(なぜこのUIを思いつけなかったのか。。。。。)
UIとしては全く問題ないのですがUXとしてちょっとだけ変更お願いできると!!
- 削除ボタンを右にしたい
- 操作が左から右に流れると考えると、削除は一番最後なので一番右がいいかな、みたいな
- 削除ボタンを赤色にしたい
- テキスト欄を削除する時にゴミ箱が出るのですが、それが確か赤色なので統一感あった方がいいかなと
- ボタンが現れるの「選択時」のみから「選択時orマウスホバー時」にしたい
- 難しいければ「マウスホバー時」のみでも
- テキスト欄の挙動がそうなっているのと、クリックするのに若干勇気がいるかもと思ったので
UXに関しては議論の余地ありかもなので、何でもコメントいただけると!!
了解しました!空いた時間で対応してみます! |
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.
遅くなってすみません!
UIとても良いと思います!!!ありがとうございます!!!
動作には関係ないのですが、より良いコーディング方法があるので提案してみました!
もしよければ・・・!
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 含む リファクタリングにご興味あれば、例えば今回の辞書ダイアログのコンポーネント分けとか、あとこちらとか。
ちょっとした機能実装にご興味あれば、保存周りの処理が必要ですがこの辺りとか。
あるいは UI の見た目を良くする系だと、ちょっと CSS との戦いになりますが こちら とか
もしご興味あれば是非・・・!!!
なるほど!リファクタリングが面白そうなので、余力のあるときにやってみます! |
ぜひぜひ!! |
内容
表題の通り、「編集」「削除」ボタンは、タスク指向的ではなく、OOUI的に配置されるのが相応しいので、そのようにスタイルを変更してみました。
ついでに、
関連 Issue
スクリーンショット・動画など
Before
After
voicevox-dict-manager-ooui-like.mp4
その他