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

[Bug]: No more tray icon on Linux #2681

Closed
6 tasks done
dtrunk90 opened this issue Apr 17, 2023 · 13 comments
Closed
6 tasks done

[Bug]: No more tray icon on Linux #2681

dtrunk90 opened this issue Apr 17, 2023 · 13 comments
Labels

Comments

@dtrunk90
Copy link

dtrunk90 commented Apr 17, 2023

Before you file a bug report

Mattermost Desktop Version

Version 5.3.1 commit: 2c3d80f

Operating System

Ubuntu Budgie Linux 20.04.6 LTS x64

Mattermost Server Version

No response

Steps to reproduce

Enable tray icon in settings and restart mattermost

Expected behavior

Tray icon should be visible

Observed behavior

There's no more tray icon

Log Output

[2023-04-17 09:09:57.321] [debug] DownloadsManager.init { 'this.json': {}, validatedJSON: {} }
[2023-04-17 09:09:57.327] [debug] DownloadsManager.saveAll
[2023-04-17 09:09:57.328] [debug] DownloadsManager.checkForDeletedFiles
[2023-04-17 09:09:57.370] [error] Log level set to: error

Additional Information

Workaround:

$ apt-cache policy mattermost-desktop
mattermost-desktop:
  Installed: 5.3.1-496
  Candidate: 5.3.1-496
  Version table:
 *** 5.3.1-496 500
        500 https://deb.packages.mattermost.com stable/main amd64 Packages
        100 /var/lib/dpkg/status
     5.2.2-382 500
        500 https://deb.packages.mattermost.com stable/main amd64 Packages
     ...
$ sudo apt install mattermost-desktop=5.2.2-382

Now the tray icon is back again.
So the issue got introduced in either 5.3.0 (which i didn't tested because there doesn't seem to be a final release) or 5.3.1. I guess it was this commit: 8efa348
and the issue is somewhere in electron 23.1.0, 23.1.1 or 23.1.2.

Maybe related to electron/electron#36602

@dtrunk90 dtrunk90 changed the title [Bug]: No more tray icon [Bug]: No more tray icon on Linux Apr 17, 2023
@razzeee
Copy link
Contributor

razzeee commented Apr 19, 2023

Is it really gone? I seem to have a black icon on a black background now, with gnome 44

@dtrunk90
Copy link
Author

Yes, it's gone on Budgie.

@IceWreck
Copy link

I have it on flatpak fedora but the latest update did mess something up.

The tray menu for mattermost now has a weird theme, and the icon looks pixelated unlike the high res tray icon before.

Other than the tray, the electron menu (File, Edit, View, Help, etc) also looks different (doesn't respect theme) compared to before.

@dtrunk90
Copy link
Author

I have it on flatpak fedora but the latest update did mess something up.

The tray menu for mattermost now has a weird theme, and the icon looks pixelated unlike the high res tray icon before.

Other than the tray, the electron menu (File, Edit, View, Help, etc) also looks different (doesn't respect theme) compared to before.

Sounds like a totally different issue related to flatpak due to it's sandboxing. At least you have a tray icon.

@IceWreck
Copy link

The packaging method did not change but something happened with the tray icon due to the latest update. Seems related.

Mine is pixelated, somebody else claims they have a black icon and you dont have one at all. Something happened.

@dtrunk90
Copy link
Author

Btw: I've noticed the same for Element (matrix Chat) which is also an Electron app.

@IceWreck
Copy link

Does element use the same electron version ? Could be that something changed on electron's end.

@razzeee
Copy link
Contributor

razzeee commented Apr 21, 2023

My problem was also with the flatpak. But it seems like I can get it to work again, by allowing the flatpak to access the session bus - but that's probably not the correct solution.

@razzeee
Copy link
Contributor

razzeee commented Apr 21, 2023

@dtrunk90
Copy link
Author

Does element use the same electron version ? Could be that something changed on electron's end.

Element uses ^23.0.0 (which is 23.2.4) in its latest version while mattermost uses 23.1.2 in its latest version.

@dtrunk90
Copy link
Author

dtrunk90 commented Apr 21, 2023

Since it's not a mattermost issue I'm closing this. It's definitely related to electron. They moved to StatusNotifier spec instead of libappindicator. Budgie's system tray is based on XEmbed and they're already about to fix this in Budgie 10.8.

So, the workaround for now is to either downgrade the software (happens on discord, element, mattermost because they all are electron apps) or just disable the "minimize to tray" features for now and keep the software always open.

@dtrunk90
Copy link
Author

Reopening due to electron/electron#40493 (comment)

@dtrunk90 dtrunk90 reopened this Nov 13, 2023
@devinbinnie
Copy link
Member

@dtrunk90 I've commented on the Electron issue. We can continue the discussion there. Closing.

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

No branches or pull requests

4 participants