Closed
Bug 1088552
Opened 10 years ago
Closed 10 years ago
[Text Selection] Hard to selectword and cannot drag selection caret inside a iframe.
Categories
(Core :: DOM: Selection, defect)
Tracking
()
RESOLVED
FIXED
mozilla36
People
(Reporter: mtseng, Assigned: mtseng)
References
Details
Attachments
(1 file, 3 obsolete files)
10.55 KB,
patch
|
Details | Diff | Splinter Review |
For example, if we open uitest app and switch to copy/paste test. Then scroll content to the bottom. Select word is hard to trigger or there is an offset between your finger and actually select word's position. Also hard to drag those carets in this state.
Assignee | ||
Comment 1•10 years ago
|
||
I found that we change GetFrameForPoint from canvas frame to root frame in Bug 1068461. But most of our points are in canvas frame's coordinate. So we are using wrong coordinate system now. So I change our most points to root frame's coordinate.
Attachment #8510906 -
Flags: review?(roc)
Assignee | ||
Comment 2•10 years ago
|
||
Update corresponding comment in header.
Attachment #8510906 -
Attachment is obsolete: true
Attachment #8510906 -
Flags: review?(roc)
Attachment #8510908 -
Flags: review?(roc)
Comment 3•10 years ago
|
||
I've verified that SelectWord works after apply this patch. However the patch does apply cleanly on master.
Attachment #8510908 -
Flags: review?(roc) → review+
Assignee | ||
Comment 4•10 years ago
|
||
Rebased. try run: https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=3596a027afa0
Attachment #8510908 -
Attachment is obsolete: true
Assignee | ||
Updated•10 years ago
|
Keywords: checkin-needed
Comment 6•10 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/a9bbad391201
Keywords: checkin-needed
Comment 7•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/a9bbad391201
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla36
You need to log in
before you can comment on or make changes to this bug.
Description
•