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

feat: allow to limit the number of tree posts #196

Merged
merged 2 commits into from
Jun 12, 2020
Merged

Conversation

reuixiy
Copy link
Owner

@reuixiy reuixiy commented Jun 11, 2020

Closes #97

@reuixiy reuixiy requested a review from palant June 11, 2020 03:43
@reuixiy reuixiy changed the title feat: allow to limit the number of posts listed per category/section entry feat: allow to limit the number of tree posts Jun 11, 2020
@reuixiy reuixiy merged commit 6f3aeba into master Jun 12, 2020
@reuixiy reuixiy deleted the limit-tree-posts branch June 12, 2020 01:49
@reuixiy
Copy link
Owner Author

reuixiy commented Jun 12, 2020

Sorry, just can't wait to merge 😆

Copy link
Collaborator

@palant palant left a comment

Choose a reason for hiding this comment

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

Again a holiday in Germany yesterday. 😄

Either way, this seems to be ok. But the logic of the tree-categories template seems unnecessarily convoluted, I'll need to see eventually whether I can simplify it.

# Set to a non-negative number to limit
# the number of posts displayed per
# category/section.
limitPostsLimit = -1
Copy link
Collaborator

Choose a reason for hiding this comment

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

Did you mean limitPostsCount? I see that I made the same mistake in my change. 😢

Copy link
Owner Author

Choose a reason for hiding this comment

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

I just referenced that, lol

ulmefors pushed a commit to ulmefors/hugo-theme-meme that referenced this pull request Nov 22, 2020
Allow to limit the number of posts listed per category/section entry

Closes reuixiy#97
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.

feature request : 分类页面增加折叠按钮
2 participants