Bug 365570 - Systray icons are big and pixelated after update to 5.7.1
Summary: Systray icons are big and pixelated after update to 5.7.1
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: System Tray (show other bugs)
Version: 5.7.1
Platform: Exherbo Linux
: NOR normal
Target Milestone: 1.0
Assignee: Plasma Bugs List
URL:
Keywords: regression
Depends on:
Blocks:
 
Reported: 2016-07-12 21:52 UTC by Heiko Becker
Modified: 2017-01-30 15:54 UTC (History)
19 users (show)

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


Attachments
screenshot of the systray icons (18.59 KB, image/png)
2016-07-12 21:52 UTC, Heiko Becker
Details
Huge systray icons, no padding (7.20 KB, image/png)
2016-07-12 22:32 UTC, Janek Bevendorff
Details
Comparison with other elements of the panel (32.33 KB, image/png)
2016-07-13 06:59 UTC, tesfabpel
Details
huge systray icons 5.7.1 (45.17 KB, image/png)
2016-07-13 21:57 UTC, Nikola Schnelle
Details
Terrible huge systray icons (75.29 KB, image/png)
2016-07-14 05:52 UTC, kikadf
Details
Refactored: John Salatas's plasma-workspace-5.7.0-systray_iconsize.patch (3.61 KB, patch)
2016-07-17 12:46 UTC, Bob Wya
Details
Refactored for Plasma 5.7.2: John Salatas's plasma-workspace-5.7.0-systray_iconsize.patch (3.68 KB, patch)
2016-07-20 23:19 UTC, Bob Wya
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Heiko Becker 2016-07-12 21:52:04 UTC
After the update to Plasma 5.7.1 my systray icons have grown and look a bit squeezed into the panel and pixelated.

I'll attach a screenshot.

Reproducible: Always
Comment 1 Heiko Becker 2016-07-12 21:52:50 UTC
Created attachment 100046 [details]
screenshot of the systray icons
Comment 2 Janek Bevendorff 2016-07-12 22:31:41 UTC
I also have huge icons with no padding around them (see attached screenshot, compare it to the Reshift Control widget to the left or the clock to the right) since 5.7.1. I assume this is a regression introduced by the fix for issue #364431.
Comment 3 Janek Bevendorff 2016-07-12 22:32:07 UTC
Created attachment 100049 [details]
Huge systray icons, no padding
Comment 4 Heiko Becker 2016-07-12 23:40:41 UTC
(In reply to Janek Bevendorff from comment #2)
> I also have huge icons with no padding around them (see attached screenshot,
> compare it to the Reshift Control widget to the left or the clock to the
> right) since 5.7.1. I assume this is a regression introduced by the fix for
> issue #364431.

Indeed. If I revert 1ba0e6e5ddd598e9b0cf87a654467f97c06b26c5 (or just edit /usr/share/plasma/plasmoids/org.kde.plasma.private.systemtray/contents/ui/main.qml) systray icons return to an appropriate size.
I also discovered https://git.reviewboard.kde.org/r/128400/ which would make the size configurable (only targeted at master though).
Comment 5 tesfabpel 2016-07-13 06:59:13 UTC
Created attachment 100055 [details]
Comparison with other elements of the panel
Comment 6 Nikola Schnelle 2016-07-13 21:57:49 UTC
Created attachment 100083 [details]
huge systray icons 5.7.1
Comment 7 kikadf 2016-07-14 05:52:55 UTC
Created attachment 100089 [details]
Terrible huge systray icons
Comment 8 Mike Goodwin 2016-07-14 13:20:19 UTC
Also seeing this on 1080p https://bugs.kde.org/show_bug.cgi?id=364431#c13 (my comment with screenshot from the original bug)
Comment 9 Xavier Corredor Llano 2016-07-15 00:06:44 UTC
This problem should be fixed with this commit: https://quickgit.kde.org/?p=plasma-workspace.git&a=commitdiff&h=7f060403a2d44fc76b2b14fc9ebf43ee272d1c54

Please test it
Comment 10 Bob Wya 2016-07-17 12:46:30 UTC
Created attachment 100134 [details]
Refactored: John Salatas's plasma-workspace-5.7.0-systray_iconsize.patch

I don't see how watering down John Salatas's original patch into a secret/ hidden setting - so basically it fixes nothing - achieves anything?? Last time I checked Plasma isn't going after the i3 DE market!!

I remember requesting this feature back in 2014 (!!) on the VDG forums - I was not alone...
https://forum.kde.org/viewtopic.php?f=285&t=121695
If user requests for features are to be ignored... What was the point of the VDG??

I've finally got the feature I wanted - thanks to John Salatas... But not really - because I've got to refactor this patch - every plasma-workspace release now... :-(

Just to add Vertical side panels are still broken - by default - with no "user visible" switch to fix them.
Comment 11 Christian Herenz 2016-07-17 17:14:46 UTC
I can confirm this bug. Happened when updating from 5.7.0 to 5.7.1

Guys, do you test stuff before you release?
Comment 12 Bob Wya 2016-07-20 23:19:54 UTC
Created attachment 100220 [details]
Refactored for Plasma 5.7.2: John Salatas's plasma-workspace-5.7.0-systray_iconsize.patch

Sigh :-(
Comment 13 Evgeny Brazgin 2016-07-23 08:00:59 UTC
The bug still appears in Plasma 5.7.2.
BTW, Bob Wya's patch works well.
Comment 14 Christian Herenz 2016-07-24 17:24:17 UTC
I can not confirm this - after update to 5.7.2 I have the icon size in the systray that I had with versions <= 5.7.0 which I assume to be the correct size.  Only version 5.7.1 showed the big icons in the panel for me.
Comment 15 Mike Goodwin 2016-07-24 21:03:56 UTC
@cherenz@aip.de 

If you're using Fedora, I persuaded rdeiter to patch it at the package level. In which case, you won't see it in Fedora.
Comment 16 Christian Herenz 2016-07-25 09:36:06 UTC
I'm on openSUSE and I use the package from the KDE:Frameworks5 repository: https://build.opensuse.org/package/show?project=KDE%3AFrameworks5&package=plasma5-desktop
Comment 17 Evgeny Brazgin 2016-07-26 06:11:49 UTC
@cherenz@aip.de 
You have no problem, because OpenSuse has already included the patch into their build: https://build.opensuse.org/package/view_file/KDE:Frameworks5/plasma5-workspace/Configuration-option-for-System-Tray-icon-size.patch?expand=1
So, waiting for this patch to be released in 5.7.3 (it is in master branch in git, but didn't come to 5.7.2).
Comment 18 Evgeny Brazgin 2016-10-01 07:51:09 UTC
Still not fixed in 5.7.5 :(
Looking into git, I see that the fix is only merged to 5.7.95/5.8.
Comment 19 Janek Bevendorff 2016-10-05 16:05:48 UTC
In 5.8.0, minimized windows show a black rectangle if the option to keep
thumbnails of hidden windows in memory is disabled. I don't think that's
any more beautiful than pixelated icons. ;-)