Bug 420385 - Folder view popup reduced from 3 columns to 2 when a scrollbar is needed
Summary: Folder view popup reduced from 3 columns to 2 when a scrollbar is needed
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Folder (show other bugs)
Version: 5.18.3
Platform: Ubuntu Linux
: NOR minor
Target Milestone: 1.0
Assignee: Eike Hein
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-04-21 14:23 UTC by Stefano
Modified: 2020-04-21 18:33 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In: 5.18.5


Attachments
2 columns accidental folder view (85.28 KB, image/png)
2020-04-21 14:23 UTC, Stefano
Details
Popup video (668.93 KB, video/mp4)
2020-04-21 17:24 UTC, Stefano
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Stefano 2020-04-21 14:23:44 UTC
Created attachment 127749 [details]
2 columns accidental folder view

SUMMARY
When opening a folder view popup of a folder with more than 6 items, thus requiring a scrollbar, the content is reduced from 3 to 2 columns. See screenshot.
I tried this with different plasma themes and fonts. This example is done with default plasma theme and Liberation Sans 11 font. The same happens with Noto Sans 10.

STEPS TO REPRODUCE
1. Create a folder on your desktop with more than 6 items in it or place a Folder View containing some folder with more than 6 items in it
2. Click on the Arrow Up icon on that folder to open a popup

OBSERVED RESULT
The items are placed on two columns instead of three when there's also a scrollbar.


EXPECTED RESULT
The popup should contain tree columns of items even when there's a scrollbar in it.


SOFTWARE/OS VERSIONS
Operating System: Kubuntu 19.10
KDE Plasma Version: 5.18.3
KDE Frameworks Version: 5.67.0
Qt Version: 5.12.4
Kernel Version: 5.3.0-46-generic
OS Type: 64-bit
Processors: 8 × Intel® Core™ i7-4790 CPU @ 3.60GHz
Memory: 15,6 GiB
Comment 1 Nate Graham 2020-04-21 17:10:31 UTC
Hmm, works for me with the default font and size. You can just make the widget a tiny bit wider, and then this won't happen.
Comment 2 Stefano 2020-04-21 17:24:22 UTC
(In reply to Nate Graham from comment #1)
> Hmm, works for me with the default font and size. You can just make the
> widget a tiny bit wider, and then this won't happen.

Is it possible to make the _popup_ larger?

See second attachment. Maybe I explained myself badly
Comment 3 Stefano 2020-04-21 17:24:54 UTC
Created attachment 127755 [details]
Popup video
Comment 4 Nate Graham 2020-04-21 17:49:27 UTC
Oh my mistake. I thought this was the widget, not the popup.
Comment 5 Nate Graham 2020-04-21 17:55:20 UTC
Git commit 080e9dd6a4dc5e1985b11a91b4cebd945c6adfe8 by Nate Graham.
Committed on 21/04/2020 at 17:53.
Pushed by ngraham into branch 'Plasma/5.18'.

[Folder View] Update popup dialog's hardcoded scrollbar width to fix faulty sizing

Will submit a patch to calculate the width automtically so we don't need
to hardcode this and manually update it again in the future.
FIXED-IN: 5.18.5

M  +1    -1    containments/desktop/package/contents/ui/FolderViewDialog.qml

https://commits.kde.org/plasma-desktop/080e9dd6a4dc5e1985b11a91b4cebd945c6adfe8
Comment 6 Stefano 2020-04-21 18:14:08 UTC
That was quick! Thanks! :D
Comment 7 Nate Graham 2020-04-21 18:33:20 UTC
You're welcome!