Skip to content
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

Merged
merged 8 commits into from
May 24, 2024

Conversation

honey32
Copy link
Contributor

@honey32 honey32 commented May 10, 2024

内容

表題の通り、「編集」「削除」ボタンは、タスク指向的ではなく、OOUI的に配置されるのが相応しいので、そのようにスタイルを変更してみました。

  • Before
    • リストのヘッダ部分
      • 削除ボタン (選択中のアイテムがあるときのみクリック可)
      • 編集ボタン (選択中のアイテムがあるときのみクリック可)
      • 追加ボタン
    • リスト本体
      • アイテム1
      • アイテム2
      • ...
  • After
    • リストのヘッダ部分
      • 追加ボタン
    • リスト本体
      • アイテム1 (選択中の場合)
        • 削除ボタン
        • 編集ボタン
      • アイテム2
      • アイテム3
      • ...

ついでに、

  • リストの一番下の部分が少し窮屈にならないように 16px の padding を追加しました。
  • リストの各項目を1行のみ表示、あふれたらエリプシス「…」に設定しました。
    • アイコンボタンが現れたり消えたりするとき、行数が変わるとガタつくので

関連 Issue

スクリーンショット・動画など

Before

改修前のUI

After

voicevox-dict-manager-ooui-like.mp4

その他

@honey32 honey32 requested a review from a team as a code owner May 10, 2024 14:18
@honey32 honey32 requested review from Hiroshiba and removed request for a team May 10, 2024 14:18
@honey32 honey32 changed the title feature: 登録単語の編集・削除ボタンの常時表示をやめてOOUI的に 【読み方&アクセント辞書 単語一覧】登録単語の編集・削除ボタンの常時表示をやめてOOUI的に May 10, 2024
@honey32 honey32 force-pushed the feature/ditionary-manager-ooui branch from a66be3d to 6db8b7b Compare May 10, 2024 14:29
@honey32
Copy link
Contributor Author

honey32 commented May 11, 2024

e2e テストの存在を忘れてた… 時間のあるときに対応します

@honey32
Copy link
Contributor Author

honey32 commented May 12, 2024

修正してみました!

Copy link
Member

@Hiroshiba Hiroshiba left a 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に関しては議論の余地ありかもなので、何でもコメントいただけると!!

@honey32
Copy link
Contributor Author

honey32 commented May 15, 2024

了解しました!空いた時間で対応してみます!

@honey32
Copy link
Contributor Author

honey32 commented May 16, 2024

AudioCell の削除ボタン、確認してみると赤ではなく、通常の色のままでした。(あと、アイコンは delete ではなく delete_outline でした。)

一旦は、そちらにスタイルを合わせてみましたが、いかがでしょうか?

image

@honey32 honey32 requested a review from Hiroshiba May 20, 2024 13:05
Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

遅くなってすみません!
UIとても良いと思います!!!ありがとうございます!!!

動作には関係ないのですが、より良いコーディング方法があるので提案してみました!
もしよければ・・・!

src/components/Dialog/DictionaryManageDialog.vue Outdated Show resolved Hide resolved
@honey32 honey32 requested a review from Hiroshiba May 24, 2024 09:28
Copy link
Member

@Hiroshiba Hiroshiba left a 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 との戦いになりますが こちら とか

もしご興味あれば是非・・・!!!

@Hiroshiba Hiroshiba merged commit e1dac6e into VOICEVOX:main May 24, 2024
8 checks passed
@honey32 honey32 deleted the feature/ditionary-manager-ooui branch May 25, 2024 01:01
@honey32
Copy link
Contributor Author

honey32 commented May 25, 2024

なるほど!リファクタリングが面白そうなので、余力のあるときにやってみます!

@Hiroshiba
Copy link
Member

ぜひぜひ!!
分からないことあったら本当に何でも聞いてください・・・!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants