Closed Bug 1379988 Opened 7 years ago Closed 7 months ago

Terrible performance on rooftopfilmclub.com/london

Categories

(Core :: Graphics: Layers, enhancement, P3)

enhancement

Tracking

()

RESOLVED WORKSFORME
Tracking Status
firefox57 --- fix-optional

People

(Reporter: jnicol, Assigned: jnicol)

References

()

Details

(Whiteboard: [gfx-noted])

There seems to be lots of changes to the layer tree, causing allocations and repainting. Probably affects desktop too, but it is incredibly noticeable on android.

I will investigate further.
Problem number one: there is a scrollframe which scrolls a tiny amount nested within the main scrollframe. The inner scrollframe captures lots of swipes, which makes scrolling feel janky. This affects android, and desktop if you make the window narrow.

It is because the body element has `overscroll-x: hidden` set on it, which according to https://stackoverflow.com/a/6433475 implies `overscroll-y: auto`. So we end up with scrollbars on the body as well as the viewport. I doubt this was the desired result, but maybe it is the correct one. Tim, do you know about this? Do you think there is anything we can do here?
Flags: needinfo?(tnikkel)
Chrome doesn't get a second scrollable like we do. Do you know why? Do all the problems go away if we don't get the second scrollable?
Flags: needinfo?(tnikkel)
Chrome does get the second scrollable for me.

No, there are other problems too: it layerizes very poorly when items move about.

If the body was tall enough to contain its content it wouldn't get a scrollbar, even with "overscroll-y: auto", right? It seems like it is only just shorter than its content, therefore we get a scrollbar which only moves a tiny amount. I can't see why it isn't the correct height.
Priority: -- → P3
Whiteboard: [gfx-noted]
Severity: normal → S3

Are you still able to reproduce?

No, it's really smooth nowadays. Probably due to switching from layers to webrender

Status: NEW → RESOLVED
Closed: 7 months ago
Flags: needinfo?(jnicol)
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.