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

Allow custom UI on "card" entities #1824

Merged
merged 1 commit into from
Oct 26, 2018
Merged

Conversation

Informatic
Copy link
Contributor

This will always render entities with custom_ui_state_card attribute set in entities state card. Fixes custom UI on media_player objects.

This PR came out of my annoyance with whole https://community.home-assistant.io/t/custom-ui-mini-media-player/40135 topic. With this patch applied it'll be no longer necessary to add dummy input_text entity and hide original media_player. (This is especially troubling with autodetected media players)

@homeassistant
Copy link
Contributor

Hi @Informatic,

It seems you haven't yet signed a CLA. Please do so here.

Once you do that we will be able to review and accept this pull request.

Thanks!

@balloob
Copy link
Member

balloob commented Oct 21, 2018

I don't know if this has any other side effects.

CC @andrey-git

@Informatic
Copy link
Contributor Author

I don't know either. In theory custom_ui_state_card can be only set explicitly by the user, so this shouldn't break anything. (except for people using customize_glob."*"...)

In practice, at least in case of media player, everything seems to work fine. (though custom UI linked above needs some adjustments, as far as I can see)

This will always render entities with `custom_ui_state_card` attribute
set in `entities` state card. Fixes custom UI on `media_player` objects.
@balloob balloob merged commit 8902328 into home-assistant:master Oct 26, 2018
@ghost ghost removed the in progress label Oct 26, 2018
@balloob
Copy link
Member

balloob commented Oct 26, 2018

Alright, guess it's fine, will be in 81.

@github-actions github-actions bot locked and limited conversation to collaborators Jul 7, 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.

None yet

4 participants