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

docs: Fix of a bug in the pagination section #8890

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

Conversation

swagar
Copy link

@swagar swagar commented Nov 25, 2021

There is a little bug in the laste code snippet in the pageination section. The link for the first page does not work when the paginate_path property points to a subfolder. In that case {{ '/' | relative_url }} points to the base url and ignores the subfolder.

I made a little demo which you can find here. There you find the link for the first page twice. One with the old link und one with the new and working link.

You can finde the source code here

Summary

Context

There is a little bug in the laste code snippet in the pageination section. The link for the first page does not work when the `paginate_path` property points to a subfolder. In that case `{{ '/' | relative_url }}` points to the base url and ignores the subfolder.

I made a little demo which you can find [here](https://swagar.github.io/wrong-page1-link/blog/page2/). There you find the link for the first page twice. One with the old link und one with the new and working link.

You can finde the source code [here](https://github.com/swagar/wrong-page1-link/blob/master/blog/index.html#L38)
@swagar swagar changed the title Fix of a bug in the pagination section docs: Fix of a bug in the pagination section Nov 25, 2021
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

1 participant