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

Use HTTP redirection from index.html to main content #7

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

aunger
Copy link

@aunger aunger commented Sep 2, 2020

Use 302 HTTP redirection to load the main page in Live Reader, instead of copying its contents into index.html.

This change was part of a false start to fixing a linking error, but it turned out to be a nice option.

It's certainly just an option though.

Advantages:

  1. Jumping to anchors from the index previously required another page load, but now that page is already the active one, so the browser only needs to scroll the target into view.
  2. There are no longer two identical URLs with the main page content.

Disadvantage:

  1. When visiting the manual's top level, the URL will look uglier. Instead of ending with /index.html or /, it will end with the main page's filename (/01-wycliffe%20associates%20translation.html in the case of tM).

This change is Reviewable

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 this pull request may close these issues.

1 participant