Xpra: Ticket #2729: right clicks on tray menus for applications running over Xpra don't work

I use Xpra to run applications that use tray icons. Right-clicking those tray icons is supposed to pop up a context menu, but when I click their tray icons, nothing happens.

However, left-clicking those icons works as expected.

To get a more specific test case:

Host system: MATE on Debian Buster amd64
Create a Docker container with Debian Buster
Install openssh-server and Xpra there, connect to it from your host
Install and launch something with tray icons (say, BiglyBT and uTorrent in Wine, to get a good variety)
Left-clicking their tray icons does what it's supposed to do (for me, show/hide the window)
Right-clicking their tray icons is supposed to bring up a context menu, but instead, it has no effect.

I have tried this on Xpra 3.0.4 , and the problem is still there.

(Please fix the ticket tags if they are wrong - I have no idea what to put there.)

Wed, 15 Apr 2020 14:25:34 GMT - Antoine Martin: status, milestone changed

status changed from new to assigned
milestone changed from 4.0 to 4.1

I'm not sure if that's fixable. Tray icon support on Linux is an awful mess, it's possible that the tray icon implementation we use with your DE does not provide right click events. Maybe another does, or maybe not.

If you can, please post the client's -d tray log output, and right click once on a tray icon.

Wed, 15 Apr 2020 14:30:23 GMT - Antoine Martin: summary changed

summary changed from Tray menus for applications running over Xpra don't work to right clicks on tray menus for applications running over Xpra don't work

Wed, 15 Apr 2020 16:25:21 GMT - darkpenguin:

I'm trying again, and now I can't even get it to run. Apparently, this time I'm doing something wrong... I'll try this again if I don't give up.

I know that tray icons are a problem, so I didn't have much hope for this - but I wanted to help reporting and diagnosing it, in case it turns out that I was wrong and this is actually simple.

Sat, 09 May 2020 05:21:34 GMT - Antoine Martin: owner, status changed

owner changed from Antoine Martin to darkpenguin
status changed from assigned to new

Can you provide a screenshot, or logs? Do I need docker to reproduce this bug at all? And ssh? Can't you reproduce the problem with just a local session?

Sat, 09 May 2020 10:34:19 GMT - darkpenguin:

No, you don't need Docker - that is simply the easiest way for me to launch a "completely clean system" without even starting a virtual machine.

Yes, I can reproduce it with a local session.

Boot https://cdimage.debian.org/debian-cd/10.3.0-live/amd64/iso-hybrid/debian-live-10.3.0-amd64-mate.iso in a virtual machine or on a test machine (just to make sure we're exactly on the same system)
sudo apt-get update; sudo apt-get install xpra qbittorrent (I know xpra in the repos is a little outdated, but I've confirmed this on 3.0.4, and this is easier to show)
Try launching qbittorrent normally: left-clicking its tray icon hides the main window, right-clicking the tray icon pops up a menu. Choose "Exit" from that menu.

xpra start :10 --start=/usr/bin/qbittorrent
xpra attach :10
Right-click qbittorrent's tray icon: nothing happens

Sat, 09 May 2020 10:37:33 GMT - Antoine Martin: owner, priority, status changed

owner changed from darkpenguin to Antoine Martin
priority changed from major to critical
status changed from new to assigned

Sat, 09 May 2020 10:47:28 GMT - darkpenguin:

I can post the log, but it's 33 Kb long, and I don't see a way to do a spoiler here...

Sat, 09 May 2020 11:01:02 GMT - Antoine Martin:

I can post the log, but it's 33 Kb long, and I don't see a way to do a spoiler here...

No worries, if it's that easy to reproduce, I'll take it from here.

Sat, 25 Jul 2020 14:42:46 GMT - Antoine Martin:

Problem confirmed. What's weird is that we already have an undocumented hack for swapping the left and right buttons when using the StatusIcon implementation: just hold shift whilst pressing the button. But this hack doesn't help either!?

And both buttons work with a win32 client, the menu shows up. So at least I can try to manually inspect the difference in the packets.

Tue, 15 Sep 2020 11:48:23 GMT - Antoine Martin: status changed; resolution set

status changed from assigned to closed
resolution set to fixed

Well, this bug wins the prize for the dumbest bit of code I've written: a right click should use button number 3, not 2... That's all there was to it. Sorry it's taken so long.

Fixed in r27457.

Sat, 23 Jan 2021 05:59:11 GMT - migration script:

this ticket has been moved to: https://github.com/Xpra-org/xpra/issues/2729