twitrssme icon indicating copy to clipboard operation
twitrssme copied to clipboard

Correctly parse tweets with multiple images

Open TheoLeCalvar opened this issue 6 years ago • 4 comments

#48 Quick fix to handle tweets with multiple images.

TheoLeCalvar avatar May 24 '18 14:05 TheoLeCalvar

Thanks for this. Nice one. I'm going to need to test if it has any big performance cost before rolling it out.

ciderpunx avatar May 26 '18 13:05 ciderpunx

Ok. I also have another patch for gifs if you are interested. It is not complete though, only gifs video are supported, others need request to the API to get the url.

TheoLeCalvar avatar May 27 '18 12:05 TheoLeCalvar

Can this code handle emojis properly too?

rootless4real avatar Sep 26 '18 18:09 rootless4real

With this new commit it should strip the image and only keep the emoji in the alt attribute. It looked OK but I don't have time to properly test it now.

TheoLeCalvar avatar Sep 26 '18 19:09 TheoLeCalvar