After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 747469 - CSD: can't open window menu using right click after moving window
CSD: can't open window menu using right click after moving window
Status: RESOLVED FIXED
Product: gtk+
Classification: Platform
Component: Class: GtkGesture
3.16.x
Other Linux
: Normal normal
: ---
Assigned To: gtk-bugs
gtk-bugs
Depends on:
Blocks:
 
 
Reported: 2015-04-07 18:21 UTC by Sebastian Keller
Modified: 2015-04-13 15:28 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
gesturesingle: Ignore motion/updates from unhandled sequences (1.36 KB, patch)
2015-04-13 14:22 UTC, Carlos Garnacho
none Details | Review

Description Sebastian Keller 2015-04-07 18:21:33 UTC
To reproduce the bug simply move a CSD window around by dragging its header bar. After that try to open the window menu using a right clicking in the header bar. It doesn't work until you left click the header bar without dragging it around.
Comment 1 LRN 2015-04-13 14:00:05 UTC
Also affects W32, FWIW.
Comment 2 Carlos Garnacho 2015-04-13 14:22:26 UTC
Created attachment 301463 [details] [review]
gesturesingle: Ignore motion/updates from unhandled sequences

This used to do the right thing, practically ignoring those event
if they went all through the event handler, unless this motion event
triggered the cancellation of this gesture from the event handler
of another controllers.

In this case, the controller would be reset, but the motion event
would still go through its handler, setting again
current_button/sequence as the motion event has buttons in its
modifiers. This leaves GtkGestureSingle with inconsistent data
that may interfere with future runs.
Comment 3 Carlos Garnacho 2015-04-13 15:28:31 UTC
Attachment 301463 [details] pushed as 8bf5b17 - gesturesingle: Ignore motion/updates from unhandled sequences