Closed Bug 1539582 Opened 6 years ago Closed 2 years ago

Janky keyboard scrolling on articles at The Athletic

Categories

(Core :: CSS Parsing and Computation, enhancement, P3)

enhancement

Tracking

()

RESOLVED WORKSFORME

People

(Reporter: RyanVM, Unassigned)

References

()

Details

(Keywords: perf)

I've noticed that articles on The Athletic scroll smoothly with the mouse wheel, but are very janky when using the keyboard.

Ryan sent me his credentials and I could repro the issue.

The problem is that the page has top-level key event listeners so we can't do APZ keyboard scrolling. And then on top of that every scroll event that gets fired triggers some really long DOM operations so that bogs down the main thread. Here's a profile of that: http://bit.ly/2TDpOll

I don't think we can do much about the APZ keyboard scrolling here on our end, but maybe we can speed up some of the DOM/JS stuff that's running.

I'm going to move this over to CSS since it seems like the lion's share of the time spent by the content main thread is reading computed styles (boxSizing in particular).

Component: Panning and Zooming → CSS Parsing and Computation

This is probably the kind of issue that bug 1381071 and co would help with.

Depends on: 1381071
Priority: -- → P3

I went back to see how things are now that bug 1381071 has landed, but I think they've changed something on their site nowadays as scrolling with the keyboard feels pretty smooth even with ESR91.

Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.