Bug 477813 - Vertical panel regression in Plasma 6 Beta: huge min widget height
Summary: Vertical panel regression in Plasma 6 Beta: huge min widget height
Status: RESOLVED DUPLICATE of bug 482612
Alias: None
Product: plasmashell
Classification: Plasma
Component: Panel (show other bugs)
Version: 5.92.0
Platform: Arch Linux Linux
: NOR normal
Target Milestone: 1.0
Assignee: Plasma Bugs List
URL:
Keywords: qt6
Depends on:
Blocks:
 
Reported: 2023-12-01 02:23 UTC by andy
Modified: 2024-05-01 20:41 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
vertical task bar: before & after (90.73 KB, image/png)
2023-12-01 02:23 UTC, andy
Details

Note You need to log in before you can comment on or make changes to this bug.
Description andy 2023-12-01 02:23:04 UTC
Created attachment 163684 [details]
vertical task bar: before & after

I'm on arch linux, wayland. I just tried switching to the [kde-unstable] repo to test the Plasma 6 Beta.

I use a vertical panel with my app menu and task manager. It's broken in Plasma 6 because the min widget height is now proportional to the width of the panel.  It wasn't an issue in Plasma 5 (is https://bugs.kde.org/show_bug.cgi?id=466566  obsolete?).

See the attached screenshot for Plasma 5 to 6 before/after. (Ignore the timestamps, I took the "Before" after I reverted to stable release). 


STEPS TO REPRODUCE
0. Start wayland session
1. Place a panel on the right side of the screen (e.g. your main "taskbar" panel). Make it 300 pixels wide
2. Have the following widgets in the panel from top to bottom:  Application Menu, Virtual desktop pager, Task Manager, System Tray, Digital Clock, and Peek at Desktop
3. In Task Manager settings, choose "Fill free space on Panel"

OBSERVED RESULT
In Plasma 6:
- I think I'm seeing every widget in the panel is min 200-300 pixels height (I wanted to measure this in edit mode with mouse hovering over individual widgets to show the precise sizes, but taking a screenshot in Edit Mode hid all the edit elements)

EXPECTED RESULT
In Plasma 5: 
- Application Menu buttons is ~40 pixels tall and others are similarly minimal
- the only widget that grows into available space is the Task Manager

SOFTWARE/OS VERSIONS
Linux/KDE Plasma: 6.1.64-1-lts
(available in About System)
KDE Plasma Version:  5.90.0-1
KDE Frameworks Version:  whatever it was in https://archlinux.org/packages/kde-unstable/
Qt Version: whatever it was in https://archlinux.org/packages/kde-unstable/

ADDITIONAL INFORMATION
Unrelated but there were also ~10-20 sad faces in the System Tray for what looked like crashed notifications. This was on my first login to the beta.
Comment 1 Bug Janitor Service 2023-12-06 16:45:07 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-desktop/-/merge_requests/1901
Comment 2 Nate Graham 2023-12-18 22:00:26 UTC
Git commit 0d14b0b153ef2bdbe1e097edb7415f3e3cef21bc by Nate Graham, on behalf of Niccolò Venerandi.
Committed on 18/12/2023 at 23:00.
Pushed by ngraham into branch 'master'.

Simplify code for applet sizes and respect max width/height of non-filling applets

Some applets don't fillWidth/Height, and yet they still set a MaxWidth/MaxHeight
(and, usually, a minWidth/minHeight that's the same value). This respects the
requested size, and tries to simplify the code using `||` as a fallback operator
instead of nested `? :` operators.
Related: bug 476925

M  +2    -2    applets/kickoff/package/contents/ui/main.qml
M  +23   -46   containments/panel/contents/ui/main.qml

https://invent.kde.org/plasma/plasma-desktop/-/commit/0d14b0b153ef2bdbe1e097edb7415f3e3cef21bc
Comment 3 andy 2024-01-18 22:11:34 UTC
I am trying the latest version of the beta and I can confirm the application menu icon in the task manager in a vertical panel is still huge.

KDE Plasma Version: 5.92.0
KDE Frameworks Version: 55.248.0
Qt Version: 6.7.0
Kernel Version: 6.6.12-1-lts
Graphics Platform: Wayland
Comment 4 Nate Graham 2024-01-18 23:00:10 UTC
Ah, I see the issue. This is actually intentional at this point in time, but there is a discussion at Bug 466566 regarding a proposal for changing the status quo.

*** This bug has been marked as a duplicate of bug 466566 ***
Comment 5 Nate Graham 2024-03-07 19:13:54 UTC

*** This bug has been marked as a duplicate of bug 467004 ***
Comment 6 Nate Graham 2024-05-01 20:41:22 UTC

*** This bug has been marked as a duplicate of bug 482612 ***