Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

toggle / ecobee issue #287

Open
1 task done
ischorr opened this issue Oct 14, 2021 · 7 comments
Open
1 task done

toggle / ecobee issue #287

ischorr opened this issue Oct 14, 2021 · 7 comments
Labels
bug Something isn't working

Comments

@ischorr
Copy link

ischorr commented Oct 14, 2021

Checklist:

  • [ x I updated to the latest version available
  • I cleared the cache of my browser

Release with the issue:
0.0.5

Last working release (if known):
n/a

Description of problem:

Great to see a new update, couple issues seen with the new toggle feature, though. In my case, with an ecobee setup with fan, heat, cool, auto support:

  • Toggle off works, but there's no visual indication to the user that the input has been received. There's no immediate UI response. If there's a lag for the integration to detect a state change, that might confuse people to think nothing happened, and try to toggle again. Although even if there's a state change, there still might not be an indicator to the user.
  • Toggle on does not work. Nothing seems to happen - system stays in off state indefinitely.
    • Ability to set cool/heat/fan mode would work around that, but hasn't been added yet
@ischorr ischorr added the bug Something isn't working label Oct 14, 2021
@ischorr
Copy link
Author

ischorr commented Oct 14, 2021

Just tagging @swingerman as an FYI

@swingerman
Copy link
Contributor

Just tagging @swingerman as an FYI

Thanks for finding this @ischorr

Here is the feedback you should experience if the card works properly:

Screen.Recording.2021-10-15.at.09.10.00.mov

Off state:
image

I think I might need to review if that climate integration the card controls actually supports off state and hide the button if not. But I need to investigate first.

@jessiewestlake
Copy link

Just tagging @swingerman as an FYI

Thanks for finding this @ischorr

Here is the feedback you should experience if the card works properly:

Screen.Recording.2021-10-15.at.09.10.00.mov

Off state: image

I think I might need to review if that climate integration the card controls actually supports off state and hide the button if not. But I need to investigate first.

I notice the dual range, heat/cool mode cards do not show the temperature on the hash marks unless it is actively running to heat or cool. I see this even in your video, is it intentional?

@jessiewestlake
Copy link

Checklist:

  • [ x I updated to the latest version available
  • I cleared the cache of my browser

Release with the issue: 0.0.5

Last working release (if known): n/a

Description of problem:

Great to see a new update, couple issues seen with the new toggle feature, though. In my case, with an ecobee setup with fan, heat, cool, auto support:

  • Toggle off works, but there's no visual indication to the user that the input has been received. There's no immediate UI response. If there's a lag for the integration to detect a state change, that might confuse people to think nothing happened, and try to toggle again. Although even if there's a state change, there still might not be an indicator to the user.

  • Toggle on does not work. Nothing seems to happen - system stays in off state indefinitely.

    • Ability to set cool/heat/fan mode would work around that, but hasn't been added yet

This happens to me too, except it’s because the thermostat will go from idle to off, but when you set it back to heat/cool or heat or cool, it stays in off mode until the temperature triggers an action (either heat or cool). So it’s not the fault of the integration.

@swingerman
Copy link
Contributor

Just tagging @swingerman as an FYI

Thanks for finding this @ischorr

Here is the feedback you should experience if the card works properly:

Screen.Recording.2021-10-15.at.09.10.00.mov

Off state: image

I think I might need to review if that climate integration the card controls actually supports off state and hide the button if not. But I need to investigate first.

I notice the dual range, heat/cool mode cards do not show the temperature on the hash marks unless it is actively running to heat or cool. I see this even in your video, is it intentional?

I guess yes, I never worked on that part though.

@jeffgoh
Copy link

jeffgoh commented Jan 19, 2022

I can turn off, but I cannot turn on using the "power button" - to turn on, I need to click on the temperature and click either the up or down chevron - so close, yet so far. Any hints how to fix?

@swingerman
Copy link
Contributor

swingerman commented Jan 24, 2022

I can turn off, but I cannot turn on using the "power button" - to turn on, I need to click on the temperature and click either the up or down chevron - so close, yet so far. Any hints how to fix?

The owners of this repo don't seem to have time to review any PRs or issues, so I don't know how to help here. I would spend my free time fixing any issues but it's no point if no one will review them and merge them.

I applied the changes to my forked repo here: https://github.com/swingerman/lovelace-thermostat-dark-card
feel free to use it instead

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants