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

ugly system tray icon [solved] #8819

Closed
DoctorSubtilis opened this issue Feb 20, 2019 · 11 comments
Closed

ugly system tray icon [solved] #8819

DoctorSubtilis opened this issue Feb 20, 2019 · 11 comments

Comments

@DoctorSubtilis
Copy link

  • Linux (Kubuntu 18.04)

Description

The system tray icon seems very strange and ugly

Steps to reproduce

  • launch riot-web
  • icon appears in system tray
  • but she is not the correct riot icon, she seems in wrong size

riot icon bug
[the riot icon is the blue one]

Log: not sent

@drathir
Copy link

drathir commented Feb 20, 2019

Interesting what i can confirm that Arch have nice (i guess valid) one: image

@DoctorSubtilis
Copy link
Author

DoctorSubtilis commented Feb 20, 2019

It's the same I have in KDE menu: but not in system tray :(

@zink-chimaera
Copy link

It's the same I have in KDE menu: but not in system tray :(

Same here. Could reproduce on Neon 16.04.

@Ksenar
Copy link

Ksenar commented Feb 21, 2019

kde neon
solved the problem with the installation libappindicator3
sudo apt install libappindicator3-1
and launch riot through XDG_CURRENT_DESKTOP=Unity
XDG_CURRENT_DESKTOP=Unity /opt/Riot/riot-web --hidden

@DoctorSubtilis
Copy link
Author

DoctorSubtilis commented Feb 21, 2019

Thank you very much: solved!
riot bug risolto

@DoctorSubtilis DoctorSubtilis changed the title ugly system tray icon ugly system tray icon [solved] Feb 21, 2019
@jryans
Copy link
Collaborator

jryans commented Feb 21, 2019

Does this mean riot-web should depend on libappindicator3? (My knowledge of Linux desktop environments is limited...)

@DoctorSubtilis
Copy link
Author

I had already installed libappindicator3; if you don't, you can try to install it.

@jryans
Copy link
Collaborator

jryans commented Feb 21, 2019

Okay, will close for now I guess... If there's something Riot itself can do to improve this, please let us know.

@jryans jryans closed this as completed Feb 21, 2019
@zink-chimaera
Copy link

So is this an error within KDE? If not you should fix the behaviour depending on XDG_CURRENT_DESKTOP imo. I'm currently hot-fixing the .desktop file locally but I wouldn't expect that from every user. ¯_(ツ)_/¯

Here's my /usr/share/applications/riot-web.desktop:
`
[Desktop Entry]
Name=Riot
Comment=A feature-rich client for Matrix.org
Exec=XDG_CURRENT_DESKTOP=Unity "/opt/Riot/riot-web" %U
Terminal=false
Type=Application
Icon=riot-web
StartupWMClass=riot
Categories=Network;InstantMessaging;Chat;

`

@jryans
Copy link
Collaborator

jryans commented Feb 21, 2019

If we should edit the .desktop file for everyone, we could do that, but it's not clear to me whether that's actually correct or not.

@akontsevich
Copy link

Think #7553 and #7624 could be closed for now: works fine if libappindicator3 (libappindicator-gtk3) not installed. Unlike #6767 and #8819 (icon look) should be reopened and fixed: stopped to work even if I run with Exec=XDG_CURRENT_DESKTOP=Unity riot %U. For now icon looks like blank black square in KDE which is weird. I think local electron version used to build riot should be manually patched like Skype done previous year, where icon looks good for now with whatever appindicator is used or not, Unity desktop set or not. Otherwise we can wait too long it will be fixed in Electron. Makes sense?

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

No branches or pull requests

6 participants