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

Page turn links to external sites are broken #186

Closed
mwcraig opened this issue Apr 30, 2019 · 2 comments · Fixed by #188
Closed

Page turn links to external sites are broken #186

mwcraig opened this issue Apr 30, 2019 · 2 comments · Fixed by #188
Labels
bug Something isn't working

Comments

@mwcraig
Copy link
Contributor

mwcraig commented Apr 30, 2019

External links are generated fine in the sidebar. In the "page turn" links at the bottom of the page they are generated as though they were internal links.

To see this, go to https://jupyter.org/jupyter-book/features/limits.html then scroll down to the bottom and hover over the "GitHub repository" link. As you can see in the screenshot, the generated link prepends the book's URL, so in this case the URL ends up being https://jupyter.org/jupyter-book/https://github.com/jupyter/jupyter-book

This link is defined in the toc at https://github.com/jupyter/jupyter-book/blob/master/jupyter_book/book_template/_data/toc.yml#L71

Feature_Limits_and_grab_status_py

@choldgraf
Copy link
Member

ah hah - that's a good catch.

For this kind of thing, what do you think is the best behavior? It feels kinda weird to leave the site by going to the "next page", although then again that is technically the book structure that's been put forward

@choldgraf choldgraf added the bug Something isn't working label May 1, 2019
@mwcraig
Copy link
Contributor Author

mwcraig commented May 1, 2019

I think it would be useful to have an option for sidebar-only links, a header with a link in the sidebar but not in the navigation. I guess Search must be handled something like that since it shows up in the sidebar but not as part of the page turn links.

I agree that suddenly navigating away from the book content is not the way to go.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants