Skip to content
This repository has been archived by the owner on Dec 5, 2018. It is now read-only.

Replace Twitter button with ShareThis including Twitter, Facebook, Pinterest, and Email #67

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

msscully
Copy link
Contributor

@msscully msscully commented Jun 7, 2013

Twitter and Facebook buttons were different sizes so used ShareThis to
get matching Twitter, Facebook, Pinterest, and Email shring buttons.

Twitter and Facebook buttons were different sizes so used ShareThis to
get matching Twitter, Facebook, Pinterest, and Email shring buttons.
<span class='st_twitter_vcount' displayText='Tweet'></span>
<span class='st_pinterest_vcount' displayText='Pinterest'></span>
<span class='st_email_vcount' displayText='Email'></span>
</div>
Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks for doing this! It looks great...

I haven't had a chance to test it locally yet, but do we still include the hashtag, text, and link in a similar way? Or does it just use the current location?

@msscully
Copy link
Contributor Author

msscully commented Jun 7, 2013

Was thinking about this on the way home. Url defaults to page url. Can set it using st_url. We can also set the title using st_title, which defaults to document.title, st_via for a default 'via @whomever on twitter, and st_msg for customizing the message. I'm working on a change incorporating these features.

@benjaminoakes
Copy link
Contributor

Cool, thanks. :) We should get a Twitter account, maybe... @AdoptaLOL seems to be abandoned.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants