Closed
Bug 849552
Opened 11 years ago
Closed 11 years ago
<input type=range> should be given focus on mousedown/touchstart
Categories
(Core :: DOM: Core & HTML, defect)
Tracking
()
RESOLVED
FIXED
mozilla22
People
(Reporter: jwatt, Assigned: jwatt)
References
Details
Attachments
(1 file)
|
1.80 KB,
patch
|
smaug
:
review+
|
Details | Diff | Splinter Review |
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 | |
Comment 1•11 years ago
|
||
Once bug 849855 and bug 849961 are fixed, this patch takes care of the focus issue.
Attachment #723626 -
Flags: review?(bugs)
|
||
Updated•11 years ago
|
Attachment #723626 -
Flags: review?(bugs) → review+
|
|
Assignee | |
Comment 2•11 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/56e5fc18e1ad
|
||
Comment 3•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/56e5fc18e1ad
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla22
You need to log in
before you can comment on or make changes to this bug.
Description
•