Skip to content

Add category pages.#11

Closed
alextsui05 wants to merge 1 commit intoellekasai:gh-pagesfrom
alextsui05:master
Closed

Add category pages.#11
alextsui05 wants to merge 1 commit intoellekasai:gh-pagesfrom
alextsui05:master

Conversation

@alextsui05
Copy link
Copy Markdown

Hi,

I like using Shiori theme but wanted category pages, so I added this. What do you think? Here's a demo.

Pages will be generated for each category and list all posts with that
category. Posts that specify a list of categories in the front matter will
be linked to the corresponding category pages.

  • Import and modify generate_categories.rb from recurser/jekyll-plugins.
    It generates /categories/#{category}/index.html for each category.
    These pages list all posts that have that category.
  • Modify post-header.html to print a post's categories, if available.
  • Add category_index.html, the layout used by the category pages.
  • Add categories to example pages to demonstrate.
  • Add an example post to demonstrate how to use it.

Posts that specify a list of categories in the front matter will be
grouped and linked to the corresponding category pages.

- Import and modify generate_categories.rb from recurser/jekyll-plugins.
  It generates /categories/#{category}/index.html for each category.
  These pages list all posts that have that category.
- Modify post-header.html to print a post's categories, if available.
- Add category_index.html, the layout used by the category pages.
- Add categories to example pages to demonstrate.
- Add an example post to demonstrate how to use it.
@ellekasai
Copy link
Copy Markdown
Owner

Hi Alex,

Thanks for the pull request! The demo page is great. I also added a few comments on your code.

Unfortunately, I can't merge your pull request because as you said, the demo doesn't work automatically with Github Pages, and I'd like to make sure Shiori works on GitHub Pages.

However, I added a section on README mentioning your demo and this pull request. Could you take a look?

Thanks for your work!
Elle

@ellekasai ellekasai closed this Jan 12, 2015
@alextsui05
Copy link
Copy Markdown
Author

Thanks for the mention and the code review! I've made the changes for your suggestions.

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