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

Add Mastodon sharing link #1344

Merged
merged 2 commits into from
Oct 29, 2023
Merged

Add Mastodon sharing link #1344

merged 2 commits into from
Oct 29, 2023

Conversation

some-natalie
Copy link
Contributor

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Improvement (refactoring and improving code)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update

Description

This PR adds a "share to Mastodon" button as a link sharing option using @justinribeiro's share-to-mastodon package (GitHub and NPM). I made a best guess at the correct CSS color variables to use and it seems to look alright.

  • Example usage was added to _data/share.yml
  • Liquid tags reworked slightly in _includes/post-sharing.html to only add the button if it was selected as an option

Additional context

Discussion of proposed change - #1324
Example site w/ change in place - https://some-natalie.dev/

Copy link
Owner

@cotes2020 cotes2020 left a comment

Choose a reason for hiding this comment

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

Thanks for submitting this feature, it works fine. But there are still a lot of improvements that can be made, such as the definition of the YAML configuration, code snippets for Liquid/HTML/CSS, etc.

Don't worry, I'll add a new commit later to complete these improvements.

@cotes2020
Copy link
Owner

cotes2020 commented Oct 28, 2023

I've added a new refactoring commit (1a28db4) for you to review. If you have any ideas for changes, feel free to leave a comment and let me know. Otherwise I'll merge this PR.

@some-natalie
Copy link
Contributor Author

some-natalie commented Oct 29, 2023

@cotes2020 Thank you! 💖

Thanks for 👀 on this, but also for refactoring it.

It was not my intention to give you work. While I don't know much front-end things, I would gladly have tried to figure it out given a pointer or two. I only intended to share a neat feature.

LGTM, but as mentioned I may not be the best one to say that. And thanks again. ✨

@cotes2020
Copy link
Owner

Okay, I'll merge this PR after I refine some details.

@cotes2020 cotes2020 merged commit 2bf87e0 into cotes2020:master Oct 29, 2023
3 of 5 checks passed
cotes2020 added a commit that referenced this pull request Nov 5, 2023
Adds a "share to Mastodon" button as a link sharing option using @justinribeiro's share-to-mastodon package

Discussion of proposed change - #1324

---------

Co-authored-by: Cotes Chung <11371340+cotes2020@users.noreply.github.com>
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