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

Hue slider #13

Open
pxsmith opened this issue Dec 22, 2016 · 4 comments · May be fixed by #16
Open

Hue slider #13

pxsmith opened this issue Dec 22, 2016 · 4 comments · May be fixed by #16

Comments

@pxsmith
Copy link

pxsmith commented Dec 22, 2016

@tristen, have you considered adding a hue slider? Here’s a mockup of what it might look like. Could be a very nice little feature to be able to get a wider variety of palettes.

ui

Edit: UI wasn’t accurate, so I updated that.

@pxsmith
Copy link
Author

pxsmith commented Dec 22, 2016

Not sure if this is useful, but here’s the zipped .sketch file.

Colorpicker for data.zip

@gouldingken
Copy link

I agree this is a great idea and would solve the only issue I could find with this brilliant color ramp tool. As it stands you can't achieve the very popular yellow through purple ramps with any saturation.

After playing with it some more, I don't even know if a slider is necessary. The gap always seems to appear on the blue side, so the same goal could be achieved by shifting the hue 180 degrees (starting at blue (180), going through red (360/0) and back to blue (180)).

@tristen would this be an easy fix? Or am I missing something?

@tristen
Copy link
Owner

tristen commented Feb 3, 2018

Check out http://tools.medialab.sciences-po.fr/iwanthue/ which illustrates the "gap" too. There just aren't colors that translate within the color space.

thorn0 added a commit to thorn0/hcl-picker that referenced this issue Feb 11, 2018
@thorn0 thorn0 linked a pull request Feb 11, 2018 that will close this issue
@thorn0
Copy link

thorn0 commented Feb 11, 2018

See #16 :)

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 a pull request may close this issue.

4 participants