Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Instant loading is broken #543

Open
xerbalind opened this issue Aug 10, 2023 · 4 comments · Fixed by #547
Open

Instant loading is broken #543

xerbalind opened this issue Aug 10, 2023 · 4 comments · Fixed by #547

Comments

@xerbalind
Copy link
Contributor

xerbalind commented Aug 10, 2023

While exploring the website, each time you navigate through the side menu or table of contents, the page undergoes a complete reload.

Within the code, two types of markdown links exist:

[link](../link)

OR

[link](link.md)

The latter choice is significantly preferred.

When clicking a link from either option, the page loads instantly as anticipated. However, only after clicking on the the second type of link, from now on, when you navigate via the menu on the left, the page loads instantly (better). With the first link type the complete reloading issue persists.

I think it's something to do with os pick redirection

@xerbalind
Copy link
Contributor Author

It seems that that when using a subdirectory like

[link](../subdir/link)

you get the same behaviour as option 2 link

@xerbalind
Copy link
Contributor Author

If you click Next at bottom of page instant loading is also fixed

@boegel
Copy link
Member

boegel commented Aug 14, 2023

Re-opening to get some feedback from @stdweird, but as far as we can tell removing the code in #547 doesn't cause trouble

@xerbalind
Copy link
Contributor Author

xerbalind commented Aug 28, 2023

Re-opening to get some feedback from @stdweird, but as far as we can tell removing the code in #547 doesn't cause trouble

It seems removing the code causes this problem: #558

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants