Social Media Sharing Bugs and Enhancements

[keithmifsud]

• Change the Twitter Sharing method to use the new Web Intent.
• Consolidate the JS functions that handle the popup window.
• Change the display of the sharing buttons for mobile devices so that they are static at the end of the article's content. - Moved to a new issue to avoid delays #Change the display of the sharing buttons for mobile devices so that they are static at the end of the article's content. #68

[danielepolencic]

About 90% of the shares comes Twitter. I'd really like to fix this, what can we do?

[keithmifsud]

I'll find a way to update the share window so that it displays the text and image preview before tweeting. On it, as we speak.

[keithmifsud]

@danielepolencic As it is, when tweeting, the tweet appears like this:

Just to make sure we're both on the same issue. Currently, once clicking on the twitter share link, a small window opens like this:

Am I right to say that you want it to look like this?

If so, Twitter is a bit different (obviously) because of the characters limit and the ability to have hash tags and a twitter username. I assume that we want the @learnk8s as the username? Do you wnat to have automated hashtags generated? If so how should I get these? Add a front-matter property on each article?

[danielepolencic]

I think it would be nice if we could customise the message and tag @learnk8s

[keithmifsud]

Yes, the user will be able to change the contents of the text box. Same as they can do now but it will initially be auto-populated with an excerpt and p[ossibly a couple of hashtags.

[keithmifsud]

The issue is that an excerpt is already provided by the OG property. What should we add to the contents? This already works as it is, the only missing things are the image and the excerpt inside the share window. Once shared, they are visible on Twitter's timeline.

[danielepolencic]

Oh I see. Perhaps we could have something like "{Title} via @learnk8s"? we know the title is always short enough for a tweet.

[keithmifsud]

Okay. I can limit the length using a liquid filter because the new "Web Intent" from Twitter's docs say that if the length is exceeded it won't be automatically cropped out and (although not clear) I assume that the text will simply not be included. Bear in mind that the URL text is included in the limit as it is displayed as part of the tweet.

[danielepolencic]

I think that should be fine for a first release.