Bug 365131 - Svg icons in systray are not rendered at all
Summary: Svg icons in systray are not rendered at all
Status: RESOLVED UPSTREAM
Alias: None
Product: plasmashell
Classification: Plasma
Component: System Tray (show other bugs)
Version: 5.7.0
Platform: Arch Linux Linux
: NOR normal
Target Milestone: 1.0
Assignee: Plasma Bugs List
URL: https://bugreports.qt.io/browse/QTBUG...
Keywords:
Depends on:
Blocks:
 
Reported: 2016-07-06 00:37 UTC by Sebastian Kuźlak
Modified: 2019-10-01 15:56 UTC (History)
9 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
Screenshot of menu issue (39.59 KB, image/png)
2016-08-04 07:01 UTC, Alex Ioannou
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Sebastian Kuźlak 2016-07-06 00:37:19 UTC
Svg icons in systray are not rendered at all for qt apps ( png, jpeg and other work fine fine). There is a empty spot where the icon is suppose to be.

Reproducible: Always

Steps to Reproduce:
Run any qt app (non kde) that loads a svg icon as system tray icon.

You can test it with Qt's example project - systray ( just add svg icon to the resource file and add it to list of at window.cpp:190 )

Actual Results:  
Icons is not rendered.

Expected Results:  
Icon should be rendered.

On other DE svg icons are renedered ( tested with ubuntu, opensuse with xfce )
Comment 1 Alex Ioannou 2016-08-04 06:54:47 UTC
Reproduced here.   The icon shows up in the menu editor but in the menu itself.
Comment 2 Alex Ioannou 2016-08-04 06:55:49 UTC
Sorry meant to say it shows up in the menu editor but NOT in the menu itself.
Comment 3 Alex Ioannou 2016-08-04 07:01:14 UTC
Created attachment 100442 [details]
Screenshot of menu issue

Sorry I misread the bug title but the the issue could be related.   The screenshot shows the missing icon when you try to set an SVG image as a menu icon.
Comment 4 Marco Martin 2016-08-16 12:22:04 UTC
menu or systemtray?
Comment 5 Sebastian Kuźlak 2016-08-16 12:26:29 UTC
for me its systemtray
Comment 6 Sebastian Kuźlak 2016-08-16 12:28:09 UTC
svgs from resource files are not rendered for me
Comment 7 Sebastian Kuźlak 2016-08-16 12:30:27 UTC
ok i can confirm also Alexes problem. svgs are not rendered correctly in menu.
Comment 8 denk 2016-10-07 07:28:16 UTC
I can confirm that one, too, at least for the menu. I'm on Plasma 5.8 now but, unfortunately I cannot remember, when this started.
Comment 9 denk 2017-03-18 17:55:48 UTC
With Plasma 5.9 SVG icons are still not shown in the menu. Anything I can do to help in investigating this?
Comment 10 David Edmundson 2017-03-19 08:50:48 UTC
> You can test it with Qt's example project - systray ( just add svg icon to the resource file and add it to list of at window.cpp:190 )

Attach a diff of that please. If you've done the work, make our lives easier.

Also can you see if it's still the case if our QPT is not loaded?
(unset KDE_FULL_SESSION and XDG_CURRENT_DESKTOP)
Comment 11 denk 2017-03-19 13:45:55 UTC
I just manually created an entry in the menu, added a svg icon and additionally added that entry to the control bar. Here is a screenshot showing that the icon is rendered in the menu editor and the control bar but, not in the menu: http://i.imgur.com/L700Ixm.png

Afterwards I made the proposed changes to window.cpp and added a svg file. The screenshot shows, that the icon is also not rendered in systray: 
http://i.imgur.com/qY97jmz.png

Just to check that the original icons of the QT systray example work: http://i.imgur.com/LOyUAnJ.png
Comment 12 Kai Uwe Broulik 2017-03-19 22:28:42 UTC
Patch for application launcher: https://phabricator.kde.org/D5109

I bet the systray bug is a similar issue in another place.
Comment 13 denk 2017-03-22 08:25:46 UTC
I found another occurence of SVG icons not rendering: in the places bar of Dolphin. http://imgur.com/28iYD9C

Should I open a new bug for that or can this be tracked in this one?
Comment 14 Nate Graham 2017-09-04 13:53:57 UTC
SVG icons not working in Dolphin is being tracked with https://bugs.kde.org/show_bug.cgi?id=368951

Let's keep this bug for tracking the original issue.
Comment 15 Nate Graham 2018-05-26 14:20:42 UTC
It turns out that this is caused by a Qt bug: https://bugreports.qt.io/browse/QTBUG-63187

We investigated working around this ourselves, but that caused other issues. We'll have to wait for or drive a Qt fix. Thanks for understanding, everyone!