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

Climate component thermostat states don't all show on mobile UI #841

Closed
ghost opened this issue Jan 27, 2018 · 5 comments
Closed

Climate component thermostat states don't all show on mobile UI #841

ghost opened this issue Jan 27, 2018 · 5 comments

Comments

@ghost
Copy link

ghost commented Jan 27, 2018

@Emacee commented on Jan 27, 2018, 9:00 AM UTC:

Home Assistant release (hass --version):
Experienced both on 0.61.0 and 0.61.1

The changelog of 0.62 doesn't mention anything about the UI being updated that is why I'm filing this report now. I'm using hass.io so am not able to test 0.62 yet (need to wait for it to be available).

Python release (python3 --version):
Not sure, using the latest hass.io version 0.84 with host OS 1.1

Component/platform:
Toon climate component for rooted devices: https://github.com/cyberjunky/home-assistant-custom-components/tree/master/climate

Haven't been able to test with another climate platform so far.

Description of problem:
In my case I am using 5 default states for the thermostat (actually that is the thermostat's default), on the desktop UI I am able to switch between all of them but on mobile I only see the last few. Annoying because it means that I can only manually set the temperature to comfort.

I'm aware that I am using a custom component but I think it is a UI problem and not based on the component since it does work on the desktop site but not on mobile (iOS).

Expected:

Problem-relevant configuration.yaml entries and steps to reproduce:

Add this to configuration.yaml:

climate:
    platform: toon
    name: Toon Thermostat
    host: !secret toon_host
    port: 10080
    scan_interval: 10

And this is part of the custom component that defines the 5 default states:

def __init__(self, name, host, port):
        """Initialize the thermostat."""
        self._data = None
        self._name = name
        self._host = host
        self._port = port
        self._current_temp = None
        self._current_setpoint = None
        self._current_state = -1
        self._current_operation = ''
        self._set_state = None
        self._operation_list = ['Comfort', 'Home', 'Sleep', 'Away', 'Holiday']
        _LOGGER.debug("Init called")
        self.update()

Additional info:
desktopui
ios-ui

This issue was moved by @MartinHjelmare from home-assistant/core/issues/11950.

@ghost
Copy link
Author

ghost commented Jan 27, 2018

@cyberjunky commented on Jan 27, 2018, 9:18 AM UTC:

This was/is a issue, I had too a while back, I even reported this and a few other users, but I think no one picked this up.
But for the lasts months this worked ok for me, maybe because I have bought another phone, and it's related to the screen resolution of the device. You have translated the state text to Dutch, done in the source code of the component or elsewhere?
This is my current view (on a LG V30 android):

toon

@ghost
Copy link
Author

ghost commented Jan 27, 2018

@cyberjunky commented on Jan 27, 2018, 9:24 AM UTC:

I am running Home Assistant 0.62.0.dev0 btw.

@ghost
Copy link
Author

ghost commented Jan 27, 2018

@Emacee commented on Jan 27, 2018, 9:31 AM UTC:

Thank you for the quick reply! Good to see it's working for you. I indeed translated the states in the customize.yaml:

climate.toon_thermostat:
  friendly_name: Toon
  operation_list:
  - Comfort
  - Thuis
  - Slapen
  - Weg
  - Vakantie

I'm now restarting with the #- Vakantie to see whether it's because that word is too long.
The phones I'm using are iPhone X (you would think the resolution wouldn't be a problem) and iPhone 6S.

Update:
Unfortunately, removing the fifth and longest state doesn't solve the issue. A temporary fix would be to comment out the least used states but that would be a pity.

ios-ui-2

@ciotlosm
Copy link
Contributor

Maybe this is related to #775 which is caused by PolymerElements/paper-dialog-scrollable#72

@NovapaX
Copy link
Contributor

NovapaX commented Mar 18, 2018

This should be fixed. If not please (ask to) reopen or file a new issue.
Thanks for your feedback!!

@NovapaX NovapaX closed this as completed Mar 18, 2018
@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.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants