-
Notifications
You must be signed in to change notification settings - Fork 311
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: GPU変更のダイアログがクリックできなかったのを修正 #2507
fix: GPU変更のダイアログがクリックできなかったのを修正 #2507
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.
気づきませんでした、ありがとうございます!!!
解決方法にちょっと危なそうな匂いを感じたので、ロジックを変更させていただこうと思います!
具体的にはchangeGpuの外でローディングを回すのではなく、中でloadingを追加しようかなと。
全体をラップするとバグるから、適宜loading dialogを表示するようにしてみました。
ちょっとロジック変わってるので良さそうかどうかレビューいただけると頼もしいです!!
コミットします!
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!!
たぶんできたはず!
良さそうだったらmergeしていたけると・・・!
あっ! |
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.
良さそう。
内容
タイトル通り。スクショのようになっていました。
関連 Issue
(なし)
スクリーンショット・動画など
その他
(なし)