Closed
Bug 1432017
Opened 7 years ago
Closed 7 years ago
Remove AccessibleCaretEventHub::PostScrollState.
Categories
(Core :: DOM: Selection, enhancement, P3)
Core
DOM: Selection
Tracking
()
RESOLVED
FIXED
mozilla60
| Tracking | Status | |
|---|---|---|
| firefox60 | --- | fixed |
People
(Reporter: emilio, Assigned: TYLin)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
From TYLin in #gfx:
10:37 <TYLin> emilio: have you try to fix the AccessibleCaret stuff yet? I should know everything about it. The LaunchScrollEndInjector() was written when we need to hide the carets when user is scrolling, but we no longer do that.
10:43 <TYLin> emilio: I feel we could eliminate AccessibleCaretEventHub's PostScrollState because that state exists just to avoid carets flickering when the momentum scroll is still going, and the user lifts the finger off the screen and then scroll again
I want the fix for bug 1431474 to land first since it feels less risky, but let's do that afterwards.
|
Reporter | |
Updated•7 years ago
|
|
Assignee | |
Updated•7 years ago
|
Blocks: AccessibleCaret
Component: Layout → Selection
| Comment hidden (mozreview-request) |
|
|
Assignee | |
Comment 3•7 years ago
|
||
|
||
Comment 4•7 years ago
|
||
| mozreview-review | ||
Comment on attachment 8953544 [details]
Bug 1432017 - Remove AccessibleCaretEventHub::PostScrollState.
https://reviewboard.mozilla.org/r/222680/#review228852
Attachment #8953544 -
Flags: review?(mats) → review+
Pushed by aethanyc@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/d48195c9d3e1
Remove AccessibleCaretEventHub::PostScrollState. r=mats
|
||
Comment 6•7 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox60:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla60
|
|
Assignee | |
Updated•7 years ago
|
Assignee: nobody → aethanyc
|
||
Updated•5 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•