Closed Bug 849552 Opened 8 years ago Closed 8 years ago

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

Categories

(Core :: DOM: Core & HTML, defect)

x86
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla22

People

(Reporter: jwatt, Assigned: jwatt)

References

Details

Attachments

(1 file)

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?
Depends on: 849855
Depends on: 849961
Attached patch patchSplinter Review
Once bug 849855 and bug 849961 are fixed, this patch takes care of the focus issue.
Attachment #723626 - Flags: review?(bugs)
Attachment #723626 - Flags: review?(bugs) → review+
https://hg.mozilla.org/mozilla-central/rev/56e5fc18e1ad
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla22
You need to log in before you can comment on or make changes to this bug.