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

--- Comment #3 from cwo <[email protected]> ---
(In reply to chaos from comment #2)
> I asked Claude to take a look at this and I think it may have found the bug:
> https://claude.ai/share/9d83b7bf-0b64-4edc-8049-8e6d67e8d4ce

That's wrong and has nothing to do with this. This is to reserve the space for
the (optional) label over the grid showing which folder it is, but that space
is taken up by the label (and is much smaller than a full row).

The real issue is that I was wrong; the intent of the code is to use the
preferred sizes as the minimum size for desktop widgets (and the preferred size
is a minimum of two rows, and three columns), see here:

https://invent.kde.org/plasma/plasma-desktop/-/blob/master/containments/desktop/package/contents/ui/main.qml?ref_type=heads#L448

To get the intended minimum sizes, you need to specify the argument true, but
we don't; we make it conditional on isPopup (which, awkwardly, isn't only for
popups, but for any in-panel Folder View. In the panel popup of FolderView,
(unless the panel is large enough to not need a popup), it works and allows you
to change it to be one column or one row. (Though the precise sizing could be
improved a bit).

I'm not sure there's a good reason for not allowing it to be smaller; I don't
see one immediately so I think it still makes sense to change this (and make
the precise sizing a bit better). But the code seems to work as intended.

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

Reply via email to