The field image tweet haven't ALT and TITLE html tags. Whitout alt and title, W3C compatibility breaks.
This patch add the following translate string:

  • alt: View image on Twitter
  • title: View image on Twitter

The string "View image on Twitter" is showed in Twitter Timelines.

CommentFileSizeAuthor
#8 twitter-accessibility-view-field-image_1.patch570 bytesvisabhishek
#6 twitter-accessibility-view-field-image_1.patch.txt570 bytesmarco.aresu
#2 twitter-accessibility-view-field-image.patch706 bytesmarco.aresu
twitter-accessibility-view-field-image.patch706 bytesmarco.aresu

Comments

marco.aresu created an issue. See original summary.

marco.aresu’s picture

marco.aresu commented
StatusFileSize
new twitter-accessibility-view-field-image.patch706 bytes
damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Version: 7.x-5.11 » 7.x-5.x-dev
Status: Active » Needs review
Issue tags: -views field, -twitter images
damienmckenna’s picture

damienmckenna
Location TN, USA
commented

Thanks for the patch, lets see what the testbot says.

Status: Needs review » Needs work

The last submitted patch, 2: twitter-accessibility-view-field-image.patch, failed testing. View results

marco.aresu’s picture

marco.aresu commented
StatusFileSize
new twitter-accessibility-view-field-image_1.patch.txt570 bytes

I removed absolute twitter module path from the patch.

visabhishek’s picture

visabhishek commented
Status: Needs work » Needs review

Marking as Need Review to call testbot

visabhishek’s picture

visabhishek commented
StatusFileSize
new twitter-accessibility-view-field-image_1.patch570 bytes

Just changed the ext of #6 to run testbot