Navigation does not remember scroll position

[Gerrit0]

Hello @Gerrit0,

I have encountered an issue related to the navigation and the changes made in commit 9751e31. Specifically, the navigation scrollbar keeps scrolling up on each page refresh. You can see a demonstration of this issue in Issue #2624.

I saw you solved this issue, but I observed that the navigation scrolling happens only when the navigation is not visible, which seems to be due to the negation in the visibility check !pageLink.checkVisibility().

Does this negation serve a specific purpose?

Originally posted by @dom-rau in #2491 (comment)

[Gerrit0]

#2624 is an entirely different issue, it is for TypeDoc's doc site which uses eleventy, not the API documentation generated by TypeDoc, (https://typedoc.org/guides/overview/, not https://typedoc.org/api/)

That negation was intended to only scroll the navigation if the page link wasn't already on the screen, which isn't the right way to check that... I incorrectly assumed that the method checked if the element was... visible...