-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
PixelPaint: Fixed some Color Masking issues #24580
Conversation
Hello! One or more of the commit messages in this PR do not match the SerenityOS code submission policy, please check the |
09860eb
to
b8a2553
Compare
https://github.com/SerenityOS/serenity/blob/master/CONTRIBUTING.md#code-submission-policy
And since it's two independent things, maybe it should be two commits? Otherwise looks good, thanks for the PR! |
This change solves a crash where it was possible that the "Color Masking" could try to access pixel coordinates that where beyond the boundaries of the image.
b8a2553
to
d64725d
Compare
This change solves a problem where the color selection via mouse click on the color wheel was computing a wrong hue angle.
d64725d
to
d402e4e
Compare
Thanks! |
This change fixes an issue where pixelpaint was crashing when trying to refine an editing mask with the "Color Masking" feature because we where accessing pixel coordinates that where beyond the boundaries. Furthermore a change was made to make the color masking hue selection work again with mouse clicks directly on the color wheel.