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

Change "Default dashboard" to "Automatic dashboard" #23508

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

NoRi2909
Copy link
Contributor

@NoRi2909 NoRi2909 commented Dec 30, 2024

Proposed change

Currently there is a duplicated use of the term "Default dashboard" – for the pre-populated dashboard type maintained by Home Assistant and the first dashboard at the top of the sidebar.

The pre-populated dashboard is currently called "Default dashboard"

Screenshot 2024-12-29 13 41 29

and the one that shows up at the top of the Sidebar is also called "Default":

Screenshot 2024-12-29 13 42 26

This PR removes that ambiguity by renaming the "Default dashboard" type as "Automatic dashboard".
It also updates the description to "… all your devices and services …"
And it finally replaces the outdated term "configuration" with "settings" in the explanation of "Take control" when making an "Automatic dashboard" editable.

This change will also make it much easier to explain the effect of switching an entity to "hidden" regarding dashboards as we can now say "not included in an automatic dashboard". See PR #23473 for details.

Type of change

  • Dependency upgrade
  • Bugfix (non-breaking change which fixes an issue)
  • New feature (thank you!)
  • Breaking change (fix/feature causing existing functionality to break)
  • Code quality improvements to existing code or addition of tests

Additional information

  • This PR fixes or closes issue: fixes #
  • This PR is related to issue or discussion:
  • Link to documentation pull request:

Checklist

  • The code change is tested and works locally.
  • There is no commented out code in this PR.
  • Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

Currently there is a duplicated use of the term "Default dashboard", for the pre-populated dashboard type maintained by Home Assistant and the first dashboard at the top of the sidebar.

This commit removes that ambiguity by renaming the "Default dashboard" type as "Automatic dashboard" plus fixing two explanations.
@NoRi2909 NoRi2909 marked this pull request as ready for review December 30, 2024 07:53
@frenck
Copy link
Member

frenck commented Dec 30, 2024

I am not sure about this one. We support multiple different automatic or generated dashboards actually (and aim to introduce more of those). The suggested wording (with that in mind) seems not right.

@NoRi2909
Copy link
Contributor Author

NoRi2909 commented Dec 30, 2024

@frenck I'm open to any other suggestions. This is just one idea to reword this.

We came to this point regarding the behavior of hidden entities which currently have this explanation:
"Hidden entities will not be shown on your dashboard or included when indirectly referenced …"

That is (no longer) accurate as a "hidden" entity can still be placed on any user-defined dashboard, it's just not shown on those "automatic" or "pre-populated" dashboards. But as they are currently called "Default dashboard(s)" we we'd get an even more misleading explanation if we used that term there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants