Closed
Bug 1168847
(gecko-carets)
Opened 10 years ago
Closed 9 years ago
[meta] Enable gecko text selection carets by default
Categories
(Firefox for Android Graveyard :: Text Selection, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: Margaret, Unassigned)
References
Details
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
Details | Diff | Splinter Review |
Let's use this bug to track the dependencies we need to fix in order to get the gecko text selection carets riding the trains.
Once we think enough of these issues are fixed, we can first enable them by default on Nightly only, in order to get more bug reports.
Comment 1•10 years ago
|
||
^ I'd like to see if it's possible for us to use our own visuals rather than FFOS here
Reporter | ||
Updated•9 years ago
|
Depends on: GeckoCaret2
Reporter | ||
Comment 2•9 years ago
|
||
We should triage this bug to figure out what's required to ship this.
Previously we said that we wanted to wait for APZ to land before we would move forward with this, but that change is imminent. What are our next steps?
Flags: needinfo?(snorp)
Flags: needinfo?(markcapella)
Comment 3•9 years ago
|
||
I've been observing feedback / worst case bugs filed by early adopters of original bug 988143, and roughing a plan for TextSelection component in general [0]. Briefly, we can go-live in nightly anytime with what we have.
In general:
I'm poking at bug 1215959 - Switch from GeckoCaret -> AccessibleCaret and expect to finish (code, test, review, etc) by published 12/4 date. This may impact TouchCaret/SelectionCaret visibility from the current experimental version, (trying to solve for Bug 1166807 while we're here).
We may delay flipping the pref in m-c until this piece gets completed. Handles/Caret visibility may be affected.
Unassigned and possible show-stopper:
Bug 1168891 - New Gecko selection carets cannot reverse their positions
Unassigned enhancements for parallel track:
Bug 1171110 - Floating text selection action bar for Android M
Bug 1097398 - Match Android L text selection handles
Valid bug needs attention:
Bug 1199967 - Text selection in subdocuments may be broken, valid bug
Related, but probably irrelevant:
Bug 1213644 - Failure of TextSelection using the new GeckoCarets ...
(verify mitigated by bug 1211443).
Bug 1166807 - Text carets overlay screen and don't stay with selection when scrolling
(verify mitigated by bug 1215959).
Broken down, I don't think this presents much work. I've currently assuming bug 1215959 for myself, but that piece and the 12/4 completion are the only planned hard-deliverables.
If you see a different prioritization, or want to pass-off any/all bits to move things along, I'm happy to share whatever I have that's useful.
[0] https://www.dropbox.com/s/2qq7fqw1be73qfo/TextSelection.ods?dl=0
Flags: needinfo?(markcapella)
Updated•9 years ago
|
Assignee: nobody → markcapella
Comment 4•9 years ago
|
||
Final patch before allowing on escape m-c to m-a and the world should be pretty much this.
We're working on doing this after APZ lands with bug 1215959 (using the newer caret impl)
Flags: needinfo?(snorp)
Updated•9 years ago
|
Assignee: markcapella → nobody
Comment 7•9 years ago
|
||
Updating more current path from local queue
Attachment #8687625 -
Attachment is obsolete: true
Comment 8•9 years ago
|
||
@snorp: Does this go hand in hand with APZ? And are we still targeting 48? The floating text selection (bug 1171110) depends on this and I'm keeping it behind a Nightly flag for now.
Flags: needinfo?(snorp)
Reporter | ||
Comment 9•9 years ago
|
||
I'm closing this meta bug out in favor of bug 1230609, which is tracking the other half of this outstanding work.
Sebsatian, I'll move your NI over there.
Status: NEW → RESOLVED
Closed: 9 years ago
Flags: needinfo?(snorp)
Resolution: --- → FIXED
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•