Skip to content

Conversation

@ildar170975
Copy link
Contributor

@ildar170975 ildar170975 commented Dec 25, 2025

Breaking change

Proposed change

Allow showing both an icon & a text title for a view.
By default, if an icon is defined, a text title is not shown - i.e. like it is now before this PR.
If a new show_icon_and_title is set, then both of them are shown.

изображение изображение

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

Additional information

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:

@ildar170975 ildar170975 marked this pull request as draft December 26, 2025 19:52
@ildar170975 ildar170975 marked this pull request as ready for review December 26, 2025 23:55
@ildar170975 ildar170975 marked this pull request as draft December 27, 2025 18:27
@ildar170975 ildar170975 marked this pull request as ready for review December 28, 2025 02:34
@silamon silamon added the Needs UX Pull requests requiring a review from the Home Assistant design team label Dec 31, 2025
@mbb365
Copy link

mbb365 commented Jan 7, 2026

We (Product UX) talked about it, and we like it. Nice work @ildar170975!

UX approved.

@mbb365 mbb365 removed the Needs UX Pull requests requiring a review from the Home Assistant design team label Jan 7, 2026
@silamon silamon enabled auto-merge (squash) January 7, 2026 18:58
@silamon silamon merged commit d195fd3 into home-assistant:dev Jan 7, 2026
11 checks passed
@ildar170975 ildar170975 deleted the view_icon_with_title branch January 7, 2026 20:45
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.

3 participants