OctoPrint-WLED icon indicating copy to clipboard operation
OctoPrint-WLED copied to clipboard

[Feature Request] Make the navbar switch stateful

Open cbrherms opened this issue 3 years ago • 2 comments

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

Navbar button is stateless and so does not sync with current state of the WLED instance. If for example you make use of the terminal commands to turn the lights on or off via another plugin or gcode, the navbar button does not reflect this change.

Describe the solution you'd like

Would be best if the on state of WLED was monitored and the navbar switch was correct for this status. Failing that, having the @WLED gcode commands change the state of the navbar

Why should this be implemented?

At present, with the switch having two colours for on/off it implies there is some knowledge of state and when that fails it doesn't appear as good from a UX point of view. This wouldn't be as much of an issue if it was the only method of toggling the lights but with there being the command method it's easy to get to a point where it is wrongly showing the wrong state on the navbar

Additional context

cbrherms avatar Dec 16 '21 00:12 cbrherms

This is semi-bug and semi new feature. The navbar button should be syncing with things like the @WLED commands, and if you have it open on more than one tab, but I just looked and the code for that is missing the one final part in the frontend - the messages are sent, but nothing is done...

That's the bug part. Syncing with external WLED changes is more complicated and not intended to work yet, but still doable if I wanted it. WLED has a websocket I can connect to, and a newer version of the library that I'm using (modified) has support for that.

It will take a long time regardless - I don't currently have the time to work on any of my projects, hopefully in the new year there will be good things coming...

cp2004 avatar Dec 16 '21 13:12 cp2004

No problem. I was unsure whether to report it as bug or feature so thanks for clarifying.

cbrherms avatar Dec 16 '21 14:12 cbrherms