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

Fix a back-link bug #106

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

Conversation

dthex5d
Copy link

@dthex5d dthex5d commented Feb 18, 2021

A bug, which creates the empty-back-link when the page.back_page property is not declared, has been found.
It seems like the 'find' filter of the liquid does not work properly.
Now, back-link.html uses both 'where' and 'first' filters to create the proper back link, only when the the page.back_page is declared in the current page.

A bug, which creates the empty-back-link when the page.back_page property is not declared, has been found.
It seems like the 'find' filter of the liquid does not work properly.
Now, back-link.html uses both 'where' and 'first' filters to create the proper back link, only when the page.back_page is declared in the current page.
@Tetr4
Copy link

Tetr4 commented May 18, 2022

I think the issue is, that the find filter requires Jekyll 4.1.0, but GitHub Pages currently only supports 3.9.2, so the back arrow is always shown, but has no href or label.

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.

2 participants