kalkih / mini-media-player

Minimalistic media card for Home Assistant Lovelace UI
MIT License
1.51k stars 206 forks source link

Issues with artwork: full-cover-fit cropping images #550

Open TMidi84 opened 2 years ago

TMidi84 commented 2 years ago

ISSUE: Using the mini media player card inside of a picture element I have found what appears to be an issue with artwork: full-cover-fit and artwork: full-cover. Based on the guide full-cover to display full artwork, and full-cover-fit for full cover without cropping. When the card is displayed in the standard lovelace view artwork settings appear to function as expected, but when the card is nested is a picture-elements card with these settings the artwork setting no longer appear to work

                  cards:
                    - type: custom:mini-media-player
                      style: |
                        ha-card {
                          height: 375px !important;
                          border: solid 32px rgb(237,245,5)
                          border-radius: 45px
                          font-weight: bold                                                                  
                        }
                      entity: media_player.office
                      artwork: full-cover-fit
                      scale: '0.85'
                      info: scroll

this one is artwork: full-cover-fit in a picture element card image

This one is artwork: full-cover in a picture element card Full cover ` This one is artwork: full-cover standard lovelace view

full cover not cropped ` I am running HA supervisor-2021.09.6 mini media player card v1.13.0

This issue has been mentioned in the mini-media-player forum, by a few others in the last few months, but no one seems to either have a solution or have raised it as an issue.

Can you please look into this?

iainsmacleod commented 2 years ago

Could this simply be that there is an issue with the variable. I have seen when configuring via UI that cover-fit uses the value cover-fit instead of full-cover-fit. Wonder if there is some issue surrounding that.