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.

Sign up for GitHub

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

Jump to bottom

Paint tool color set to black on cancel #5

Closed
leemi opened this issue Dec 20, 2011 · 2 comments
Closed

Paint tool color set to black on cancel #5

leemi opened this issue Dec 20, 2011 · 2 comments

Comments

@leemi
Copy link
Collaborator

leemi commented Dec 20, 2011

If you bring up the color picker and press cancel, the color gets set to Black, instead of not changing anything.
@sdouglas
Copy link
Owner

Fixed (only tested on mac). None of the normal signals work for native color dialogs, and a color is always returned even if the dialog is canceled. Solved by checking the returned QColor's "isValid" flag, which gets set to false if the user canceled the dialog.

@leemi
Copy link
Collaborator Author

leemi commented Dec 20, 2011

Confirmed fixed on Win7

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@sdouglas @leemi