-
-
Notifications
You must be signed in to change notification settings - Fork 207
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
fix icons not showing in new frontend version #556
Conversation
@kalkih can you please CR? |
Thanks, will take a look tonight. |
Looks good! I changed the base branch to |
sure will take care of it now |
@kalkih rebased and fixed |
@kalkih thanks for the CR :-) when can we expect a new release? |
Thank you for the contribution! Beta now: https://github.com/kalkih/mini-media-player/releases/tag/v1.14.0-beta Release in a few days maybe, if things seem smooth 🙂 |
Breaking changes made recently to the frontend, specifically in the
ha-icon-button
element, causes the media buttons to not show up. This PR fixed that issue while maintaining backward compatibility. I tested it locally on my hass instance.You can see the change here:
https://github.com/home-assistant/frontend/blob/master/src/components/ha-icon-button.ts#L10-L11
home-assistant/frontend@0c940be#diff-b7749fac30d4d29a769a04d77d9e72093cfdfecd6e325fba66649575229bde9c
i.e I'm using the latest beta version of HA that was released a couple of hours ago