Please add alt text. This module is making my new site fail wave accessiblity validation.

CommentFileSizeAuthor
#4 AltText-For-AddThis-1335510-4.patch505 bytesmgifford

Comments

Everett Zufelt’s picture

Everett Zufelt commented
Component: Code » User interface

Can you please be mor clear about what is required, or link to the WAVE report of an active / development site?

Thanks

matglas86’s picture

matglas86 commented

Hi @benjarlett, I know this is a beta1 question but since version 4 is progressing I would like to know what you would want to see to have better accesibility standard in the latest version of AddThis. Please let me know so we can start a issue on that. Thanks.

mgifford’s picture

mgifford
he/him
Primary language English
commented

In the config page under - Additional configuration - there is a checkbox for - 508 compliant

I'm not sure why that's not selected by default. I'm also not sure what it does. It's good sign, but would be more clear for everyone if it focused on the global WCAG 2.0 standard.

mgifford’s picture

mgifford
he/him
Primary language English
commented
StatusFileSize
new AltText-For-AddThis-1335510-4.patch505 bytes

This patch at least gets the first image in there. It's based on Git. The images inside of the AddThis box are all displayed as CSS background images from the looks of it and have representative text under them so it should be fine.

matglas86’s picture

matglas86 commented
Version: 7.x-2.1-beta1 » 7.x-4.x-dev
Status: Active » Needs review
matglas86’s picture

matglas86 commented
Status: Needs review » Fixed

Fixed and commit in e4fc45.

Automatically closed -- issue fixed for 2 weeks with no activity.