Closed Bug 1065138 Opened 10 years ago Closed 10 years ago

Use Platform selection carets

Categories

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

x86_64
Linux
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED DUPLICATE of bug 988143

People

(Reporter: wesj, Unassigned)

References

(Depends on 1 open bug)

Details

B2G recently landed platform support for some selection carets. Our text selection code is... not the most elegant thing. If/When the platform code is good enough (i.e. both performance and interaction), we should use it. I'll mark some bugs blocking here that I saw when I turned it on so we can track if its ready. Its fairly easy to do 1.) Set touchCaret.enabled and selectioncaret.enabled to true. 2.) Add the images to our package-manifest: http://mxr.mozilla.org/mozilla-central/source/b2g/installer/package-manifest.in#692 have fun.
Blocks: 1065142
No longer blocks: 1065142
Depends on: 1065142
Depends on: 1065146
Depends on: 1065148
I will be so happy when we can rip out all our Fennec text selection code! This was already filed as bug 988143. wesj, do you want to dupe and move your dependencies over?
Component: General → Text Selection
(In reply to Wesley Johnston (:wesj) from comment #0) > B2G recently landed platform support for some selection carets. Our text > selection code is... not the most elegant thing. If/When the platform code > is good enough (i.e. both performance and interaction), we should use it. FWIW, I am *very* interested in us fixing all of the performance and interaction issues you guys face so that we can turn this on for Fennec. That work is also going to benefit b2g, and I like to kill multiple birds with a single stone. :-)
I'm going to mark this as a dup of bug 988143 (vs the other way around). There's a bigger cc: list on that one.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → DUPLICATE
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.