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

Adds navigation for posts marked as a part of a series. #72

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

koundinya
Copy link

@koundinya koundinya commented Aug 18, 2019

Context -
http://www.joesacher.com/blog/2017/08/03/implementing-series-in-hugo/

Kiss has the ability to add related posts, but when you are writing a series of posts, it would be helpful to have a way to navigate between them.

Current implementation is based on the link mentioned above.

Screenshot 2019-08-18 at 9 07 13 PM

  1. This can be made configurable from config.toml , would need changes in the pr.
  2. Currently I've added
    {{ partial "series_navigation.html" . }}
    after the content it could be placed above as well.

@ribice
Copy link
Owner

ribice commented Oct 20, 2019

Sorry for replying this late. I'd like to address this PR now.

What other changes are needed in config.toml from your side to get this merged?

@koundinya
Copy link
Author

I was thinking it should be left to the user to decide if they need series navigation or not and that should be configured via the config.toml, wasn't sure if this is the direction you would like to take and didn't implement. In terms of code, {{ partial "series_navigation.html" . }} should have a check to see if the feature is enabled in the config. What are your thoughts on this?

@ribice
Copy link
Owner

ribice commented Nov 28, 2019

I'd keep it at the bottom only for now and make it configurable in config.toml

How do you mark a post as being in series?

@ribice
Copy link
Owner

ribice commented Jun 28, 2022

Is this PR ready @koundinya ?

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