search

mkpaz / atlantafx

Modern JavaFX CSS theme collection with additional controls.
https://mkpaz.github.io/atlantafx
MIT License
791 stars 64 forks source link

Incorrect height of list cells #80

Open dlemmermann opened 9 months ago

dlemmermann commented 9 months ago

The screenshot below shows how PDFViewFX is trying to render the page thumbnails in a ListView on the left-hand side. For some reason the cell height remains fixed with a low value instead of adjusting to the height of the thumbnails when using AtlantaFX. PDFViewFX does not explicitly set a height for the cells.

Bildschirmfoto 2023-11-21 um 10 53 42
dlemmermann commented 9 months ago

It is possible to work around this by explicitly adding -fx-cell-size: -1; to the pdf-view.css file, but I am worried that AtlantaFX changed the default behaviour, hence breaking a lot of apps that are looking into using it. This in turn will lead to a lower adoption rate.

mkpaz commented 1 month ago

Yes, that's correct. Table cells have a fixed height by default. To set any custom content other than string, the cell size must be reset. This should be specified explicitly in the doc.