https://bugs.kde.org/show_bug.cgi?id=441672
--- Comment #6 from Matej Starc <matej.st...@protonmail.com> --- (In reply to sh_zam from comment #5) > “Matej Starc” <bugzilla_nore...@kde.org> writes: > > > <https://bugs.kde.org/show_bug.cgi?id=441672> > > > > — Comment #4 from Matej Starc <matej.st...@protonmail.com> — > > (In reply to sh_zam from comment #3) > >> “Matej Starc” <bugzilla_nore...@kde.org> writes: > >> > >> > <https://bugs.kde.org/show_bug.cgi?id=441672> > >> > > >> > — Comment #2 from Matej Starc <matej.st...@protonmail.com> — > >> > (In reply to sh_zam from comment #1) > >> >> This seems to be due to lack of an attached `Layout.minimumHeight` in > >> >> <https://invent.kde.org/frameworks/kirigami/blob/master/src/controls/> > >> >> templates/AbstractCard.qml#L119-128. I can see a merge request started > >> >> by > >> >> you to address this, but I wonder why you didn’t proceed with it? > >> >> <https://invent.kde.org/frameworks/kirigami/-/merge_requests/712> > >> > > >> > I won’t get merged until kf6 or ever. Depends on whay the “og” kirigami > >> > devs > >> > decide (there was talk of removing CardsGridView completely) > >> > >> I think this is more of a bug with Kirigami.Cards than CardsGridView. This > >> bug > >> is reproducible even if you use the vanilla GridsView. > > > > Its because cards expand out of their cell. This can be either be fixed by > > limiting the height of the card to the cell height, or allowing the user to > > customize the cell size (as I’ve done in my MR). > > Yes, in my case I *had* limited the size of the Card to cellHeight, but then > the > image (headerItem) expanded into the contentItem (see AbstractCard.qml in > Kirigami) because there is no Layout.minimumHeight for it or anything else > for > that matter :) The reason AbstractCard does not limit the height of the header is because there is no need to do it. If you want to limit the header size, you can just limit the size within your code. AbstractCard only offers the standard card layout (with extra functionality). -- You are receiving this mail because: You are watching all bug changes.
