SaturationBar not properly picking up Value about holocolorpicker

[DaXmYsT]

Hi @bermudaparty , I´m interested in your fix! :)

[sikaiser]

So I have a fix now that works. It is very rough around the edges though. I had to rewrite code in many places. I would appreciate instructions on how best to share, if someone is willing to explain. Otherwise I will look it up myself.

[DaXmYsT]

You could fork that project I think. Last changes are 2 years old. If I where you I would ask LarsWerkman if he´s willing to update the project or not.

[DaXmYsT]

@bermudaparty Any updates about this? :)

[sikaiser]

I've forked the project and uploaded my changes into there. https://github.com/bermudaparty/HoloColorPicker
Hope it works like this, I didn't have time to familiarize myself with the proper way of doing it.

[sikaiser]

This is all really old by now. But if anyone is still interested, I just pushed some updates to my fork (/bermudaparty/HoloColorPicker) that now make the whole thing properly operable (in my opinion).

• All the bars update each other real time.
• This initially caused some wobbling and innaccuracies due to conversion between HSV and RGB. -> Everything is now handled in HSV and the wobbling gone.
• This brings the added advantage that it was very easy to merge Saturation- and ValueBar into one class (OmniBar) that can be told at runtime whether it should set saturation or value (the only difference is the location in the HSV array). This does not really make a difference for using the whole thing, but it greatly simplifies further development as there is one less class to maintain now.

Maybe if I get around to it I'll create a pull request, but it doesn't look like it would ever be merged. So I might not bother.