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

Update Social Sharing links section #4066

Merged
merged 2 commits into from
May 5, 2024

Conversation

jstrong013
Copy link
Contributor

This is a documentation change.

Summary

Update social sharing section for appropriate Reddit example and resource locations.

Context

Could use a second set of eyes. Open to edits.

Related to Issue #1656 and pull request #1657

Update social sharing section for appropriate Reddit example and resource locations.
Copy link
Collaborator

@iBug iBug left a comment

Choose a reason for hiding this comment

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

LGTM except for the unnecessary onclick attribute. Might be better to keep it simple in the documentation.

Also default: 'Share on' is not required. There's more to break if site.locale is nonsensical.

@iBug iBug merged commit 0378a31 into mmistakes:master May 5, 2024
1 check passed
@jstrong013 jstrong013 deleted the reddit-social-share-docs branch May 17, 2024 12:43
minyoongi96 pushed a commit to minyoongi96/minyoongi96.github.io that referenced this pull request Aug 26, 2024
* Update 10-layouts.md

Update social sharing section for appropriate Reddit example and resource locations.

* Update as reviewed

---------

Co-authored-by: iBug <[email protected]>
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