Skip to content

Entity registry settings: Remove "Override" string + use domain icon as fallback#7320

Merged
bramkragten merged 3 commits intohome-assistant:devfrom
spacegaier:friendly-name-override
Oct 17, 2020
Merged

Entity registry settings: Remove "Override" string + use domain icon as fallback#7320
bramkragten merged 3 commits intohome-assistant:devfrom
spacegaier:friendly-name-override

Conversation

@spacegaier
Copy link
Member

@spacegaier spacegaier commented Oct 12, 2020

Breaking change

Proposed change

As outlined in the linked issue: Make it clearer what we are overriding:

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

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:

@zsarnett
Copy link
Contributor

We should drop the friendly part everywhere and just show name.

@zsarnett
Copy link
Contributor

Keep the backend the same but frontend should be Name

@spacegaier
Copy link
Member Author

Then it looks interesting, since the attribute is still named differently, but since that's in dev tools, I think we can ignore that.
image

I searched a bit, but I cannot find any place in the UI where we actually show "friendly". Am I missing something?

@spacegaier
Copy link
Member Author

One thing I just realized is that the inputs for name and icon behave differently. The name always shows the input label (e.g. Name Override) above the input area, where as the icon input uses the input label as long as there is no value for the input.

Should we have two components that behave the same in that regard?

@zsarnett In your opinion, the fields should just be named "Name" and "Icon" without the "Override" or did I get you wrong?

@bramkragten
Copy link
Member

bramkragten commented Oct 12, 2020

They do behave the same
image

But if there is no placeholder the label will take that place when not active. There will only be a placeholder when the entity registry has an icon, we could maybe use the domain default as a fallback...

@zsarnett
Copy link
Contributor

Yes I do think we should drop the Override

@spacegaier
Copy link
Member Author

@bramkragten @zsarnett I incorporated both your proposals.

@spacegaier spacegaier changed the title Make input clearer: "Friendly Name Override" Entity registry settings: Remove "Override" string + use domain icon as fallback Oct 15, 2020
@bramkragten bramkragten merged commit 713e057 into home-assistant:dev Oct 17, 2020
@spacegaier spacegaier deleted the friendly-name-override branch October 17, 2020 20:49
@bramkragten bramkragten mentioned this pull request Oct 21, 2020
@github-actions github-actions bot locked and limited conversation to collaborators Jul 5, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Change "Name Override" to "Friendly Name Override" for clarity

4 participants