Firefox for Metro should support mouse gestures/buttons for back/forward

RESOLVED FIXED in mozilla30

Status

defect
P2
normal
RESOLVED FIXED
6 years ago
23 days ago

People

(Reporter: KWierso, Assigned: jimm)

Tracking

({platform-parity})

Trunk
mozilla30
x86_64
Windows 8.1

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: p=1 s=it-30c-29a-28b.1 r=ff30, URL)

Attachments

(2 attachments)

(Reporter)

Description

6 years ago
I have the Microsoft Touch mouse, which includes a touch gesture for going back/forward in a browser's history (or in other programs where it makes sense) by swiping my thumb either down or up against the side of the mouse. This works in desktop Firefox and IE, as well as Metro IE. 

It doesn't seem to do anything in Metro Firefox. I think it should, at least for parity with IE.

The MozillaZine thread linked in this bug's URL field says that the hardware back/forward buttons on Logitech mice also don't work in Metro Firefox, while it does work everywhere else. I'd guess it's the same issue for both the gestures and the buttons?
Blocks: 785473
Keywords: pp, ux-efficiency
Whiteboard: [metro-mvp?][LOE:1]
(Assignee)

Comment 1

6 years ago
not blocking but wanted as soon as we have the time.
Whiteboard: [metro-mvp?][LOE:1] → [metro-mvp][LOE:1]
(Assignee)

Updated

6 years ago
Whiteboard: [metro-mvp][LOE:1] → [metro-mvp-][LOE:1]
(Assignee)

Updated

6 years ago
OS: Windows 8 → Windows 8 Metro

Updated

6 years ago
No longer blocks: 785473
Keywords: pp, ux-efficiency
Whiteboard: [metro-mvp-][LOE:1]
No longer blocks: metrov2planning
Whiteboard: [feature]
Whiteboard: [feature] → [feature] p=0
Depends on: 967819
Removing 'ux' keyword as UX dependency bug has been created which blocks bug 819613.
Keywords: ux-efficiency
(Assignee)

Comment 3

5 years ago
We need to add support for application command messages - 

http://mxr.mozilla.org/mozilla-central/source/widget/windows/nsWindow.cpp#5091
http://mxr.mozilla.org/mozilla-central/source/widget/windows/nsWindow.cpp#3698
Component: Input → Widget: WinRT
Product: Firefox for Metro → Core
Version: unspecified → Trunk
(Assignee)

Updated

5 years ago
OS: Windows 8 Metro → Windows 8.1
(Assignee)

Comment 4

5 years ago
I have to find a keyboard i can test this with.
Assignee: nobody → jmathies
(Assignee)

Comment 5

5 years ago
This is working in widget, but we also need front end support - 

http://mxr.mozilla.org/mozilla-central/source/browser/base/content/browser.js#1471
Hey Jim, can you provide a point value.
Status: NEW → ASSIGNED
Flags: needinfo?(jmathies)
Priority: -- → P2
QA Contact: jbecerra
Whiteboard: [feature] p=0 → [feature] p=0 s=it-30c-29a-28b.1
Target Milestone: --- → mozilla30
No longer depends on: 967819
(Assignee)

Comment 7

5 years ago
Attachment #8370395 - Flags: review?(sfoster)
Flags: needinfo?(jmathies)
(Assignee)

Updated

5 years ago
Attachment #8370375 - Attachment description: patch v.1 → widget patch v.1
Attachment #8370375 - Flags: review?(tabraldes)
(Assignee)

Updated

5 years ago
Whiteboard: [feature] p=0 s=it-30c-29a-28b.1 → [feature] p=0 s=it-30c-29a-28b.1 p=1
Whiteboard: [feature] p=0 s=it-30c-29a-28b.1 p=1 → [feature] s=it-30c-29a-28b.1 p=1
Comment on attachment 8370395 [details] [diff] [review]
browser patch v.1

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

Not sure how to test? Code-wise looks fine though

::: browser/metro/base/content/browser-ui.js
@@ +1145,5 @@
>      }
>    },
>  
> +  handleAppCommandEvent: function (aEvent) {
> +    switch (aEvent.command) {

Nit: I would normally indent the switch block?
Attachment #8370395 - Flags: review?(sfoster) → review+
(Assignee)

Comment 9

5 years ago
(In reply to Sam Foster [:sfoster] from comment #8)
> Comment on attachment 8370395 [details] [diff] [review]
> browser patch v.1
> 
> Review of attachment 8370395 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> Not sure how to test? Code-wise looks fine though
> 
> ::: browser/metro/base/content/browser-ui.js
> @@ +1145,5 @@
> >      }
> >    },
> >  
> > +  handleAppCommandEvent: function (aEvent) {
> > +    switch (aEvent.command) {
> 
> Nit: I would normally indent the switch block?

I was not able to test much either aside from Back/Forward. The widget code comes from win32 widget though which we now share, and the front end code is pulled from desktop's browser.js, so we should be ok.
Comment on attachment 8370375 [details] [diff] [review]
widget patch v.1

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

Seems pretty straightforward and it sounds like you tested with the 'back' and 'forward' keys
Attachment #8370375 - Flags: review?(tabraldes) → review+
(Assignee)

Comment 13

5 years ago
Odd, never run into this before, B2G Desktop Windows Opt failed to link due to missing nsGkAtoms in nsWindowBase. Windows builds were fine.
(Assignee)

Comment 14

5 years ago
(In reply to Jim Mathies [:jimm] from comment #13)
> Odd, never run into this before, B2G Desktop Windows Opt failed to link due
> to missing nsGkAtoms in nsWindowBase. Windows builds were fine.

B2G Desktop Windows Op isn't a unified build.
https://hg.mozilla.org/mozilla-central/rev/d0e1eff20097
https://hg.mozilla.org/mozilla-central/rev/5fd318a789f6
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Whiteboard: [feature] s=it-30c-29a-28b.1 p=1 → p=1 s=it-30c-29a-28b.1 r=ff30
For testing and verification.  Reopen if any defects found.
Flags: needinfo?(kamiljoz)
(Reporter)

Comment 18

5 years ago
Setting needinfo on myself to try to verify this once I track down the dongle for my mouse.
Flags: needinfo?(kwierso)
We've also ordered the mouse to the TO office so will have it for any future test cases. If it comes in before Wes can find his dongle, I'll complete verification.
(Reporter)

Comment 20

5 years ago
Yeah, I've looked in all of the places where the dongle could possibly be and I don't see it. Guess this is up to the TO office to verify, now. :(
Flags: needinfo?(kwierso)
no worries Wes, I ordered a mouse and it shipped today.. Hopefully it arrives tomorrow or by Monday - Tuesday. Thanks for checking!
Clearing the "needinfo" as the metrofx project has been cancelled. Unfortunately the mouse came in after the project was cancelled :(
Flags: needinfo?(kamiljoz)

Updated

23 days ago
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.