The way HSL works in Photo Lab doesn’t make any sense to me. It’s unlike every other editor I’ve used. It’s borderline unusable. There are 3 main issues with the implementation of HSL in PhotoLab:
For some reason almost nothing ever registers as “green” on the green hsl slider. All of my landscape photos involving trees aren’t affected very much by using this slider. The yellow slider affects both yellow, and almost all shades of green as well! I’m GUESSING this could be brought about by using the same algorithm between SRGB and AdobeRGB modes, since SRGB has a limited green gamut compared to adobe. If that is the case, you need to change your algorithm to work differently for the SRGB gamut. Or maybe it was designed by someone with mild color blindness to certain colors? Some of the other 5 colors seem a little off too, but green is by far the worst. Whatever is causing this, it needs to be changed as soon as possible.
The “Lightness” slider affects saturation!!! If I turn down the lightness on a blue sky, the sky becomes grey, instead of a darker shade of blue! We already have a saturation slider directly above the lightness slider, let me control saturation! There’s no reason why “lightness” should also affect saturation. It seems like the code for the “Channel Mixer for black and white” slider was copy/pasted into the HSL module, and someone forgot to make changes before shipping the product. “Lightness” and “Channel Mixer for Black and White” seem to do exactly the same thing, making them redundant.
We need 8 colors in HSL. If you can fix the problems I’ve listed above, then give us 8 colors instead of 6, I won’t have to go into other programs to do color correction.
There is already a thread for #3, but I thought #1 and #2 were just as important to having a usable HSL module. There’s the obivous issue of backwards compatibility with older files, but you could always do what Adobe does and have a dropdown menu that allows you to choose the old or new version of the module. Old photos that have already been edited would default to the old version of the HSL module. Please comment if you are noticing these same issues!