Bug 417374

Summary: System Tray popups are generally too small
Product: [Plasma] plasmashell Reporter: Gerion <gerion.entrup>
Component: System TrayAssignee: Nate Graham <nate>
Status: RESOLVED FIXED    
Severity: normal CC: nate, plasma-bugs
Priority: NOR Keywords: usability
Version: 5.17.5   
Target Milestone: 1.0   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed In: 5.20
Sentry Crash Report:
Attachments: scrollbar.png

Description Gerion 2020-02-10 11:39:47 UTC
Created attachment 125815 [details]
scrollbar.png

SUMMARY

Since quite a time, I have a scrollbar in my battery widget. See the attached screenshot. While this is technically irrelevant, it is a bit annoying from the user experience view since there is plenty of free room directly under the widget.


STEPS TO REPRODUCE
1. Click on the battery icon

OBSERVED RESULT
Get a scrollbar.


EXPECTED RESULT
Get no scrollbar.


SOFTWARE/OS VERSIONS
Linux/KDE Plasma: Gentoo, Linux 4.19.97
(available in About System)
KDE Plasma Version: 5.17.5
KDE Frameworks Version: 5.66.0
Qt Version: 5.14.1
Comment 1 Nate Graham 2020-02-12 21:51:43 UTC
The scrollbar is needed because the content does not entirely fit in the view. What I think you're saying is that the view should just be bigger in the first place. However no matter how big it is, if you have enough batteries, it will always become scrollable at some point.
Comment 2 Gerion 2020-02-13 08:57:00 UTC
You are right. The maximum height can be the height of my monitor minus the Plasma control panes. This is far more than the current setting.
Comment 3 Nate Graham 2020-06-15 18:08:03 UTC
I have submitted a merge request to make the popup a bit taller by default: https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/84
Comment 4 Nate Graham 2020-07-14 19:13:53 UTC
Git commit 32e43ecf03a07b8c58b182eb5947e7c1fbdc2f25 by Nate Graham.
Committed on 14/07/2020 at 19:13.
Pushed by ngraham into branch 'master'.

[applets/systemtray] Increase height of popup a bit

This generally improves things for applets that can display a lot of
data, which are now less likely to become scrollable.
FIXED-IN: 5.20

M  +1    -1    applets/systemtray/package/contents/ui/ExpandedRepresentation.qml

https://invent.kde.org/plasma/plasma-workspace/commit/32e43ecf03a07b8c58b182eb5947e7c1fbdc2f25