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

Fix: ダウンロードダイアログのOS変更時の挙動を修正、OSによって初期選択が変わるように #232

Merged

Conversation

sevenc-nanashi
Copy link
Member

@sevenc-nanashi sevenc-nanashi commented Oct 27, 2024

内容

#231 を修正します。

関連 Issue

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

(なし)

その他

Windows -> CPU -> Linuxみたいな順にクリックしたときの挙動がホンの少し変わりますがまぁそんなことをする人はめったにいないので大丈夫かと

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.

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

OS選ぶ前に先にCPU版やzip版を選んで、あとで他のOSに変えたときにも反映されてほしかった気持ちがあったんですよね〜〜〜
とはいえバグってるのは一番ダメだから、メインブランチの実装よりはこの実装の形のが良さそう・・・!

@sevenc-nanashi
Copy link
Member Author

sevenc-nanashi commented Oct 27, 2024

そういえば:これ、デフォルトをUser AgentあたりからOSを推測してそれにするとかできそうな気がします。

OS選ぶ前に先にCPU版やzip版を選んで、あとで他のOSに変えたとき

これをそこそこ減らせる...かも?

@Hiroshiba
Copy link
Member

たしかに!!!!
他のOSを選んだときの挙動をどうするかは置いといて、デフォルトのOSがよしなに決まってるのはとても良さそう!!!

@sevenc-nanashi
Copy link
Member Author

あ、これ多分CPU/GPUでも同じことが起こりますね

@sevenc-nanashi
Copy link
Member Author

image

User Agentを読んで良い感じに変わるようにしました。
Apple Silicon MacとIntel MacはUAに差が無いらしいので諦めました。

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!!!

ちょっと一部、オプショナルになってる引数を必須にする変更だけさせていただこうと思います!!

@Hiroshiba Hiroshiba changed the title Fix: ダウンロードダイアログのOS変更時の挙動を修正 Fix: ダウンロードダイアログのOS変更時の挙動を修正、OSによって初期選択が変わるように Oct 28, 2024
@Hiroshiba
Copy link
Member

多分問題ないと思うのでマージします!

@Hiroshiba Hiroshiba merged commit 03a2762 into VOICEVOX:master Oct 28, 2024
1 check passed
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.

Download Bug https://voicevox.hiroshiba.jp/
2 participants