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

Support for alt text on images posted to Twitter (blocked on mf2py) #756

Closed
miklb opened this issue Jul 5, 2017 · 10 comments
Closed

Support for alt text on images posted to Twitter (blocked on mf2py) #756

miklb opened this issue Jul 5, 2017 · 10 comments
Labels

Comments

@miklb
Copy link

miklb commented Jul 5, 2017

It seems that it is doable with the Twitter REST API to pass alt text for images

Not 100% sure how to do so with Bridgy, unless bridgy can parse the alt attribute from the image with a u-photo or u-featured class. Then it would be pretty straight forward I would guess.

@gRegorLove
Copy link

@snarfed
Copy link
Owner

snarfed commented Jul 10, 2017

this is currently blocked on mf2py supporting img alt text explicitly, microformats/mf2py#83, based on microformats/microformats2-parsing#2.

(also, for my own notes, i'm tentatively planning to serialize HTML alt into AS1 displayName. alternative was summary.)

@snarfed snarfed changed the title Support for alt text on images posted to Twitter Support for alt text on images posted to Twitter (blocked on mf2py) Jul 10, 2017
@snarfed snarfed added the now label May 9, 2018
@snarfed snarfed removed the now label Jun 5, 2018
snarfed added a commit to snarfed/granary that referenced this issue Jul 20, 2018
@snarfed
Copy link
Owner

snarfed commented Jul 27, 2018

this is deployed! still needs more testing and bug fixes - eg i need to ellipsize at 420 chars, and i think it doesn't work with multiple photos right now - but feel free to try it!

@snarfed
Copy link
Owner

snarfed commented Jul 27, 2018

fixed both of those issues.

@snarfed
Copy link
Owner

snarfed commented Sep 17, 2018

recent example of a twitter publish with alt text:

@jamietanna
Copy link
Contributor

@snarfed I'm not sure if this may have broken again, but looking at this tweet in Granary, I don't see the alt tag returned, unless I'm doing something wrong?

@snarfed
Copy link
Owner

snarfed commented Feb 1, 2020

ah, i only implemented this for publishing, not consuming. i take it you're interested in the consuming side in granary? looks doable! interested in tackling it? 😁

@jamietanna
Copy link
Contributor

Sorry yep, I'm only looking for the consuming side as I've seen that Bridgy does publish the alt tags to Twitter.

I wasn't looking where I was and thought this issue was on Granary - sorry! I'll raise an issue for it, not sure if I'll get round to it soon, but it'll at least be there for someone else to

@Ryuno-Ki
Copy link

Ryuno-Ki commented Feb 2, 2020

@jamietanna Perhaps helpful to crosslink it here when you open the ticket.

@jamietanna
Copy link
Contributor

@Ryuno-Ki snarfed/granary#183 is the issue, of which snarfed has already implemented a fix 🙌🏼

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

No branches or pull requests

5 participants