Note: There are a few cases of duplicates in user autocompletion which are being worked on.

Add new edgeui simple gesture for win8, and update tap gesture

RESOLVED FIXED in mozilla16

Status

()

Core
DOM: Events
RESOLVED FIXED
5 years ago
3 years ago

People

(Reporter: jimm, Assigned: jimm)

Tracking

Trunk
mozilla16
x86_64
Windows 8.1
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

Comment hidden (empty)
(Assignee)

Comment 1

5 years ago
Created attachment 632659 [details] [diff] [review]
patch v.1
(Assignee)

Comment 2

5 years ago
Comment on attachment 632659 [details] [diff] [review]
patch v.1

This add a new gesture for win8 metro which fires when the user swipes inward across the display edge. It's used to invoke edge based ui in metro. I've also added a clickCount (similar to mouse click events) to the tap event to support new single and double tap events on Win8.
Attachment #632659 - Flags: review?(felipc)
Comment on attachment 632659 [details] [diff] [review]
patch v.1

Review of attachment 632659 [details] [diff] [review]:
-----------------------------------------------------------------

I don't see anything here that gets this gesture from the OS and dispatches it.. Don't you need to add a new case in the switch statement at nsWinGesture::ProcessGestureMessage for it?

::: dom/interfaces/events/nsIDOMSimpleGestureEvent.idl
@@ +58,5 @@
>   * (XXX Not implemented on Mac)
>   *
> + * MozEdgeUIGesture - Generated when the user swipes the display to
> + * invoke edge ui.
> + * (XXX Win8 tablets only)

s/tablets/Metro ?

@@ +123,1 @@
>    void initSimpleGestureEvent(in DOMString typeArg,

hmm most others init*Event contains all possible fields for the event. although these functions are mostly just used for testing. I think it would be nice to stay consistent

::: widget/nsGUIEvent.h
@@ +430,5 @@
>  #define NS_SIMPLE_GESTURE_ROTATE_UPDATE  (NS_SIMPLE_GESTURE_EVENT_START+5)
>  #define NS_SIMPLE_GESTURE_ROTATE         (NS_SIMPLE_GESTURE_EVENT_START+6)
>  #define NS_SIMPLE_GESTURE_TAP            (NS_SIMPLE_GESTURE_EVENT_START+7)
>  #define NS_SIMPLE_GESTURE_PRESSTAP       (NS_SIMPLE_GESTURE_EVENT_START+8)
> +// win8 metro speicifc

typo. but this comment doesn't fit too well in this file where the other comments describe different sections of the file. You can just drop it, or maybe call the definition NS_SIMPLE_GESTURE_METRO_EDGEUI if you want.
(Assignee)

Comment 4

5 years ago
Comment on attachment 632659 [details] [diff] [review]
patch v.1

(In reply to Felipe Gomes (:felipe) from comment #3)
> Comment on attachment 632659 [details] [diff] [review]
> patch v.1
> 
> Review of attachment 632659 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> I don't see anything here that gets this gesture from the OS and dispatches
> it.. Don't you need to add a new case in the switch statement at
> nsWinGesture::ProcessGestureMessage for it?

This code is on elm and is only fired by the metro interface - 

http://mxr.mozilla.org/projects-central/source/elm/widget/windows/winrt/GestureInput.cpp

This bug is part of the process of moving all the elm code over to mc. The widget code hasn't landed yet, it will probably land last. 

> @@ +123,1 @@
> >    void initSimpleGestureEvent(in DOMString typeArg,
> 
> hmm most others init*Event contains all possible fields for the event.
> although these functions are mostly just used for testing. I think it would
> be nice to stay consistent

Wondered about that, will update. There are no optional params on these so I was trying to avoid updating all the callers. Not a big deal though.

Will fix the typos as well and repost.
Attachment #632659 - Flags: review?(felipc)
(Assignee)

Comment 5

5 years ago
Created attachment 633169 [details] [diff] [review]
patch v.2
Attachment #632659 - Attachment is obsolete: true
(Assignee)

Comment 6

5 years ago
Comment on attachment 633169 [details] [diff] [review]
patch v.2

There weren't as many calls to that init function as I thought so it wasn't a big deal.
Attachment #633169 - Flags: review?(felipc)
Attachment #633169 - Flags: review?(felipc) → review+
(Assignee)

Comment 7

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/542d031d6c3f

Thanks felipe!

Comment 8

5 years ago
https://hg.mozilla.org/mozilla-central/rev/542d031d6c3f
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla16
OS: Windows 8 Metro → Windows 8.1
You need to log in before you can comment on or make changes to this bug.