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

Draft: Color picker cursor not restricted to bounds #29

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

418Cat
Copy link

@418Cat 418Cat commented Sep 23, 2023

When picking a color in the Devices tab, the mouse cursor doesn't need to be strictly in the rectangle and circle bounds anymore.

It can be useful to pick edge colors or to pick a hue in the color circle without having to follow strictly the bounds.
A small QOL improvement

When picking a color in the Devices tab, the mouse cursor doesn't need to be strictly in the rectangle and circle bounds anymore
@CalcProgrammer1
Copy link
Owner

ColorWheel originally came from this project:

https://github.com/liuyanghejerry/Qt-Plus

I'm not fond of changing dependencies without updating the upstream, but since this hasn't been updated in 10 years maybe it's OK. I would still probably submit a PR against that repo to give upstream the chance to take the fix.

@418Cat
Copy link
Author

418Cat commented Oct 14, 2023

Will do, I'll open a pr for that repo to hopefully get it merged

@418Cat
Copy link
Author

418Cat commented Dec 28, 2023

It seems like my pr on the Qt-Plus repo isn't really about to get pushed after more than 2 months. I don't know if there's gonna be updates on this particular dependency other than upstream

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants