home-assistant / frontend

:lollipop: Frontend for Home Assistant
https://demo.home-assistant.io
Other
3.8k stars 2.6k forks source link

2024.7 Tile card with numeric input added not spaced correctly #21210

Closed wormiedk closed 13 hours ago

wormiedk commented 6 days ago

Checklist

Describe the issue you are experiencing

If I add a tile card to a dashboard, add the "numeric input" feature to the card and change the layout to one row, 4 columns: image

The card will not add enough spacing which means that other cards overlap: image

Describe the behavior you expected

Enough spacing below the card or slider and entity on same row

Steps to reproduce the issue

  1. Add tile card for numeric entity
  2. Add the numeric input feature
  3. Change layout to one row and 4 columns ...

What version of Home Assistant Core has the issue?

2024.7.0b3

What was the last working version of Home Assistant Core?

No response

In which browser are you experiencing the issue with?

No response

Which operating system are you using to run this browser?

No response

State of relevant entities

No response

Problem-relevant frontend configuration

No response

Javascript errors shown in your browser console/inspector

No response

Additional information

No response

silamon commented 4 days ago

I think this is sort of to be expected. The tile cards always take up a row, and are extending rows with each feature you add. The customization of the rows/cols, now coming to UI in this release, allow to make it bigger, but won't relayout things in places where there is now space available.

piitaya commented 13 hours ago

CleanShot 2024-07-04 at 18 49 43

It's fixed by https://github.com/home-assistant/frontend/pull/21244. You can now longer have feature overflow because the min height is calculated according to the number of features.