Bug 379312 - gtk systray icons aren't receiving the forwarded mouse events
Summary: gtk systray icons aren't receiving the forwarded mouse events
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: XembedSNIProxy (show other bugs)
Version: 5.8.6
Platform: Debian unstable Linux
: NOR normal
Target Milestone: 1.0
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-04-28 13:19 UTC by Maximiliano Curia
Modified: 2017-06-17 08:28 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In: 5.10.1
Sentry Crash Report:


Attachments
Minimal (1.31 KB, text/x-python)
2017-04-28 13:19 UTC, Maximiliano Curia
Details
Patch for 5.8.7 (9.14 KB, patch)
2017-06-17 08:28 UTC, Maximiliano Curia
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Maximiliano Curia 2017-04-28 13:19:23 UTC
Created attachment 105241 [details]
Minimal

Hi,

A GtkStatusIcon under plasma doesn't receive the "activate" "popup-menu" or "button-press-event" signals, but the xembedsniproxy shows the event in it's debug output:
 kde.xembedsniproxy: Received click 1 with passed x*y 1537 1052
 kde.xembedsniproxy: Received click 3 with passed x*y 1537 1052
 kde.xembedsniproxy: Received click 2 with passed x*y 1537 1052

Also, the tooltip text isn't correctly updated. I'm attaching a minimal program that starts a GtkStatusIcon which works correctly in other desktops that still support the systray (like cinnamon).

I've also tried applying 9df815e843a4385465fff0cb9a76ddc94ed35b38, and other than a missing unused header (xcb/xinput.h) that I had to comment it made no difference.

Happy hacking,
Comment 1 David Edmundson 2017-05-25 22:55:24 UTC
That works here.

Can you check under latest Plasma, and if it still doesn't work we must have something different in our GTKs.
Comment 2 Wolfgang Bauer 2017-06-15 15:33:22 UTC
Should be fixed meanwhile:
https://cgit.kde.org/plasma-workspace.git/commit/?id=7df184afa19f148c1cd09ae9588645bb2b4556fc
Comment 3 Maximiliano Curia 2017-06-17 08:26:48 UTC
(In reply to Wolfgang Bauer from comment #2)
> Should be fixed meanwhile:
> https://cgit.kde.org/plasma-workspace.git/commit/
> ?id=7df184afa19f148c1cd09ae9588645bb2b4556fc

I'm currently taking xembed-sni-proxy/* from the mentioned patch on top of 5.8.7, and the status icons now work as expected.
(I guess that's 9df815e843a4385465fff0cb9a76ddc94ed35b38, f0e45cd42ba7e67ec444a4c14bac855b0ac195df and 7df184afa19f148c1cd09ae9588645bb2b4556fc)

I'm attaching the patch as used in the Debian packages.
Comment 4 Maximiliano Curia 2017-06-17 08:28:04 UTC
Created attachment 106135 [details]
Patch for 5.8.7