Bug 406806

Summary: Folder View widget is not an icon on vertical panels
Product: [Plasma] plasmashell Reporter: Noah Davis <noahadvs>
Component: FolderAssignee: Eike Hein <hein>
Status: RESOLVED FIXED    
Severity: normal CC: aldo-public, kainz.a, kde, nate, plasma-bugs
Priority: NOR    
Version: master   
Target Milestone: 1.0   
Platform: openSUSE   
OS: Linux   
Latest Commit: Version Fixed In: 5.18.7
Attachments: Folder View on a vertical panel

Description Noah Davis 2019-04-23 10:43:46 UTC
Created attachment 119585 [details]
Folder View on a vertical panel

SUMMARY
The folder view widget is not an icon on vertical panels like it is on horizontal panels. This makes it unusable on vertical panels.

STEPS TO REPRODUCE
1. Create a vertical panel
2. Add a Folder View widget to it

OBSERVED RESULT
Rather than being an icon, Folder View shows a list of files and folders in the space of the vertical panel.

EXPECTED RESULT
The Folder View should be an icon on the panel that pops out a list of files and folders when opened.

SOFTWARE/OS VERSIONS
Linux Kernel:  5.0.8
KDE Plasma Version: 5.15.80 (git master)
KDE Frameworks Version: 5.58 (git master)
Qt Version: 5.12.3
Comment 1 Kai Uwe Broulik 2019-04-23 10:52:33 UTC
It's supposed to be a list in vertical panels but the minimum size seems broken/way too small.
Comment 2 Nate Graham 2019-11-05 18:24:15 UTC
*** Bug 364410 has been marked as a duplicate of this bug. ***
Comment 3 Aldoo 2020-05-15 09:44:08 UTC
There is clearly some issue with automatic choice of mode between icon and list.
Notwithstanding, adding a simple checkbox, in the widget settings, to let the user choose would solve many problems. Even in cases where there is room for a list view, some users may prefer a simple icon anyway.
Comment 4 Bug Janitor Service 2020-12-23 20:53:18 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-desktop/-/merge_requests/278
Comment 5 Nate Graham 2021-01-07 16:45:37 UTC
Git commit 607675367baab275c2faab255ca85df0b9a5e3f1 by Nate Graham.
Committed on 07/01/2021 at 16:45.
Pushed by ngraham into branch 'master'.

[folder view] Fix display on not-skinny vertical panels

Folder view has some logic to switch between its compact and full
representations for vertical panels, but it is faulty, and causes the
switch to happen at approximately 50px width. At this width, the display
is completely unusable as the text is elided away into nothingness.

This commit remedies the situation by imposing a more sane minimum width
based on gridUnit. With the default 10pt font, the minimum thickness is
now 144px, which leaves enough room for the labels to be at least
partially visible. At lower panel widths, the applet displays its compact
representation.
FIXED-IN: 5.18.7

M  +2    -2    containments/desktop/package/contents/ui/main.qml

https://invent.kde.org/plasma/plasma-desktop/commit/607675367baab275c2faab255ca85df0b9a5e3f1
Comment 6 Nate Graham 2021-01-07 16:50:08 UTC
Git commit dec75f0da52562b719b4d259b9061a3665b6c16b by Nate Graham.
Committed on 07/01/2021 at 16:49.
Pushed by ngraham into branch 'Plasma/5.18'.

[folder view] Fix display on not-skinny vertical panels

Folder view has some logic to switch between its compact and full
representations for vertical panels, but it is faulty, and causes the
switch to happen at approximately 50px width. At this width, the display
is completely unusable as the text is elided away into nothingness.

This commit remedies the situation by imposing a more sane minimum width
based on gridUnit. With the default 10pt font, the minimum thickness is
now 144px, which leaves enough room for the labels to be at least
partially visible. At lower panel widths, the applet displays its compact
representation.
FIXED-IN: 5.18.7

M  +2    -2    containments/desktop/package/contents/ui/main.qml

https://invent.kde.org/plasma/plasma-desktop/commit/dec75f0da52562b719b4d259b9061a3665b6c16b