Bug 366459

Summary:	system tray icons are 32px in size but plasma breeze didn't support this size
Product:	[Plasma] plasmashell	Reporter:	andreas <kainz.a>
Component:	System Tray widget	Assignee:	Plasma Bugs List <plasma-bugs-null>
Status:	RESOLVED FIXED
Severity:	normal	CC:	kde, kilian, notmart
Priority:	NOR
Version First Reported In:	master
Target Milestone:	1.0
Platform:	Other
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:
Attachments:	22px system tray icon size 32px system tray

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