-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Card height no longer respected after updating to 2024.11.0b0 #22616
Comments
It seems to be related to this PR : #22593 |
@alexruffell |
It think it's because most of stock cards have 100% height for the root element while custom button card doesn't. |
Then this related to a custom card & should be fixed/styled on a custom card's side. |
@ildar170975 How can you be certain it is not a bug introduced in the frontend given the change happened with the latest core beta? I guess the Also, @piitaya found and fixed what I believe to be the same exact issue just a few versions (beta) back. I therefore assumed it was the same issue popping up again. What I really need to do, is see if I can move back to the stock card on as many of these buttons as possible to avoid the added complications but I recall I chose to use this card on dozens of my views because it was significantly more flexible in its customization (time to revisit!). |
Proposed to revert changes made in #22593. |
@ildar170975 Are you using panel view for your examples? |
Ofc, this seems to happen in Panel only. |
@piitaya
If a stack is used inside another horizontal stack - |
By design the panel will take 100% height. It's not only a problem with horizontal stack. If you put any card it will take 100% of the height. |
Yes, any card (which is not a stack) takes 100% in a panel mode. or this (look at the left vert stack): So, my proposal is to NOT allow occupying 100% for horiz stack - and only allow it if a horiz stack is placed inside another horiz stack. But imho ideally would be adding smth like |
Fixed with #22617 |
@ildar170975 It think it's more a problem with the panel than a stack issue. |
I am away from PC these days and will try to dig it again in a week if come back. Thanks for a feedback. |
Checklist
Describe the issue you are experiencing
Height of
type: custom:button-card
set to 100% is no longer respected. If I set it in px it works but doing so causes issues when viewing dashboard on different devices. The two tiny buttons should fill in the vertical space beneath them as they did in 2024.10.4 and prior versions. This issue (or very similar one) also happened one or two months ago and was resolved quickly upon reporting by @piitaya but I forget the issue#.Describe the behavior you expected
Buttons fill available space showing their icon and title as all the other buttons in the snapshot above.
Steps to reproduce the issue
YAML below is for the entire column. Using this YAML the problem will be visible.
What version of Home Assistant Core has the issue?
2024.11.0b1
What was the last working version of Home Assistant Core?
2024.10.4
In which browser are you experiencing the issue with?
Google 130.0.6723.70 (Official Build) (64-bit)
Which operating system are you using to run this browser?
Windows 11
State of relevant entities
No response
Problem-relevant frontend configuration
The text was updated successfully, but these errors were encountered: