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

The color wheel contains colors with different L values. #5

Open
sheitmann opened this issue May 22, 2015 · 1 comment
Open

The color wheel contains colors with different L values. #5

sheitmann opened this issue May 22, 2015 · 1 comment
Labels

Comments

@sheitmann
Copy link

The colors on the border have a L value of 50. If you move the mouse more to the center you see (for example when using the ColorEditorManager) that the L value jumps sometimes to 49.

@cyotek
Copy link
Owner

cyotek commented May 23, 2015

Thanks for the report. This is probably rounding issues - the ColorWheel generates a 0.5 luminance HSL colour, but then converts it to a RGB Color which is used by IColorEditor. It's something I've mulled over before, but I have gone with ease of use - it's easier to consume RGB.

I think it you want as much accuracy as possible, you'd have to change the whole suite to run off HSL. I checked the conversion code, and this isn't code I wrote - and I don't seem to have made a note of where I sourced it from, slightly annoying.

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

No branches or pull requests

2 participants