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

Fixed "Yes, don't ask again" image uploader prompt not working on macOS #5011

Merged
merged 1 commit into from
Dec 10, 2023

Conversation

pajlada
Copy link
Member

@pajlada pajlada commented Dec 10, 2023

Description

The value returned from exec wasn't really feasible if it changed per platform, so we use the recommended way of comparing button pointers instead.

I have confirmed this works as expected on:

  • macOS on Qt6
  • Linux on Qt6

I would appreciate a Windows tester making sure that the "Yes, don't ask again" image uploader prompt works as expected with this PR

Fixes #5010

@brian6932
Copy link
Contributor

Tested on Windows, works as expected

@pajlada
Copy link
Member Author

pajlada commented Dec 10, 2023

image

@pajlada pajlada changed the title Fixed "Yes, don't ask again" image uploader prompt not working on macos Fixed "Yes, don't ask again" image uploader prompt not working on macOS Dec 10, 2023
@pajlada pajlada merged commit e425816 into master Dec 10, 2023
18 of 20 checks passed
@pajlada pajlada deleted the fix/image-uploader-prompt-macos branch December 10, 2023 12:28
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.

Skip image uploading confirmation if you clicked "Yes, don't ask again" in the alert
2 participants