WLED icon indicating copy to clipboard operation
WLED copied to clipboard

"White" channel only segments confusingly provide 2 or 3 sliders

Open frenck opened this issue 3 years ago • 1 comments

Is your feature request related to a problem? Please describe.

Currently, when configuring a device with a single segment with a single white channel, creates a confusing UI to control and a confusing API to work with.

In the end, the case described above is basically a dimmable LED strip (single color, assumingly just "white").

The point is, WLED will give you 3 sliders for controlling this LED strip, providing the same functionality, but are fully decoupled:

  • The master slider for the brightness.
  • The brightness slider for the segment.
  • The white value slider for the segment.

All of these do the same thing in this case: Controlling the brightness; which makes no sense and is (in my opinion) highly confusing.

Describe the solution you'd like

The master brightness makes sense in a way, as in, that is part of how WLED works in general (although, still confusing in a single-segment setup).

I would suggest removing the slider for the white values in the case as described above and only providing a brightness slider for the segment (omitting the white value).

Describe alternatives you've considered

I've not, really. For external control, I've considered sending the same values for brightness and white value (so external control is at least logical).

Additional context

CleanShot 2022-12-28 at 10 25 24

Ran into while working on: https://github.com/home-assistant/core/pull/75248

Thank you for your ideas for making WLED better!

frenck avatar Dec 28 '22 09:12 frenck

I could agree on white channel and global brightness. Segment is another beast. As you may have several GPIOs attached to independent strips and those can be assigned to different segments, each of which can be controlled independently.

With the other two: for white channel and global brightness it is not feasible to devise a unique UIs for single color LEDs, combination of single color and RGB and RGB only LEDs on a constrained device such as MCU.

blazoncek avatar Dec 28 '22 13:12 blazoncek

I agree that having 3 sliders for apparently the same purpose in typical use cases (1 segment, solid effect) is confusing.

However, keep in mind that even "White only" segments can take advantage of segment color slots (consider the Blink effect blinking between maximum brightness and a dimmer background color value)

Aircoookie avatar Jan 02 '23 10:01 Aircoookie

So this is solved now? :)

Trying to get everything closed and solved to finally get this added to HA :) https://github.com/home-assistant/core/pull/75248

To finally have CCT support for WLED in HA :)

Thanks!

ezcGman avatar Mar 09 '23 10:03 ezcGman

This is not blocking the referenced PR in Home Assistant Core.

frenck avatar Mar 09 '23 10:03 frenck

Gotcha! Still a valid question, so can be closed :)

ezcGman avatar Mar 09 '23 10:03 ezcGman

The core of this issue is not "fixed" and probably will never be. But. I have added UI logic to hide one of the sliders (segment opacity/brightness) if only one segment is used. So the number of possible options to change brightness is reduced from 3 to 2 sliders (one being global brightness the other being "color").

blazoncek avatar Mar 09 '23 12:03 blazoncek

Yeah, I'm with you. I like the improvement for this now! Thx!

ezcGman avatar Mar 09 '23 12:03 ezcGman