Prevent scrolling to top before page has rendered

[pascalbaljet]

Fixes #2125. It's hard to reproduce because it's only noticeable on very heavy pages, and even then it doesn’t always consistently fail.

I've tried with requestAnimationFrame() and setTimeout(cb, 0), but that's not enough.

Still draft because it misses tests, and I'm not sure if the new behavior is desired.

[wescopeland]

Hi @pascalbaljet 👋

Thanks for your work on the issue (and your work on Inertia in general).

We're affected heavily by this, and it seems there's a question of whether this PR's changes are desirable. Short of forking/patching Inertia, is it possible for developers to implement some flavor of this fix in their own apps? Or is it a core problem that requires changes in Inertia itself?

[pascalbaljet]

We're affected heavily by this, and it seems there's a question of whether this PR's changes are desirable.

Yes, I definitely want to merge it, but I haven't found a reliable way to reproduce/test it yet with Playwright.

Short of forking/patching Inertia, is it possible for developers to implement some flavor of this fix in their own apps? Or is it a core problem that requires changes in Inertia itself?

I'm afraid this should really be fixed in Inertia itself. I'll try to work on this next week!

[WarcraftLogs]

Any update here? I have sites that are heavily affected by this.

[wescopeland]

Still indeed affected by this. Would love to have a fix available in some form, even if it's a configurable option.

[pascalbaljet]

Closed in favor of #2721.