Skip to content

Add option to show icons for areas dashboard strategy #25627

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

Open
wants to merge 5 commits into
base: dev
Choose a base branch
from

Conversation

timmo001
Copy link
Contributor

@timmo001 timmo001 commented May 28, 2025

Breaking change

Proposed change

Adds a configuration option to show icons (if assigned) for the areas dashboard strategy

Default:
image
image

Enabled:
image
image

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

Example configuration

type: areas
show_icons: true

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:

@@ -7836,7 +7836,8 @@
},
"areas": {
"areas_display": "Areas to display",
"no_entities": "No entities in this group, the section will not be displayed"
"no_entities": "No entities in this group, the section will not be displayed",
"show_icons": "Show assigned area icons on the dashboard tabs"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not 100% sure on the phrasing here, does someone has a better description for this?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe just "Display area icons?", short and simple.

@timmo001 timmo001 marked this pull request as ready for review May 28, 2025 16:59
@silamon silamon requested a review from piitaya May 28, 2025 17:23
@bramkragten bramkragten added the Needs UX Pull requests requiring a review from the Home Assistant design team label May 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed Needs UX Pull requests requiring a review from the Home Assistant design team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants