795063 - Investigate using additional events from GestureRecognizer in widget

Status: RESOLVED FIXED

(Core :: Widget: Win32, defect)

Description

[Tim Abraldes [:TimAbraldes] [:tabraldes]]

We already use the `PointerPressed`, `PointerReleased`, and `PointerMoved` events to send 'mousedown', 'mouseup', 'mousemoved', 'touchstart', 'touchend', and 'touchmove' events.  There are higher-level events that Windows sends us that we should investigate using in widget.

GestureRecognizer provides the `RightTapped` event to signify that the user has done something that is equivalent to right-clicking in a specific location.  This could mean that the user has right-clicked with a physical mouse, that the user has held her/his finger in a specific location on a touchscreen, or done something with a pen/stylus that indicates a context menu should appear.  We should investigate sending the `NS_CONTEXTMENU` event when Windows raises the `RightTapped` event.

GestureRecognizer provides the `Tapped` event to signify that the user has completed a click or tap.  On a mouse, this means pushing the left mouse button down and releasing it before some Windows-defined time interval expires.  On a touch screen, this means tapping with a single finger.  With a stylus, I imagine this means tapping the screen with the stylus.  We should investigate sending the `NS_MOUSE_CLICK` event when Windows raises the `Tapped` event.  To handle double-clicks, Windows provides us with a `TapCount` that we can use when generating our content events.

In general, taking advantage of the higher-level input events that GestureRecognizer can send us in widget code will allow us to act more like other Metro applications and will reduce the need for complex input event processing in our own code.

Comment 1

[Tim Abraldes [:TimAbraldes] [:tabraldes]]

oops, set this flag on the wrong bug

Comment 2

[Tim Abraldes [:TimAbraldes] [:tabraldes]]

The patch I'm writing for bug 808847 will effectively close this bug, so I'm marking it as metro-it1

Comment 3

[Tim Abraldes [:TimAbraldes] [:tabraldes]]

Please look at bug 794621 for updated patches that include a fix for this bug.

Comment 4

[Tim Abraldes [:TimAbraldes] [:tabraldes]]

The patches in bug 794621 have been submitted to elm.  The patch that contains the changes for this bug is:
https://hg.mozilla.org/projects/elm/rev/e38fa39ccf69

This build contains the changes:
https://tbpl.mozilla.org/?tree=Elm&rev=e38fa39ccf69