Bug 435738 - Tray icon for Albert launcher fails to display
Summary: Tray icon for Albert launcher fails to display
Status: RESOLVED UPSTREAM
Alias: None
Product: plasmashell
Classification: Plasma
Component: System Tray (show other bugs)
Version: 5.21.4
Platform: Neon Linux
: NOR normal
Target Milestone: 1.0
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-04-14 20:26 UTC by RedBearAK
Modified: 2021-04-16 22:08 UTC (History)
4 users (show)

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


Attachments
listIcons.sh (855 bytes, application/x-shellscript)
2021-04-16 21:51 UTC, Konrad Materka
Details

Note You need to log in before you can comment on or make changes to this bug.
Description RedBearAK 2021-04-14 20:26:54 UTC
SUMMARY

An application (Albert) has a tray icon that was not being displayed (invisible) in KDE Frameworks 5.79.0, then suddenly it was working in KDE Frameworks 5.80.0, then once again it is not displaying properly with the KDE Frameworks 5.81.0 update. 

Seeking assistance with a link to the exact changes that could be affecting this tray icon display issue with Albert. Don't know whether it is a bug in the KDE Frameworks that was fixed and then broken again, or a bug in the way Albert tries to add its icon to the tray. Have already been working with the Albert maintainer trying to find a fix. But it was not working in 5.79.0, definitely working fine with 5.80.0, and then broke again with 5.81.0. So the secret is somewhere in the difference between the three updates, related to the tray indicators. 

Other than the icon issue, the Albert tray indicator exists and is fully usable. The panel/tray just doesn't want to show the icon outside of Frameworks version 5.80.0. 

I'm not certain this (frameworks-plasma/libplasma) is the correct place to put this, but I couldn't find anything that seemed more relevant to a part of the KDE Frameworks that might specifically affect the indicator tray icons. Feel free to move it if necessary. 


STEPS TO REPRODUCE
1. Install Albert on a KDE system with Frameworks 5.81.0
2. Look for the Albert tray icon (looks like a 3D square with an "alpha" symbol). 
3. An item will show a tooltip of "Albert" but will have no icon displayed. 


SOFTWARE/OS VERSIONS
Linux/KDE Plasma: KDE Neon
(available in About System)
KDE Plasma Version: 5.21.4
KDE Frameworks Version: 5.79.0/5.80.0/5.81.0
Qt Version: 5.15.2
Comment 1 Konrad Materka 2021-04-15 18:32:17 UTC
Can you provide a link to the application?
Comment 2 RedBearAK 2021-04-15 21:49:16 UTC
(In reply to Konrad Materka from comment #1)
> Can you provide a link to the application?

The Albert GitHub thread that led to me asking for assistance on narrowing down the cause of this issue: 

https://github.com/albertlauncher/albert/issues/343

How to install Albert launcher: 

https://albertlauncher.github.io/installing/

Direct link to the openSUSE Open Build Service (OBS) page I usually use to install Albert on various distros: 

https://software.opensuse.org/download.html?project=home:manuelschneid3r&package=albert
Comment 3 Konrad Materka 2021-04-16 21:51:34 UTC
Created attachment 137662 [details]
listIcons.sh
Comment 4 Konrad Materka 2021-04-16 22:00:41 UTC
Just checked and the application is not setting the icon correctly. You can run attached scrict to list DBus properties of all System Tray icons. On my system for Albert is returns:
{"AttentionIconName" = [Variant(QString): ""], "AttentionIconPixmap" = [Variant: [Argument: a(iiay) {}]], "AttentionMovieName" = [Variant(QString): ""], "Category" = [Variant(QString): "ApplicationStatus"], "IconName" = [Variant(QString): ""], "IconPixmap" = [Variant: [Argument: a(iiay) {}]], "IconThemePath" = [Variant(QString): ""], "Id" = [Variant(QString): "Albert"], "ItemIsMenu" = [Variant(bool): false], "Menu" = [Variant: [ObjectPath: /MenuBar]], "OverlayIconName" = [Variant(QString): ""], "OverlayIconPixmap" = [Variant: [Argument: a(iiay) {}]], "Status" = [Variant(QString): "Active"], "Title" = [Variant(QString): "Albert"], "ToolTip" = [Variant: [Argument: (sa(iiay)ss) "", [Argument: a(iiay) {}], "", ""]], "WindowId" = [Variant(int): 0]}

Both IconName and IconPixmap are empty.

Maybe the problem is that Albert set's the icon in a weird way?
https://github.com/albertlauncher/albert/blob/dev/src/app/trayicon.cpp#L15
Most probably QIcon::fromTheme should be used (as mention in app bug report).
Comment 5 RedBearAK 2021-04-16 22:08:32 UTC
(In reply to Konrad Materka from comment #4)
> Both IconName and IconPixmap are empty.
> 
> Maybe the problem is that Albert set's the icon in a weird way?
> https://github.com/albertlauncher/albert/blob/dev/src/app/trayicon.cpp#L15
> Most probably QIcon::fromTheme should be used (as mention in app bug report).

Thanks for taking a look. I will forward this info and ask the dev to try changing it.