Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Comment | File | Size | Author |
---|---|---|---|
#8 | panopoly-add_test_image_widgit-2149157.patch | 1.62 KB | bwood |
#2 | add-test-image-widget.patch | 1.45 KB | lsolesen |
Comments
Comment #1
dsnopekComment #2
lsolesen CreditAttribution: lsolesen commentedMaybe there should be a test here also for whether the image is actually shown on the page.
Comment #3
dsnopekYeah, it'd be great to be able to say:
But then we'll need to also upload the image and get the "Alternate text" field.
Comment #4
lsolesen CreditAttribution: lsolesen commentedIn wetkit the accessibility test has a check for alt text
Comment #5
dsnopekAh, sweet! I found the code here:
http://drupalcode.org/project/wetkit.git/blob/refs/heads/7.x-1.x:/tests/...
Can you add this to your patch and use it for the image test?
Comment #6
lsolesen CreditAttribution: lsolesen commentedCheckout https://travis-ci.org/lsolesen/panopoly/builds/14969671 --> For some reason I could not get it to work correctly. And I have no more time to dig into it for now. It is probably a simple spelling mistake somewhere?
Comment #7
dsnopekComment #8
bwood CreditAttribution: bwood commentedHere's a patch that adds the alt text test.
Comment #9
dsnopekWorked for me, so I've committed! Thanks @lsolesen and @bwood!
@bwood: You now have you're first commit listed on your Drupal.org profile page. :-)