508-A Administration / WSRP page icons do not have ALT text value assigned. This violates 508 certification requirements: § 1194.22 (a) Text Equivalents Provision a text equivalent for every non-text element shall be provided (e.g., via "alt", "LongDesc", or in element content).
Created attachment 744618 [details] wsrp page icons missing alt texts
Fixed in master. https://issues.jboss.org/browse/GTNPORTAL-2983
I'm rejecting the PR for WSRP admin since I believe that CSS-background, non-semantic images shouldn't have an alt description attached to them. It would only add noise and actually be detrimental to accessibility. See http://christianheilmann.com/2009/02/25/so-how-do-you-add-alternative-text-to-background-images/ for more information on this.
Lucas Ponce <ponce.ballesteros> made a comment on jira GTNPORTAL-2983 PR for gatein-portal https://github.com/gatein/gatein-portal/pull/444 PR for gatein-wsrp https://github.com/gatein/gatein-wsrp/pull/76
Lucas Ponce <ponce.ballesteros> made a comment on jira GTNPORTAL-2983 Summary of changes: adding title="<description>" attribute in those <a> <div> elements that use background images in their style.
Hi, We don't have a feedback from Criterion company about background images. In their document (page 13) they showed WSRP admin page as an example of what have to be fixed. I've proposed a couple of non-intrusive changes in the following PR: https://github.com/gatein/gatein-wsrp/pull/76/files That adds a "title" attribute into <a> tag. "title" attribute is used by assistance technologies for additional info about link. We can merge this change or we can wait until we have a feedback from audit company. As this change is just decorative/non-intrusive I would suggest that we should merge, in case that we have a limited of # feedbacks from audit company. Regards, Lucas
Thomas Heute <theute> made a comment on jira GTNPORTAL-2983 +1 to merge
-1 on merging as I previously mentioned, this only adds noise and actually contradicts the goal of 508. Plus I'm not even sure that adding titles will fix the issue since the image is a background one and the title will only apply to the <a> tag and not the image itself. I guess the end goal here is are we trying to make GateIn easier to use for disabled people or are we trying to satisfy a stupid conformance tool?