search

flathub / com.protonvpn.www

https://flathub.org/apps/details/com.protonvpn.www
12 stars 1 forks source link

KDE System Tray icon #100

Closed mkkvcs closed 11 months ago

mkkvcs commented 11 months ago

Since the 4.0 update, the new app incorrectly displays on the system tray, in the place of the icon only a blank space appears, however remains clickable, quick connect and show app buttons work fine. I am experiencing this issue on Fedora Silverblue, the app is installed as system flatpak and fully updated. The StatusNotifierWatcher is correctly set as talk.

doogie544 commented 11 months ago

I can verify this as well, the space is there and can be interacted with, so the app is running in the tray, but no icon displays. I am using standard fedora 38, not the silverblue version. I am also using kde and the breeze dark theme but I verified that it also does not display with the light theme either.

As a note, this is an improvement over the old app as it did not display any icon either but in that case it also didn't have even the blank space so it did not even run in the system tray.

mkkvcs commented 11 months ago

I just realised my mistake from reading your reply, I am running Fedora Kinoite (hence KDE), not silverblue. I cannot currently verify whether gnome has this issue or not, I will try it in a few days.

doogie544 commented 11 months ago

lol I didn't even catch that, I saw the title and when I saw silverblue I was just not associating it with gnome even tho I knew gnome was silverblue and kinoite was kde....

In any case it might be an issue with kde specifically I have not tried gnome either

mkkvcs commented 11 months ago

Update: I have tried com.cononical.AppMenu.Registrar set to talk, org.freedesktop.Notifications set to talk, org.kde.KGlobalSettings set to talk and org.kde.kconfig.notify set to talk, neither of which has worked. These were all my ideas based on other flatpak's permissions which had system tray icons.

proletarius101 commented 11 months ago

It seems like the tray icon initialized by AppIndicator3.Indicator.new() is created by the host system according to the instructions sent through the DBus, so the icon path must exist on the hosting system.

proletarius101 commented 11 months ago

According to https://matrix.to/#/!RfXaBjokqHAbzZrgHz:matrix.org/$SfREn4ZAfSFxVLFhA8fDB7ht-l4lkz1GGHwCk6_82dU?via=matrix.org&via=gnome.org&via=kde.org, we can use org.example.App.status-foo.png

https://github.com/flatpak/flatpak/issues/4495#issuecomment-946036227

So the icon will be exported if its name should is prefixed with the application ID

proletarius101 commented 11 months ago

Related: https://github.com/flathub/org.remmina.Remmina/issues/84

doogie544 commented 11 months ago

I hate to comment with the recent issues coming up with this package but.....

The most recent update caused a regression of this issue