Closed Bug 936005 Opened 11 years ago Closed 11 years ago

Defect - Double tab!! What does it mean??!? (touchstart.preventDefault does not prevent clicks in Metro chrome)

Categories

(Core Graveyard :: Widget: WinRT, defect, P2)

x86_64
Windows 8.1
defect

Tracking

(Not tracked)

VERIFIED FIXED
mozilla28

People

(Reporter: mbrubeck, Assigned: mbrubeck)

References

Details

(Whiteboard: [block28] feature=defect c=tbd u=tbd p=1)

Attachments

(1 file)

When clicking the "new tab" overlay button on the right edge of the screen in Metro Firefox, I see two new tabs appear instead of one.
This is reproducible only with touch, not with mouse. NavButtonSlider.js performs its own click/drag detection. It calls preventDefault on touchstart to prevent widget clicks, but it appears to receive a click anyway.
The problem with preventDefault is bug 861876.
Depends on: 861876
Actually, bug 861876 was for win32 (desktop). This is a chrome-only bug in WinRT widget. Taking this.
Assignee: nobody → mbrubeck
Status: NEW → ASSIGNED
Component: Browser → Widget: WinRT
No longer depends on: 861876
Product: Firefox for Metro → Core
QA Contact: mbrubeck
Summary: Double tab!! What does it mean??!? → Defect - Double tab!! What does it mean??!? (touchstart.preventDefault does not prevent clicks in Metro chrome)
Whiteboard: p=1
Attached patch it's so beautiful (deleted) — Splinter Review
Chrome should use the same logic as content here.
Attachment #828681 - Flags: review?(jmathies)
Blocks: metrov1it18
Priority: -- → P2
QA Contact: mbrubeck → jbecerra
Whiteboard: p=1 → [block28] feature=defect c=tbd u=tbd p=1
Attachment #828681 - Flags: review?(jmathies) → review+
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla28
Blocks: 943451
Keywords: verifyme
Moving the keyword to bug 943451
Keywords: verifyme
Depends on: 953012
Keywords: verifyme
Verified as fixed with Firefox 28 beta 7 on a Dell XPS 12 ultrabook with Windows 8 64-bit.
Status: RESOLVED → VERIFIED
Keywords: verifyme
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: