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

[Dev docs] Fix previous and next page links #1601

Merged
merged 1 commit into from
Jul 29, 2019

Conversation

nguyenhuy
Copy link
Member

@nguyenhuy nguyenhuy commented Jul 26, 2019

Our pages are organized in 2 dirs: /docs/ and /development/.

Within a dir, prevPage and nextPage are relative (to the dir) so its path should be appended to their href links (see docs/_layouts/docs.html). The dir is available in the page's permalink var.

Some prevPage and nextPage point to a page in a different dir in which case they must not be relative. Instead they must include the other dir's path and thus their links won't be modified.

Also removed some pages that are no longer needed and fixed some prevPage and nextPage values.

I tested this diff locally and all links worked fine.

Our pages are organized in 2 dirs: /docs/ and /development/. Within a dir, prevPage and nextPage are relative so the dir should be appended to the href link. The dir is available in permalink var. Some prevPage and nextPage point to a page in a different dir in which case they must not be relative. Instead they must include the dir and thus won't be modified.
@nguyenhuy nguyenhuy requested a review from mikezucc July 29, 2019 18:17
@nguyenhuy nguyenhuy merged commit 0568a53 into TextureGroup:master Jul 29, 2019
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.

None yet

3 participants