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

Allow selective summary hiding in front matter #292

Merged
merged 1 commit into from Mar 18, 2021
Merged

Allow selective summary hiding in front matter #292

merged 1 commit into from Mar 18, 2021

Conversation

mohnoor94
Copy link
Contributor

The previous style of getting params does not allow hiding post summary from the listing page in the front matter. With this change, the below can be done to hide the summary for a specific post or page. This change mirrors how ShowBreadCrumbs already works.

---
title: "Test Post"
hideSummary: true
---

post contents...

@sonarcloud
Copy link

sonarcloud bot commented Mar 6, 2021

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@adityatelange
Copy link
Owner

What is the use case here ?

@mohnoor94
Copy link
Contributor Author

When you set the summary var to null or empty it shows an annoying three dots without anything before them in the home/blog pages. So this will allow you to hide it completely (while still providing a summary in case you want to show it later, just flip or remove the new option).

@adityatelange adityatelange merged commit ad81825 into adityatelange:master Mar 18, 2021
@mohnoor94 mohnoor94 deleted the hide-summary branch March 18, 2021 20:23
kylethedeveloper pushed a commit to kylethedeveloper/hugo-PaperMod that referenced this pull request Feb 21, 2023
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.

None yet

2 participants