Skip to content

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

Merged
jekyllbot merged 1 commit intojekyll:masterfrom
swagar:patch-1
Sep 15, 2024
Merged

docs: Fix of a bug in the pagination section#8890
jekyllbot merged 1 commit intojekyll:masterfrom
swagar:patch-1

Conversation

@swagar
Copy link
Copy Markdown
Contributor

@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
Copy link
Copy Markdown
Member

@ashmaroli ashmaroli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!

@ashmaroli
Copy link
Copy Markdown
Member

Thanks @swagar
@jekyllbot: merge +doc

@jekyllbot jekyllbot merged commit 34e3528 into jekyll:master Sep 15, 2024
jekyllbot added a commit that referenced this pull request Sep 15, 2024
github-actions bot pushed a commit that referenced this pull request Sep 15, 2024
Simon Wagar: Fix of a bug in the pagination section (#8890)

Merge pull request 8890
@jekyll jekyll locked and limited conversation to collaborators Oct 14, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants