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.

Sign up for GitHub

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

Jump to bottom

Climate More-info does not allow dual temp target setting #236

Closed
lekobob opened this issue Mar 9, 2017 · 6 comments
Closed

Climate More-info does not allow dual temp target setting #236

lekobob opened this issue Mar 9, 2017 · 6 comments

Comments

@lekobob
Copy link

lekobob commented Mar 9, 2017

When the more-info-climate panel was updated with arrow buttons in place of the slide, there is no way to set a hi and low target. See demo climate component.
@balloob
Copy link
Member

balloob commented Mar 11, 2017

Oh, this is a use case that we forgot to handle

CC @sdague

@balloob
Copy link
Member

balloob commented Mar 20, 2017

I wonder what a solution would be to fix this without having to revert the change. Maybe have two temp selectors below one another?

@BioSehnsucht
Copy link
Contributor

I've noticed this is an issue with my EcoBee as well. I can change temp just fine in heat or cool mode, but in auto I can't set anything.

I'm surprised more people haven't noticed / complained, but perhaps most just simply change their thermostat mode twice a year and don't use an auto setting ...

@jawilson
Copy link

I have my thermostat almost exclusively in heat-cool so it's obnoxious that it's completely unusable via the HASS UI (though my automations usually take care of any adjustments I'd want to make).

I'm looking into this, I think two selectors side-by-side would be nice, otherwise one below each other.

@jawilson jawilson mentioned this issue Jun 20, 2017
Closed
@ciotlosm ciotlosm mentioned this issue Dec 28, 2017
Merged
@ciotlosm
Copy link
Contributor

This could get fixed by #765 after review is complete. Unsure if the current state of the PR is acceptable considering you get back some control.

andrey-git pushed a commit that referenced this issue Dec 30, 2017
@ciotlosm @andrey-git
Fixing Climate More-info does not allow dual temp target setting (#765)
5085c78 
* Fixing #236

* Fixed build errors

* Updated PR based on feedback from @andrey-git

* Improved side by side style for dual temperature control

* Removed unused styling

* Removed code that is no longer needed after correction from feedback

* Updated based on feedback on discord

* Corrected errors from lint and refactored more vars to const
Fixed lint
@MartinHjelmare
Copy link
Member

This can be closed.

@balloob balloob closed this as completed Jan 2, 2018
@bramkragten bramkragten mentioned this issue Jan 29, 2020
Merged
tkdrob pushed a commit to tkdrob/frontend that referenced this issue Apr 20, 2021
@hagenuck1 @TomBrien
update dynamic notification for map/ camera (home-assistant#236)
dbec20f 
* Update dynamic-content.md

Added an information, that push categories must be set if you send a attached map or camera and that the names start with map/ camera

* Apply suggestions from code review

Co-authored-by: Tom Brien <TomBrien@users.noreply.github.com>

* Moved to line 70

Co-authored-by: Tom Brien <TomBrien@users.noreply.github.com>
@github-actions github-actions bot locked and limited conversation to collaborators Jul 7, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
6 participants
@BioSehnsucht @jawilson @balloob @MartinHjelmare @ciotlosm @lekobob