marklieberman / foxygestures

Mouse gestures for Firefox
GNU General Public License v3.0
190 stars 20 forks source link

New gesture is appended to previous unknown gesture for current page #172

Open rgorel opened 7 years ago

rgorel commented 7 years ago

Steps to reproduce:

GIVEN

WHEN

THEN

OSX El Capitan, FF 57.0, FoxyGestures 1.0.14, "Double right-click to open context menu" is enabled

marklieberman commented 7 years ago

I reproduce this somewhat using the following steps. Tell me if this sounds familiar.

1) Make sure the Firefox window is not the active/focused window. 2) Right-click into the unactive Firefox window and draw a gesture. (Firefox will stay unactive while you draw the gesture.) 3) All subsequent gestures will be broken until you left-click and activate Firefox.

From further investigation it seems to be Firefox itself reporting large values for MouseEvent deltas. When the Firefox window is inactive, it doesn't see the mouse until right-click. If you right-click and do an unknown gesture then move the mouse and right-click again, the first mousemove event will have deltaX,deltaY equal to the distance from the end of the first gesture to the start of the second gesture.

rgorel commented 7 years ago

@marklieberman yes, it reproduces this way as well