| Summary: | Widget won't shrink to less than 2 rows and 2 columns | ||
|---|---|---|---|
| Product: | [Plasma] plasmashell | Reporter: | chaos <cjamez> |
| Component: | Desktop icons & Folder View widget | Assignee: | Plasma Bugs List <plasma-bugs-null> |
| Status: | CONFIRMED --- | ||
| Severity: | normal | CC: | cwo.kde, hein, kdedev |
| Priority: | NOR | ||
| Version First Reported In: | 6.5.5 | ||
| Target Milestone: | 1.0 | ||
| Platform: | Other | ||
| OS: | Linux | ||
| Latest Commit: | Version Fixed/Implemented In: | ||
| Sentry Crash Report: | |||
|
Description
chaos
2026-02-05 16:46:15 UTC
Intent of the code seems to be to have a minimum height/width of 1 icon: https://invent.kde.org/plasma/plasma-desktop/-/blob/master/containments/desktop/package/contents/ui/main.qml?ref_type=heads#L101 Not sure yet why it doesn't work. 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 (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. |