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

[Linux] Tray menu not working #102

Closed
anatoli26 opened this issue Nov 20, 2016 · 11 comments
Closed

[Linux] Tray menu not working #102

anatoli26 opened this issue Nov 20, 2016 · 11 comments

Comments

@anatoli26
Copy link

When the app is minimized to tray, the tray menu on Ubuntu 16.04 is not working, i.e. clicking on the icon shows the menu, but the "Toggle window" and "Quit" options have no effect on the app. The window doesn't appear, nor the app exits on Quit.

At the same time, the auto-type combination works well, i.e. it brings to front the credentials window.

@TheZ3ro
Copy link
Contributor

TheZ3ro commented Nov 20, 2016

I think you are on the same macchine and you use Unity as DesktopEnvironment but please fill this form

## Your Environment
* KeePassXR version/commit used: 
* Qt version (e.g. Qt 5.3): 
* Compiler (e.g. Clang++3.6.0): 
* Operating System and version: 
* Desktop Environment:

I think it's a problem under unity or your desktop environment. On my XFCE it works fine.
When I know your DE I can do some test to reproduce this bug

@anatoli26
Copy link
Author

@TheZ3ro, here it goes:

  • KeePassXR version/commit used: 29f9359
  • Qt version (e.g. Qt 5.3): Qt version 5.5.1, QMake version 3.0
  • Compiler (e.g. Clang++3.6.0): gcc 5.4.0 20160609 (Ubuntu 5.4.0-6ubuntu1~16.04.4)
  • Operating System and version: Ubuntu 16.04.1 LTS
  • Desktop Environment: Unity 7.4.0
  • Tray: libappindicator1 (12.10.1+15.04.20141110-0ubuntu1) / libappindicator3-1 (12.10.1+15.04.20141110-0ubuntu1)

@TheZ3ro
Copy link
Contributor

TheZ3ro commented Nov 25, 2016

This seems related to #73

Please @anatoli26 check that Issue

@anatoli26
Copy link
Author

Indeed, related. Please see an update in my comment for more details.

@pseudolog
Copy link

pseudolog commented Jan 13, 2017

Just wanted to confirm this behaviour.

This bugreport indicates a problem with QSystemTrayIcon::isSystemTrayAvailable();

Commenting QSystemTrayIcon::isSystemTrayAvailable(); in gui/MainWindow.cpp does indeed fix this issue for me. I know next to nothing about QT programming, so I'm not sure which other implications this might bring.

#73

@TheZ3ro
Copy link
Contributor

TheZ3ro commented Jan 13, 2017

So it's confirmed it's an Appmenu-qt5 bug.
We can't really comment QSystemTrayIcon::isSystemTrayAvailable(), on other platforms this works as expected.
You need to wait for a fix by them.

@phoerious
Copy link
Member

Can this be closed?

@anatoli26
Copy link
Author

I guess yes, with the resolution == "won't fix", as the problem is not solved, but it appears that the root cause is in Appmenu-qt5. I'm not sure if the Appmenu-qt5 team has any plans for fixing it however, so if there's a simple workaround for KPXR maybe it should be implemented.

@anatoli26
Copy link
Author

anatoli26 commented Jan 25, 2017

Just talked with one of the appmenu-qt5 maintainers (https://bugs.launchpad.net/appmenu-qt5/+bug/1574699) and this issue is solved in the repository and in Ubuntu 16.10, he'll evaluate now if to release a new version for inclusion in 16.04. With the version from his PPA the KPRC tray icon now works correctly.

@anatoli26
Copy link
Author

The fix for appmenu-qt5 is in xenial-proposed now, if anyone is using Ubuntu 16.04, you can test it following the instructions in the comment 7 to the original bug: https://bugs.launchpad.net/appmenu-qt5/+bug/1574699/comments/7.

@anatoli26
Copy link
Author

The fix is in all versions of Ubuntu now (16.04, 16.10), so this particular issue is completely fixed for everyone.

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

5 participants