Skip to content

index.html: Display post summaries below date#12

Open
devurandom wants to merge 1 commit intogetzola:masterfrom
devurandom:feature/display-post-summary
Open

index.html: Display post summaries below date#12
devurandom wants to merge 1 commit intogetzola:masterfrom
devurandom:feature/display-post-summary

Conversation

@devurandom
Copy link
Copy Markdown
Contributor

This adds a little bit of information to the post title by displaying its description or summary, if available.

@devurandom devurandom force-pushed the feature/display-post-summary branch from 29586c5 to a587524 Compare September 3, 2018 15:23
@Keats
Copy link
Copy Markdown
Collaborator

Keats commented Sep 3, 2018

Can that be a theme settings? The original hyde theme doesn't have summaries so that would be a deviation from it that should be gated.

Comment thread templates/index.html Outdated
<span class="post-date">{{ page.date | date(format="%Y-%m-%d") }}</span>

<span class="post-summary">
{% if page.description %}
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

i would also put page.summary as the first condition as description is something used more for SEO/metadata typically

@devurandom devurandom force-pushed the feature/display-post-summary branch from a587524 to 9edb002 Compare September 3, 2018 17:35
@devurandom
Copy link
Copy Markdown
Contributor Author

Theme settings: Would be fine with me. I can also simply create a derived theme and just leave Hyde alone in its current form.

@NyxCode NyxCode mentioned this pull request Sep 25, 2018
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