Opened 8 years ago

Last modified 3 years ago

#33 assigned enhancement

360-color hue slider

Reported by: ssaammp Owned by: PulkoMandy
Priority: trivial Milestone: 2.9
Component: GrafX2 Version: 2.4
Keywords: color, ui Cc:

Description

The hue slider in grafx2 ranges from 0-255.

It seems that most prefer 0-360.

Change History (7)

comment:1 by pulkomandy, 7 years ago

In the current implementation, the 3 sliders need to have the same range. I'm not sure having S and V range 0-359 as well would be a good thing.
It looks like the code in palette.c will also need some changes to allow this (hardcoded 255 and 256 values instead of using Color_count and Color_max).

comment:2 by PulkoMandy, 6 years ago

Milestone: 2.52.6

comment:3 by PulkoMandy, 6 years ago

Owner: changed from Adrien Destugues to PulkoMandy
Status: newassigned

comment:4 by PulkoMandy, 5 years ago

Milestone: 2.62.7

Move open tickets to 2.7 milestone

comment:5 by PulkoMandy, 4 years ago

Milestone: 2.72.8

comment:6 by PulkoMandy, 3 years ago

Milestone: 2.82.9

comment:7 by yrizoud@…, 3 years ago

I only wanted to bump this as I noticed someone else mentioned the oddity again (cels, on a PJ post)
For hue, 360° is clearly a more common scale, this may be important for people who use several graphical tools and transfer / compare colors.

Remember that the Palette screen has only 128 (or 129 ?) pixels for slider length, so the actual accuracy when dragging slider can't exceed this number of distinct values. More accurate numbers can be selected by clicking arrows, which will increase/decrease the numeric value by 1, no matter the slider's range.

If somebody thinks of changing the precision of S and V sliders from 255 to 100 as well, I'm slightly worried it will makes the RGB -> HSV -> RGB conversion more lossy. However, even Photoshop doesn't care about precision under % for S and V (it actually rejects decimals if you enter them manually) so it may not be a big deal.
Fortunately Grafx2 only performs this back-and-forth conversion when you actually move one of the HSV sliders.

Note: See TracTickets for help on using tickets.