Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
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