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

Better blog pages #798

Merged
merged 5 commits into from
Jul 5, 2022
Merged

Better blog pages #798

merged 5 commits into from
Jul 5, 2022

Conversation

h-enk
Copy link
Member

@h-enk h-enk commented Jun 24, 2022

Summary

  • Add featured image support (with fallback to default image)
  • Add categories + tags support (next to already existing contributors support)
  • Add related posts to single blog post page
  • Configure related logic in ./config/_default/config.toml:
    [related]
      threshold = 80
      includeNewer = true
      toLower = false
        [[related.indices]]
        name = "categories"
        weight = 100
        [[related.indices]]
        name = "tags"
        weight = 80
        [[related.indices]]
        name = "date"
        weight = 10
    

Basic example

Include a basic example, screenshots, or links.

https://deploy-preview-798--doks.netlify.app/

Snag_1533075 Snag_152f570

Motivation

Why are we doing this? What use cases does it support? What is the expected outcome?

Checks

  • Read Create a Pull Request
  • Supports all screen sizes (if relevant)
  • Supports both light and dark mode (if relevant)
  • Passes npm run test

@james-d-elliott
Copy link
Contributor

This looks awesome!

I'm wondering if the categories are better below the title and above the byline. Thoughts?

@h-enk
Copy link
Member Author

h-enk commented Jun 28, 2022

I guess you right. Or, make it part of the byline, like so:

Posted in Startups, News on June 24, 2022 by Henk Verlinde ‐ 3 min read

@james-d-elliott
Copy link
Contributor

I guess you right. Or, make it part of the byline, like so:

Posted in Startups, News on June 24, 2022 by Henk Verlinde ‐ 3 min read

Ah yep! That's probably better, I'm not any kind of visual designer. Just they felt a bit out of place.

@h-enk
Copy link
Member Author

h-enk commented Jul 5, 2022

Hyas CI fails because of https://github.com/h-enk/doks/blob/master/config/_default/params.toml#L53. To be fixed after merge

@h-enk h-enk merged commit 97b7af3 into master Jul 5, 2022
@h-enk h-enk deleted the blog-pages branch July 5, 2022 09:52
@h-enk
Copy link
Member Author

h-enk commented Jul 5, 2022

93d9202

@h-enk h-enk mentioned this pull request Jul 5, 2022
4 tasks
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