home-assistant / frontend

:lollipop: Frontend for Home Assistant
https://demo.home-assistant.io
Other
3.82k stars 2.61k forks source link

Add Stop Effect to Lights more-info dialog #3619

Closed SeanPM5 closed 4 years ago

SeanPM5 commented 4 years ago

Description of problem: You can turn on light effects from the more-info dialog for Lights, but there is no apparent way to turn these effects off. Seems that you have to turn the light off and then back on again, which is not obvious and can be a little confusing. Some examples of community posts referencing this: here, here, here.

It seems that sending effect: stop will stop the effect and restore your lights. I've tested this with Hue and it seems to work, but I am not 100% positive if this is a standard Home Assistant thing that all light platforms support?

I think that there should either be a "stop" option added inside of the dropdown when an effect is currently active like this (mockup 1):

stopeffectmock1

Or a stop button to the right of the "Effect" dropdown as pictured here (mockup 2):

stopeffectmock2

balloob commented 4 years ago

If we would go for a button, only show it if we know an effect is active. I think I prefer making it part of the dropdown.

SeanPM5 commented 4 years ago

There is one minor discoverability issue with the dropdown idea however.

If the "stop effect" choice is hidden until an effect is active, the user doesn't see it when opening the dropdown menu and they wouldn't necessarily know it was added either because the dropdown closes immediately after they make their selection.

So if this is possible, maybe have the "show effect" option be visible in the dropdown at all times but just in an unselectable/disabled until there is an active effect? And it probably should be at the very bottom of the effect list too.

dshokouhi commented 4 years ago

Maybe instead of "Stop Effect" we change it to "No Effect" and make that the default option?

iantrich commented 4 years ago

I like "No Effect". Going to add that.

iantrich commented 4 years ago

Looked at the code and it is just a list of effects, so for now just going to have "stop" for now. It will not be hidden.

Edit: Changing my mind. Going to go with a button. That will be cleaner

iantrich commented 4 years ago

I only have the demo light to play with for this but it has an effect of 'none'. Do yours not?

image

dshokouhi commented 4 years ago

This is what I got:

Lightify: image

Hue: image

Limitlessled: image

iantrich commented 4 years ago

Can you check if there is an effect attribute after calling the turn_on with an effect of "stop"?

BjoernPetersen commented 4 years ago

As I understand the (partial) fix has been reverted in #3672, right? Could this issue be reopened then? I understand that there's no perfect solution for all devices right now, but having to power-cycle the light to turn off an effect is still annoying.

iantrich commented 4 years ago

I don't believe this is a frontend issue. The backend would have to change to have all integrations follow a similar method of stopping effects, if even possible before the frontend could support it.

ghost commented 4 years ago

This issue was moved by iantrich to home-assistant/core#32446.