449870 2022-02-09 15:39:10 +0000 Tray icons don't react to right click 2022-02-14 18:50:05 +0000 1 1 4 Plasma plasmashell System Tray widget 5.24.0 Neon Linux RESOLVED FIXED regression VHI normal 1.0 1 voron1 plasma-bugs-null dr_thodt g111 gonterman.matt holo-doc kde login+kde-bugs materka mo78 nate nicolas.fella noahadvs realnc voron1 https://invent.kde.org/plasma/plasma-workspace/commit/b53fec3714c68ae6191f3c2207e7b7a06b53d9f5 5.24.1 0 oldest_to_newest 2102461 0 voron1 2022-02-09 15:39:10 +0000 SUMMARY *** Tray icons like discord and gxneur (legacy), don't react to right mouse click. No menu shows up. Worked in the previous version (5.23.5). *** SOFTWARE/OS VERSIONS 5.16.8-gentoo Linux/KDE Plasma: 5.24 (available in About System) KDE Plasma Version: 5.24 KDE Frameworks Version: 5.90 Qt Version: 5.15.2 2102469 1 voron1 2022-02-09 16:11:31 +0000 Notably Viber's tray icon menu works as it should. 2102540 2 behzad.a_ir 2022-02-09 19:35:44 +0000 *** Bug 449879 has been marked as a duplicate of this bug. *** 2102585 3 voron1 2022-02-09 21:08:19 +0000 Skype's icon doesn't show the menu either. 2102683 4 realnc 2022-02-10 04:32:34 +0000 I have the exact same issue. Downgrading to 5.23.5 fixes it. A workaround is to install libappindicator, which is detected by discord and makes it use a different method to implement its tray icon. However, this is far from a good workaround because then there's no left click support anymore. Both right and left clicking on the tray icon will simply open the same context menu. Normally, what you want is not have libappindicator installed so that a left click on the tray icon will unminimize discord, and a right click will open the context menu, like in virtually any other application that has a tray icon. 2102774 5 holo-doc 2022-02-10 09:34:03 +0000 Same problem here since the upgrade from 5.23.90 to 5.24 - Discord and Gajim right click menus in the taskbar are no longer working. At the moment of the right click the following entry appears in the journal: plasmashell[818]: Could not find DBusMenu interface, falling back to calling ContextMenu() 2102802 6 holo-doc 2022-02-10 10:47:19 +0000 OK you can ignore my previous post about the journal entry regarding "DBusMenu interface", as it also appears when downgrading to the still working plasma-workspace 5.23.90. I browsed through the git history and reverting part of the changes in StatusNotifierItem.qml of commit ae5f355a992786834babc952c616b21929cd3a97 makes the right click menu appear again. I just readded the following at line 61 and then everything works as expected: case Qt.RightButton: openContextMenu(pos); break; 2102828 7 behzad.a_ir 2022-02-10 12:09:16 +0000 *** Bug 449928 has been marked as a duplicate of this bug. *** 2102884 8 noahadvs 2022-02-10 14:28:16 +0000 I can't reproduce this on git master. Maybe a commit depending on KF5.91 was cherry-picked to Plasma 5.24 too soon? 2102885 9 nicolas.fella 2022-02-10 14:30:53 +0000 Which app did you test? It doesn't happen with apps using "proper" SNIs, so all KDE apps, Electron etc I can reproduce it on master with Gajim 2102889 10 noahadvs 2022-02-10 14:33:24 +0000 (In reply to Nicolas Fella from comment #9) > Which app did you test? It doesn't happen with apps using "proper" SNIs, so > all KDE apps, Electron etc > > I can reproduce it on master with Gajim I tested Discord since the reporter said they had a problem with it. 2102892 11 noahadvs 2022-02-10 14:37:15 +0000 Ok, I can reproduce this with Gajim 2102894 12 noahadvs 2022-02-10 14:39:04 +0000 (In reply to Nicolas Fella from comment #9) > Which app did you test? It doesn't happen with apps using "proper" SNIs What makes an SNI "proper" or "improper"? 2102902 13 nicolas.fella 2022-02-10 15:00:05 +0000 What I'm referring to is StatusNotifierItem vs xembed tray icons. Although I'm not 100% sure that's the relevant difference here 2102920 14 voron1 2022-02-10 15:45:28 +0000 (In reply to Claudio from comment #6) > I browsed through the git history and reverting part of the changes in > StatusNotifierItem.qml of commit ae5f355a992786834babc952c616b21929cd3a97 > makes the right click menu appear again. > > I just readded the following at line 61 and then everything works as > expected: > > case Qt.RightButton: > openContextMenu(pos); > break; Could you make it into a patch on top of 5.24 so I could test? I'd very much appreciate that. 2102937 15 voron1 2022-02-10 16:19:08 +0000 Okay, I did the manual partial undoind of ae5f355a99 and it fixes the issue. 2103027 16 mo78 2022-02-10 19:10:30 +0000 My findings so far: Tixati, Teams, Discord, Skype, Jdownloader. All with missing context menu. Is 5.24 still in alpha??? 2103029 17 mo78 2022-02-10 19:13:35 +0000 *** Bug 449963 has been marked as a duplicate of this bug. *** 2103307 18 login+kde-bugs 2022-02-11 12:20:10 +0000 Slack and 1password are also affected (both electron) 2103312 19 dr_thodt 2022-02-11 12:41:45 +0000 Same problem in Deadbeef player and Altus (Whatsapp client) 2103317 20 mo78 2022-02-11 12:55:01 +0000 @Michael How we can apply your patch? This weird bug literally ruins my workflow :( 2103336 21 noahadvs 2022-02-11 13:30:25 +0000 (In reply to Nick Stefanov from comment #20) > @Michael > How we can apply your patch? This weird bug literally ruins my workflow :( A quicker self-fix would be to install libappindicator. It seems we missed this bug because every KDE dev who tested the patch had libappindicator installed. 2103344 22 mo78 2022-02-11 13:43:29 +0000 @Noah Davis Thank you for your suggestion but unfortunately ibappindicator is not an option for me. Some of the reasons are very well described in comment 4: https://bugs.kde.org/show_bug.cgi?id=449870#c4 2103358 23 nate 2022-02-11 14:34:43 +0000 Heh maybe I'll remove libappindicator so I can catch future regressions in this use case. 2103366 24 voron1 2022-02-11 14:40:52 +0000 (In reply to Nick Stefanov from comment #20) > @Michael > How we can apply your patch? This weird bug literally ruins my workflow :( There is no patch. I went and partially undid this commit https://invent.kde.org/plasma/plasma-workspace/-/commit/ae5f355a992786834babc952c616b21929cd3a97 namely the changes to StatusNotifierItem.qml and then rebuilt and reinstalled. How to do all that in your distro, I frankly have no idea. 2103384 25 mo78 2022-02-11 15:47:21 +0000 Ah, thanks, let's hope it will be fixed quick. 2103385 26 voron1 2022-02-11 15:51:54 +0000 (In reply to Noah Davis from comment #21) > A quicker self-fix would be to install libappindicator. It seems we missed > this bug because every KDE dev who tested the patch had libappindicator > installed. I don't know about that. I played with libappindicator, and while it does help discord, old gtk apps (gxneur) still don't have the tray menu. I suspect it'll be the same for gajim. 2103548 27 holo-doc 2022-02-11 19:19:23 +0000 For a workaround just add to the following file on your system (make a backup copy before): /usr/share/plasma/plasmoids/org.kde.plasma.private.systemtray/contents/ui/items/StatusNotifierItem.qml at line 61 the following 3 lines with your texteditor of choice (nano, kate or whatever): case Qt.RightButton: openContextMenu(pos); break; Now run "plasmashell --replace" and the right click menu is working again - no need to compile anything yourself. 2103565 28 voron1 2022-02-11 19:45:45 +0000 @Claudio Thanks, I didn't realize you don't need that file for compilation. 2103663 29 mo78 2022-02-11 22:26:46 +0000 @Michael Yep, it's working! Thank you so much!!! 2103781 30 gonterman.matt 2022-02-12 07:36:51 +0000 @Claudio - You are awesome! Thank you so much! 2104535 31 bug-janitor 2022-02-14 16:37:38 +0000 A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/1455 2104576 32 nate 2022-02-14 18:14:20 +0000 Git commit 03f982261b7f329b42d490651efca1f115cbf097 by Nate Graham, on behalf of Claudio Holo. Committed on 14/02/2022 at 18:13. Pushed by ngraham into branch 'master'. applets/systemtray: fix SNI context menu usage without libappindicator StatusNotifierItem.qml does not explicitly handle the case of right-clicking, which causes right-clicks to be ignored unless you also happen to have the libappindicator package installed, which injects a handler for this itself. But this should work even if you don't have that package installed; this commit fixes it. FIXED-IN: 5.24.1 M +3 -0 applets/systemtray/package/contents/ui/items/StatusNotifierItem.qml https://invent.kde.org/plasma/plasma-workspace/commit/03f982261b7f329b42d490651efca1f115cbf097 2104577 33 nate 2022-02-14 18:14:48 +0000 Git commit b53fec3714c68ae6191f3c2207e7b7a06b53d9f5 by Nate Graham, on behalf of Claudio Holo. Committed on 14/02/2022 at 18:14. Pushed by ngraham into branch 'Plasma/5.24'. applets/systemtray: fix SNI context menu usage without libappindicator StatusNotifierItem.qml does not explicitly handle the case of right-clicking, which causes right-clicks to be ignored unless you also happen to have the libappindicator package installed, which injects a handler for this itself. But this should work even if you don't have that package installed; this commit fixes it. FIXED-IN: 5.24.1 (cherry picked from commit 03f982261b7f329b42d490651efca1f115cbf097) M +3 -0 applets/systemtray/package/contents/ui/items/StatusNotifierItem.qml https://invent.kde.org/plasma/plasma-workspace/commit/b53fec3714c68ae6191f3c2207e7b7a06b53d9f5 2104600 34 mo78 2022-02-14 18:50:05 +0000 Thank you!