Bug 366459 - system tray icons are 32px in size but plasma breeze didn't support this size
Summary: system tray icons are 32px in size but plasma breeze didn't support this size
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: System Tray widget (show other bugs)
Version: master
Platform: Other Linux
: NOR normal
Target Milestone: 1.0
Assignee: Plasma Bugs List
URL:
Keywords:
: 366009 (view as bug list)
Depends on:
Blocks:
 
Reported: 2016-08-05 21:31 UTC by andreas
Modified: 2016-09-03 08:12 UTC (History)
3 users (show)

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


Attachments
22px system tray icon size (2.74 KB, image/png)
2016-08-10 08:23 UTC, andreas
Details
32px system tray (3.52 KB, image/png)
2016-08-10 08:24 UTC, andreas
Details

Note You need to log in before you can comment on or make changes to this bug.
Description andreas 2016-08-05 21:31:53 UTC
the system tray icons are shown as 32px icons but 90% of the plasma icons are only available in 22px so 90% of the icons have to be scaled by default.

Please switch to 22px icons as the kde applications use by default for toolbars or add 32px icon support for plasma breeze theme.

Reproducible: Always
Comment 1 Kai Uwe Broulik 2016-08-10 08:06:47 UTC
I don't understand. How does Breeze not "support this"? The icons look just fine here. Also, they're SVG.
Comment 2 andreas 2016-08-10 08:23:50 UTC
Created attachment 100521 [details]
22px system tray icon size

plasma-workspace/applets/systemtray/package/contents/config/main.xml
<entry name="iconSize" tpe="Int">
   <default>1</default>
Comment 3 andreas 2016-08-10 08:24:54 UTC
Created attachment 100522 [details]
32px system tray

plasma-workspace/applets/systemtray/package/contents/config/main.xml 
<entry name="iconSize" tpe="Int"> 
    <default>2</default>
Comment 4 andreas 2016-08-10 08:29:31 UTC
the plasma theme support 95% only 22px icons, cause most icons are available in 22px size. some like the klipper icon is available in 22px and 32px. In attachment 100522 [details] (32px size) you see that the clipper icon is sharp and the other icons are scalled. 

That's a design issue of breeze cause breeze icons use 1px lines so when you scale from 22px to 32px the line width will be 1.45px and it look blur. I will add the 32px icons for the breeze plasma theme but now you can see the issue.
Comment 5 Kai Uwe Broulik 2016-08-10 08:38:30 UTC
Thanks, I never noticed (or cared) about the slightly blurred lines :)
Comment 6 Marco Martin 2016-08-16 11:53:07 UTC
(In reply to andreas from comment #4)
> That's a design issue of breeze cause breeze icons use 1px lines so when you
> scale from 22px to 32px the line width will be 1.45px and it look blur. I
> will add the 32px icons for the breeze plasma theme but now you can see the
> issue.

hmm, master should *not* be 32 pixels i think?
are you sure this is still happening on master (what will be 5.8?)

on high dpi displays however, the icons *will* be scaled, so yes, needs bigger icons as well for more dense screens
Comment 7 Marco Martin 2016-08-16 12:03:50 UTC
*** Bug 366009 has been marked as a duplicate of this bug. ***
Comment 8 andreas 2016-08-16 19:02:13 UTC
Master is 22px and 5.7.x is 32px
Comment 9 andreas 2016-09-03 08:12:54 UTC
fixed in plasma 5.8