-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Riot Desktop system tray icon invisible #12518
Comments
+1 happend a week ago with Awesome WM:
|
+1
|
I attached a rageshake to this issue. |
The issue persists after updating electron to 8.0.2 |
The issue appears to be caused by Electron 8. |
Given the above, we're reverting to an older version of electron - release should be out tomorrow. Will leave this open to track the electron 8 issues. |
Presumably this is related to electron/electron#21445, see electron/electron#21445 (comment) specifically |
I encountered this issue and resolved it by installing and activating the "appindicator" extension on my gnome-shell desktop. |
Are you sure about this change? Riot 1.5.11 on Electron 8.0.1 tray icon works just fine in KDE openSUSE - seems first time it looks and works fine - first completely fixed Electron version. Are You sure 7.x will work and look fine with correct icon resolution? |
I have the AppIndicator extension as well but it only works for a little while. After a few minutes the icon disappears and I see this crash in my logs:
Arch Linux with gnome shell. |
It really depends on the type of Linux system used. If used Linux system/DE/panel/tray-plugin doesn't support Status Notifier tray icons standard then you won't get the tray icon as @electron since v8 switched to use Status Notifier tray icons (the change actually came from the upstream project named @chromium). |
@MazeChaZer, i3 related stuff (apparently there is no Status Notifier tray icons support for i3 implemented yet): |
You could try
|
@akontsevich we're consciously not going to upgrade to electron 8 at this time, given people's icons are going missing. We'd rather they were blurry than not there. For now let's keep this issue about the lack of icon rather than KDE's blurry icons. |
Was so many hopes on Riot, for now it is completely unusable because of such bugs, so using Viber instead which works out of the box with encryption, etc. |
Not upgrading Electron or other major deps might potentially lead to sticking with security issues that were revealed and fixed in newer @chromium / @electron versions. Btw, in some cases Electron v8 makes tray icons blurry on HiDPI Linux desktops electron/electron#22207 |
Issue tracking SNI/libappindicator support for the flatpak build: flathub/im.riot.Riot#100 |
Please note that at least on Xfce, the riot-update from 1.5.x to 1.6 broke the status-notification icon (again). I use the official Deb package and run Debian Buster and have the xfce4-statusnotifier-plugin installed and running (;, as it is mentioned in the corresponding electron issue. |
+1 riot-web: 1.6.0 |
In Awesome on Arch https://git.sr.ht/~steef/snixembed has resolved the issue. |
If I get this right, snixembed translates status-notifier protocol to systemtray-spec protocol. While this (quoting the author) "dirty hack" may be acceptable as a temporary workaround for panels/window managers which do not have a status-notification area (but only a tray), it certainly makes no sense in other desktop environments like Mate or Xfce4, which do provide it. |
Could that be a dbus problem? The Riot flatpak had a similar issue. It was fixed yesterday with a dbus permission, and verified on Xfce 4.14 desktop with the Status Notifier panel plugin. Note that the Riot flatpak uses the executable from the official deb package, but it may use its own Electron libs. |
+1 riot-web: 1.6.0 |
Systray icon not showing here either. |
+1 riot-desktop: 1.6.0 |
No system tray icon here as well after updating to Riot 1.6.0 on latest Debian Buster. |
It isn't something which Riot can do anything about. It is an issue between Electron (Chromium) and your DE/WM. |
As stated above. Riot upgraded to electron 8 which uses chromium 80.x which changed its tray icon support. |
Other electron apps will suffer the same fate when they choose to upgrade. |
So how can i use a lot of apps in electron (like stremio, telegram) and all of them using the same version as riot and all of these i get the tray icons properly?
Yea, no, that's not happening. |
Most electron apps are not yet using electron 8. For telegram if you mean tdesktop then it isn't electron: https://github.com/telegramdesktop/tdesktop |
Stremio for linux uses electron. You're showing me Windows version. |
There's some reports that making the source image 32x32 resolves the issue for most desktop environments - we might want to try that if we can do so without impacting other environments (like Windows, mac, etc). Ideally we'd somehow specify the tray icon image specifically and for specific environments. |
https://aur.archlinux.org/packages/stremio/
Doesn't look like it? |
+1. i3 (i3status/i3bar or i3blocks, doesn't matter), no icon in tray after upgrading to 1.6.x. |
I'm locking this because it's not something we can fix at the moment. It requires OS/window manager vendors to talk to Chromium about how this is supposed to work. At the moment we cannot downgrade electron for various reasons external to this one - we are trying to work out a solution that gets us the benefit of electron 8 (and therefore latest Chromium) as well as having a task tray icon. |
Electron fixed this upstream. riot-desktop will be using the new electron in the next release. |
Description
Since a week or so, the system tray icon of Riot Desktop is not displayed anymore under i3/i3bar.
Steps to reproduce
Version information
The text was updated successfully, but these errors were encountered: