libappindicator breaks left-mouse actions for most applications

Status changed to 'Confirmed' because the bug affects multiple users.

@Sebastien, there are probably reasons for low priority. Nevertheless, there are hundreds of applications that suffer from this issue (e.g. google for "electron tray left click" - most are caused by this issue). And since gkt4 will drop direct api support, libappindicator becomes more important. Would you mind to re-think the priority?

Thank you,
Michael.

The issue you describe doesn't happen on xubuntu with the indicator, trying nmapplet as an example the menu opens on top of the panel and not at the cursor location, also relaxing the click doesn't close it nor trigger the action. The placement and rendering of the menu would rather be an issue with the applet or desktop environment and not one of the library. The priority is low for Ubuntu since there is no report of such problems on Ubuntu despite indicator being enabled in the default desktop environments for almost a decade

> doesn't happen on xubuntu with the indicator

But it does happen on Ubuntu.

> The priority is low for Ubuntu since there is no report of such problems on Ubuntu

Here comes that report. To reproduce:
1. Download the latest Ubuntu (21.04, live stick is enough).
2. Download and open Joplin (https://github.com/laurent22/joplin/releases/download/v1.7.11/Joplin-1.7.11.AppImage), which is an app that uses Electron and therefore libappindicator.
3. In the Options under Application enable "Show tray icon", which should show up at the top right of the screen.
4. Left-click on the icon. Observe how the context menu opens, just like with a right click, even though the application should open the main window instead (https://github.com/laurent22/joplin/blob/80b16dd17e227e3f538aa221d7b6cc2d81688e72/packages/app-desktop/ElectronAppWrapper.ts#L297-L299), and indeed it does so under Windows for example.

The fucking tray icon doesn't work. Please for the love of god. I'm just trying to left click to open my app.

App:
Element Element version: 1.7.32.

Operating System: KDE neon 5.22
KDE Plasma Version: 5.22.3
KDE Frameworks Version: 5.83.0
Qt Version: 5.15.3
Kernel Version: 5.12.15-xanmod1 (64-bit)
Graphics Platform: Wayland
Processors: 12 � AMD Ryzen 5 3600 6-Core Processor
Memory: 15,6 GiB of RAM
Graphics Processor: AMD Radeon HD 7800 Series

This is a huge problem affecting many distros, including of course Ubuntu, that rely on this package. You can see tons of people reporting this across the web. So why is this low priority?

It seems like "Activate" is not even implemented, only "SecondaryActivate". https://git.launchpad.net/ubuntu/+source/libappindicator/tree/src/app-indicator.c#n1198

KDE had to work around this by just falling back to opening the context menu:
https://github.com/KDE/plasma-workspace/blob/e0609d88d029ec0700768fa8ec1db3e32b628495/applets/systemtray/package/contents/ui/items/StatusNotifierItem.qml#L54

I experimented with implementing the Activate method, it gets called when left-clicking the status icon. But there is nothing to activate when Activate is called.
AppIndicator has a "menu" reference, but "menu" can not be activated, only MenuItem.
AppIndicator itself can not be activated as well.
(Or nobody listens for that signals)

The half-assed workaround Electron is doing is to create a dummy MenuItem, with label set the same as the status icon tooltip and add a "click-action-item" flag.

https://github.com/electron/electron/blob/8532e1239ec929e31b764ffcd314c8db3e09a19f/shell/browser/ui/gtk/app_indicator_icon_menu.cc#L65

But this item never gets actually added to the Menu, as HasClickAction always returns false.
https://github.com/electron/electron/blob/8532e1239ec929e31b764ffcd314c8db3e09a19f/shell/browser/ui/tray_icon_gtk.cc#L64

Note: "SecondaryActivate" is only called on middle-button click. On right click, the menu is open directly, without any method called on the AppIndicator DBus. On left click, the "Activate" method is called if it exists. (at least in KDE Plasma 5.24.6)

I succesfully tested the changes with custom build of Electron, will do a pull request for Electron soon. Here is the libappindicator patch

The relevant Electron PR is here: https://github.com/electron/electron/pull/35319

The attachment "Add support for Activate method to AppIndicator" seems to be a patch. If it isn't, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are a member of the ~ubuntu-reviewers, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issues please contact him.]

The Electron team is willing to accept the PR when this is fixed in libappindicator first.
https://github.com/electron/electron/pull/35319#pullrequestreview-1072278155