https://bugs.kde.org/show_bug.cgi?id=441672

--- Comment #5 from sh_zam <sh...@sdf.org> ---
“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 :)

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to