home-assistant / frontend

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

More info "Controls" tab naming #6844

Closed SeanPM5 closed 4 years ago

SeanPM5 commented 4 years ago

Checklist

The problem

The first tab on more info dialogs is now called "Controls" which fits well for anything that has interactive controls there - like Lights, Media Players, Remotes, Alarms, Thermostats, Counters, etc...

But on some others like Sensors, Sun, Device Trackers, Weather, etc which are read only, this naming feels a bit wrong and weird? And for newer users it might be confusing if they're expecting to see controls when that's not actually supported.

image

Expected behavior

I think for domains that offer interactivity it should display "Controls" and for others that are read-only it should display "Details" or "Info" or "State" or something else....

Steps to reproduce

Environment

State of relevant entities

Problem-relevant configuration

Javascript errors shown in your browser console/inspector

Additional information

bramkragten commented 4 years ago

https://github.com/home-assistant/frontend/issues/6856