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

一括出力失敗時のダイアログを外側クリックで閉じられるようにする #1449

Closed
1 of 3 tasks
sevenc-nanashi opened this issue Aug 1, 2023 · 6 comments · Fixed by #1457
Closed
1 of 3 tasks

Comments

@sevenc-nanashi
Copy link
Member

内容

タイトル通りです。

Pros 良くなる点

image

このような状態の時にスクロールする手間がなくなる。Enterキーでも閉じられはしますが気がつく人は多分少ないです

Cons 悪くなる点

ないはず?

実現方法

ダイアログを出すときに「外側クリックでも閉じられるようにする」オプションがあったはず?

VOICEVOXのバージョン

(開発版)

OSの種類/ディストリ/バージョン

  • Windows
  • macOS
  • Linux

その他

(なし)

@Hiroshiba
Copy link
Member

Hiroshiba commented Aug 1, 2023

issueありがとうございます!!

Cons 悪くなる点

間違えて閉じちゃえるようになるのは悪くなる点かもです。


どれだけ長くてもスクロール領域は狭めて、一番下にある「確認」ボタンが最初から表示されてるようにするのが丸いかも・・・?

@sevenc-nanashi
Copy link
Member Author

間違えて閉じちゃえるようになるのは悪くなる点かもです。

正直、AviUtlとか開いてるとEBUSYが出るのを理解している(確定している?)ので余り気にしないような気もします。EBUSYだけなら閉じられる様にするとかいうのも思いつきましたが、

どれだけ長くてもスクロール領域は狭めて、一番下にある「確認」ボタンが最初から表示されてるようにするのが丸いかも・・・?

これが丸そう。

@thiramisu
Copy link
Contributor

#1404 でまとめたところですね。ただレシートみたいな長いログが出るのは想像できていなかったので、そこだけ共通化部分から外してしまってもいい気がします。

@sevenc-nanashi
Copy link
Member Author

おっと、 0.15.0-checkshellbash.0で試してました。最新のmainでちょっと試してみます

@thiramisu
Copy link
Contributor

thiramisu commented Aug 1, 2023

あ、少し勘違いしてたかもです。Component渡してるとこでしたか、そこはユーザー視点では何も弄ってないと思います。
タイトルの「音声書き出し結果」の右にも閉じるボタンつけちゃうとかどうですかね?

@Hiroshiba
Copy link
Member

Hiroshiba commented Aug 2, 2023

タイトル横のボタンでも良いと思うのですが、UXを揃えるならまあ「ダイアログ=右下にボタン」がベターだろうな〜という気持ちです!

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

Successfully merging a pull request may close this issue.

3 participants