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

Add minimum_tag_count param to _layouts/tags.html #1697

Closed
wants to merge 1 commit into from
Closed

Add minimum_tag_count param to _layouts/tags.html #1697

wants to merge 1 commit into from

Conversation

e28eta
Copy link
Contributor

@e28eta e28eta commented Jun 3, 2018

Adds a page parameter minimum_tag_count. Any tags with fewer than minimum_tag_count (no bounds checking, less than 1 is like 1) will be omitted from the tags page.

My tags (currently) outnumber the posts, and I felt it was really noisy going to the Tags page and seeing a long list of tags with only one post. So, adding this parameter allows adding minimum_tag_count: 2 to the front matter of the Tags page, and it hides any tag with just 1 post.

It is semi-broken because the tag on the post links to the Tags page, which then doesn't have that tag.

Thanks for all the awesome work on this theme, and I won't mind if you don't think this is right for the project (or if there's a better way to achieve this). I had the idea, tried it out, and thought it might be worth sharing. I don't know if this is useful enough to merge, or if I'm in the minority wanting this feature.

Adds a `page` parameter `minimum_tag_count`. Any tags with *fewer* than `minimum_tag_count` (no bounds checking, less than 1 is like 1) will be omitted from the tags page.
@mmistakes
Copy link
Owner

I can see the utility of this, but as you mention it breaks the anchor references to tags that might be omitted on the "tag archive page".

This seems like the perfect use case for doing a local override of the layout if it's something you need.

Thanks for taking the time to submit the PR. I'll give it the enhancement tag so others can potentially find it in the closed issues and use if they'd like.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants