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.
The current icons for credit cards in the payment options do not have alt tags, making them less accessible.
I have a patch that adds these with a simple alt tag for the card type (e.g. visa, mastercard, etc).
Kim
Comment | File | Size | Author |
---|---|---|---|
#3 | add-alt-tags-to-card-types-1615754-3.patch | 587 bytes | kim.pepper |
#1 | add-alt-tags-to-card-types-1615754-1.patch | 786 bytes | kim.pepper |
Comments
Comment #1
kim.pepperPatch attached.
Comment #2
TR CreditAttribution: TR commentedtheme('image', ...) should be used here instead of manually creating the img tag, and the explicit styles should be removed and put into the .css. I'd also strongly prefer that this be done in 7.x-3.x first, to ensure that improvements in the code base get propagated forward.
Comment #3
kim.pepperAdded patch against 7.x-3.0-dev
Comment #5
longwavePorted the code from uc_paypal that does something very similar, which gives better text - e.g. "American Express" instead of "amex".
http://drupalcode.org/project/ubercart.git/commitdiff/4d96430
Comment #6
longwavePorted and committed in http://drupalcode.org/project/ubercart.git/commitdiff/49c2840