doc: remove incorrect scroll-padding-top offset

[avivkeller]

Fixes#53594

This PR modifies the behavior of the docs to not begin scrolling halfway through the screen, but rather at the browser default.

Before

screen-capture.2.webm

After

screen-capture.4.webm

[nodejs-github-bot]

Review requested:

• @nodejs/nodejs-website

[ovflowd]

The title is erroneous, no?

[avivkeller]

I don't think so, this PR updates the scrolling speed to be slower than what it was, what do you recommend for the commit?

[ovflowd]

No? The title should be: "doc: removing wrong scroll padding"

[avivkeller]

Got it, thanks! I'll fix it now!

[ovflowd]

I guess I understand now what that other PR was, but isn't this a duplicated PR then?

[avivkeller]

I guess I understand now what that other PR was, but isn't this a duplicated PR then?

What other PR? The linked item is an issue. It was describing a behavior that this PR fixes, but the way they worded the behavior was a bit odd IMO.

[ovflowd]

Ah interesting! I thought the same author made a PR reverting their changes that got merged back a few while ago or something like that. Too many notifications, I might be seeing things.

[jakecastelli]

I think you might be correct about the other PR @ovflowd, do you mean this one?

kudos to both of you for looking into and fixing this annoying bug, I literally gave up on selecting text on the doc page and copied directly from the source code 😄

[avivkeller]

Ha! I didn't see that PR, closing in favor.

Great minds think alike (and apparently time alike too)