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

Add fanart_height option #41

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Add fanart_height option #41

wants to merge 4 commits into from

Conversation

hjbotha
Copy link

@hjbotha hjbotha commented Dec 19, 2019

The fanart_height option applies when image_style is fanart and adjusts the height of each entry on the cards. The start and mid points for the gradients are determined automatically based on the adjusted width of the fan art.

If you have multiple cards with different heights the gradients get messed up, but as this is an existing issue for other styles I thought it might be ok. I don't know enough about JS/Lovelace/CSS to fix that.

@WongGendheng
Copy link

Me for, this does not work. Running Home Assistant 0.105.1.
I have the following configuration:

    cards:
      - # Custom Layout Card
        # https://github.com/thomasloven/lovelace-layout-card
        type: custom:layout-card
        layout: horizontal
        min_columns: 1
        max_columns: 3
        column_width: 350px
        cards:
          - # Trakt 
            type: custom:upcoming-media-card
            entity: sensor.anstehende_serien
            title: Serien (heute)
            clock: 24
            image_style: fanart
            max: 15
            text_shadows: true
            box_shadows: false
            title_size: medium
            fanart_height: 100
            line_size: small
            accent_color: transparent
            border_color: transparent

Looking like this:
series

I would love the e.g. Vikings fanart to have the following height:
series2

Anything I might have configured poorly that comes to mind? I have the feeling that

"line3_default": ""
"line4_default": ""

kind of reserve the empty space?

@mkanet
Copy link
Collaborator

mkanet commented Feb 1, 2024

@hjbotha my apologies for the very late response. It looks like the code has changed significantly since you made this PR 5 years ago. If you still want to see this feature, I can try to add it on my own.

@WongGendheng
Copy link

WongGendheng commented Feb 7, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants