Skip to content

feat: add configurability which share links to present per post #1 #5233

Draft
cschwartz wants to merge 2 commits intommistakes:masterfrom
cschwartz:master
Draft

feat: add configurability which share links to present per post #1 #5233
cschwartz wants to merge 2 commits intommistakes:masterfrom
cschwartz:master

Conversation

@cschwartz
Copy link

This is an enhancement or feature.

Summary

This allows users to select (in _config.yml) via

share_on:
  x: true
  facebook: true
  linkedin: true
  bluesky: true

which social share links are displayed per post.
Defaults are set to true, so that there should be no breakage for default installations.

Happy to work with your requirements and make changes as needed :)

Context

No, I was not sure if there is an issue category for feature requests.
If this is not something you want in this project, we can still close the PR.

cschwartz added 2 commits May 17, 2025 08:56
This allows users to select (in _config.yml) via
```yaml
share_on:
  x: true
  facebook: true
  linkedin: true
  bluesky: true
```
which social share links are displayed per post.
Defaults are set to true, so that there should be no breakage for default installations.

Happy to work with your requirements and make changes as needed :)
…e-links

feat: add configurability which share links to present per post
@iBug
Copy link
Collaborator

iBug commented Jun 15, 2025

This might not work as intended. Liquid documentation:

default will show its value if the input is nil, false, or empty.

So an explicitly specified site.share_on.x = false will cause the button to show instead.

Test your code to show I'm wrong.

@iBug iBug marked this pull request as draft June 15, 2025 16:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants