Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tray icon is missing since v5.0.0 (Linux) #936

Closed
1 of 3 tasks
fauust opened this issue Apr 21, 2020 · 18 comments
Closed
1 of 3 tasks

Tray icon is missing since v5.0.0 (Linux) #936

fauust opened this issue Apr 21, 2020 · 18 comments

Comments

@fauust
Copy link

fauust commented Apr 21, 2020


Hi! Since I upgraded to version 5.0.0 the tray icon is missing from systray.

  • Operating System:
    • Windows
    • Linux/Ubuntu
    • macOS

WM: i3 version 4.18 + polybar 3.4.2
OS: Debian Buster (amd64)

  • Clear steps to reproduce the issue:
    Install latest Zulip desktop version from zulip repository (because of The SUID sandbox helper binary was found #912, I am not able to test with the Appimage version).

  • Relevant error messages and/or screenshots:
    Zulip is working perfectly but the tray icon is missing (I am getting notification with dunst).

@akashnimare
Copy link
Member

@fauust did you install the app using our deb installer from the release page?

@fauust
Copy link
Author

fauust commented Apr 21, 2020

From https://zulipchat.com/help/desktop-app-install-guide

sudo apt-key adv --keyserver pool.sks-keyservers.net --recv 69AD12704E71A4803DCA3A682424BE5AE9BD10D9
echo "deb https://dl.bintray.com/zulip/debian/ stable main" | \
sudo tee -a /etc/apt/sources.list.d/zulip.list
sudo apt update
sudo apt install zulip

@devurandom
Copy link

devurandom commented Apr 21, 2020

I can confirm this behaviour for the Flatpak on multiple systems:

(#910 (comment)):
On KDE Plasma, using the Flatpak, I never see any tray icon at all on these two systems (i.e. not even right after start of the application, which is a problem if startMinimized=true in ~/.config/Zulip/config/settings.json):

Operating System: Gentoo Linux 
KDE Plasma Version: 5.18.4
KDE Frameworks Version: 5.68.0
Qt Version: 5.14.1
Kernel Version: 5.5.14
OS Type: 64-bit
Operating System: Fedora 31
KDE Plasma Version: 5.17.5
KDE Frameworks Version: 5.67.0
Qt Version: 5.13.2
Kernel Version: 5.5.13-200.fc31.x86_64
OS Type: 64-bit

See-Also: flathub/org.zulip.Zulip#18

@abhigyank
Copy link
Collaborator

Hi @fauust @devurandom Chromium has changed how it shows tray icon, probably uses StatusNotifier tray icons now. This comment has the fix for some OSes, you can check other threads to support your OS if you can fine. If your OS doesn't support StatusNotifier tray, you might want to place an issue in that OS's tracker.
@akashnimare there isn't anything for us to do here. We can close.

@abhigyank
Copy link
Collaborator

@fauust For polybar, I found the upstream tracker here - polybar/polybar#2010 , they haven't added support yet, but seems to be planning on doing it :)

@abhigyank
Copy link
Collaborator

@akashnimare I am putting a list up of StatusNotifier tray icons support status from different threads I found, so that it is easier for us to link -

Some Linux users who still experience the tray icon issue with Electron-based apps might want to try https://git.sr.ht/~steef/snixembed thing.

@fauust
Copy link
Author

fauust commented Apr 23, 2020

@abhigyank, thanks for pointing me the snixembed project, that seems to work for the moment, see polybar/polybar#2010 (comment).

@akashnimare
Copy link
Member

@abhigyank awesome, we should document this somewhere (maybe in limitations section). cc @manavmehta

@devurandom
Copy link

Hi @fauust @devurandom Chromium has changed how it shows tray icon, probably uses StatusNotifier tray icons now. This comment has the fix for some OSes, you can check other threads to support your OS if you can fine. If your OS doesn't support StatusNotifier tray, you might want to place an issue in that OS's tracker.

I am not sure I understand. Wayland environments do not support the X11 XEMBED extension for obvious reasons and never did. There is only SNI (Status Notifier Item) on Wayland. Swaybar supports it, but does not display a Zulip icon. Waybar also supports it and also does not display a Zulip icon. Other Flatpak applications' SNI (e.g. com.borgbase.Vorta) display their icons fine, without using xembedsniproxy or similar hacks.

@abhigyank
Copy link
Collaborator

abhigyank commented Apr 24, 2020

@devurandom It is not not really a problem with Electron or Zulip. There isn't anything either can do here. If SNI support is present, I'd assume tray should work. I have read it works out of the box for KDE DE. You could create an upstream issue at Swaybar and Waybar. I did find a thread (probably on Arch) where people have reported that electron apps like Signal, Riot too don't have tray icon on Swaybar (issue with Swaybar maybe?).
P.S. is Vorta an electron app?

@devurandom
Copy link

@devurandom It is not not really a problem with Electron or Zulip. There isn't anything either can do here. If SNI support is present, I'd assume tray should work.

Is Zulip using a build of Electron that includes this change? electron/electron#21904

I have read it works out of the box for KDE DE.

I will try with KDE Plasma once I'm back on the computer running it.

You could create an upstream issue at Swaybar and Waybar.

There do not seem to be any open ATM:

Will open a report once I am back on the machine running Sway.

I did find a thread (probably on Arch) where people have reported that electron apps like Signal, Riot too don't have tray icon on Swaybar (issue with Swaybar maybe?).

Link?

P.S. is Vorta an electron app?

Does not look like it. Looks more like Python + Qt: https://github.com/flathub/com.borgbase.Vorta/blob/master/com.borgbase.Vorta.json

@abhigyank
Copy link
Collaborator

@devurandom Can you try in the latest release? Electron has reverted to the older way, so it should work as before.

@fauust
Copy link
Author

fauust commented Jun 25, 2020

Hi @abhigyank!
I can confirm that latest version is working, snixembed is no more needed.

@akashnimare
Copy link
Member

Alright. Closing this now. Thanks, everyone.

@devurandom
Copy link

devurandom commented Jun 25, 2020

@devurandom Can you try in the latest release? Electron has reverted to the older way, so it should work as before.

Using KDE Plasma 5.19.2 (xembedsniproxy is running) and Flatpak 1.6.3 with Zulip 5.2.0, I do not see a tray icon. Permissions are at their defaults:

[Context]
shared=network;ipc;
sockets=x11;wayland;pulseaudio;
filesystems=xdg-download;xdg-documents;xdg-pictures;

[Session Bus Policy]
org.kde.StatusNotifierWatcher=talk
org.freedesktop.Notifications=talk

@abhigyank
Copy link
Collaborator

@devurandom please try on Zulip 5.3.0 .

@pskopek
Copy link

pskopek commented Jun 26, 2020

Zulip 5.3.0 works fine on Fedora 31 using GNOME and Topicons Plus.

@Moilleadoir
Copy link

This option no longer exists in System Settings.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants