Closed Bug 765072 Opened 12 years ago Closed 12 years ago

All text is selected when one text selection handle moves over another

Categories

(Firefox for Android Graveyard :: Text Selection, defect)

ARM
Android
defect
Not set
normal

Tracking

(firefox15 verified, firefox16 verified, firefox17 verified)

VERIFIED FIXED
Firefox 16
Tracking Status
firefox15 --- verified
firefox16 --- verified
firefox17 --- verified

People

(Reporter: Margaret, Assigned: Margaret)

References

Details

Attachments

(1 file)

It looks like we're running into weird issues with our fake mouse events. Maybe one handle is accidentally receiving the mouse event when the other handle moves over it, causing unexpected behavior.

Thinking about this now actually makes me wonder if that's the problem I encountered when I saw similar problems while developing my patch for bug 695173.
Attached patch patchSplinter Review
This patch prevents us from sending mouse events to the handles. This means that you can't move one handle to a point overlapping the other handle (stock browser does allow this), but this feels like a good compromise for now to prevent all the text from getting selected.

For the optimal solution, we'd have to figure out how to make the mouse event pass through the handle to the text under it, and I was having difficulty with that when working on bug 766789.
Assignee: nobody → margaret.leibovic
Attachment #635806 - Flags: review?(mark.finkle)
Attachment #635806 - Flags: review?(mark.finkle) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/d65f19e1815e
Target Milestone: --- → Firefox 16
https://hg.mozilla.org/mozilla-central/rev/d65f19e1815e
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Uplifted to aurora as part of a roll-up patch:
https://hg.mozilla.org/releases/mozilla-aurora/rev/2fb0a358eaf6
Component: General → Text Selection
Status: RESOLVED → VERIFIED
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: