HestiaPi / hestia-touch-one-ui

ONE UI files shown on the touch LCD
GNU General Public License v3.0
5 stars 7 forks source link

Setting or config to disable humidity display. #31

Open OneHoopyFrood opened 1 month ago

OneHoopyFrood commented 1 month ago

It would be nice to be able to optionally not display humidity if the sensor isn't present, such as in the US model.

OneHoopyFrood commented 1 month ago

Is there a precedent here? Should this be a config option somewhere? That would require ssh-ing in, and if the intent is to make this accessible to lower tech skill levels, an OpenHab setting seems appropriate.

jaythomas commented 1 month ago

Settings are configured via OpenHAB rather than from the touchsceen. For instance, changing the system from EU to US will hide humidity: https://github.com/HestiaPi/hestia-touch-one-ui/blob/f0a6281296053e9681eedd3b011558b00eec1499/src/plugins/store.js#L220-L236

If the option to enable/disable humidity independent of the system type was needed, we would need to modify the OpenHAB interface to expose that option. The code for that lives over at: https://github.com/HestiaPi/hestia-touch-openhab

After changing it there, we can subscribe to the new MQTT message in the frontend app here and start using it: https://github.com/HestiaPi/hestia-touch-one-ui/blob/f0a6281296053e9681eedd3b011558b00eec1499/src/plugins/store.js#L295-L298

OneHoopyFrood commented 1 month ago

I see! But I don't seem to have an option in OpenHab to switch from EU to US. Where is it supposed to be? I'm on 1.4-dev

jaythomas commented 1 month ago

I haven't used the latest image so that question might be better suited for the forums.

OneHoopyFrood commented 1 month ago

Huh. Well, getsystemtype.sh says US.

jaythomas commented 1 month ago

Ah! I believe we were talking about two different things here. You're talking about the humidity percentage that is displayed next to the temperature, NOT the humidity mode which is a separate thing (which you probably don't even see since you're set up for US). There was never an option to disable the humidity sensor, but if there was the precedent would be to expose it as an option in OpenHAB. If you want to make a ticket there we can link the two, but it would be quicker to make your own frontend build. The html you want to comment out is in src/components/home-screen.vue: https://github.com/HestiaPi/hestia-touch-one-ui/blob/f0a6281296053e9681eedd3b011558b00eec1499/src/components/home-screen.vue#L84