Closed Bug 1144650 Opened 5 years ago Closed 5 years ago

With APZ enabled, touch-based scrolling also selects text

Categories

(Core :: Widget: Win32, defect)

All
Windows 8
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla39
Tracking Status
firefox39 --- fixed

People

(Reporter: kats, Assigned: kats)

References

Details

Attachments

(1 file)

After applying patches from bug 1144324 and it's dependencies, if I set the prefs layers.async-pan-zoom.enabled and dom.w3c_touch_events.enabled to true, I can use APZ to touch-scroll. However, doing this still dispatches mouse events to content and therefore a touch-down also generates a mouse-down and a touch-move generates a mouse-move. This results in text selection happening while I'm panning with my finger. We shouldn't be dispatching mouse events for touch input; the only time we need to do this is to dispatch a mousemove,down,up in response to a tap for backwards compatibility. This is already taken care of in the APZ code, so the widget doesn't need to do it.
Attachment #8579340 - Flags: review?(jmathies) → review+
https://hg.mozilla.org/mozilla-central/rev/83c5d1c58506
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla39
Blocks: 1153135
You need to log in before you can comment on or make changes to this bug.