<input type=range> should be given focus on mousedown/touchstart

RESOLVED FIXED in mozilla22

Status

()

RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: jwatt, Assigned: jwatt)

Tracking

Trunk
mozilla22
x86
Mac OS X
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
The patch for 846380 sets nsEventStatus_eConsumeNoDefault for mousedown/touchstart to prevent attempts by the user to move the thumb from causing the page to scroll in Firefox Mobile on Android. Unfortunately this also appears to prevent the input from being given focus when the mousedown/touchstart events occur.

Smaug, what are your thoughts here? I'm not sure what the best way to resolve this is. Perhaps it would just be best to do the focusing manually in the NS_MOUSE_BUTTON_DOWN/NS_TOUCH_START case in the switch in nsHTMLInputElement::PostHandleEventForRangeThumb?
(Assignee)

Updated

6 years ago
Depends on: 849855
(Assignee)

Updated

6 years ago
Depends on: 849961
(Assignee)

Comment 1

6 years ago
Created attachment 723626 [details] [diff] [review]
patch

Once bug 849855 and bug 849961 are fixed, this patch takes care of the focus issue.
Attachment #723626 - Flags: review?(bugs)
https://hg.mozilla.org/mozilla-central/rev/56e5fc18e1ad
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla22
You need to log in before you can comment on or make changes to this bug.