Description

Moving @bstopp's PR from aem-boilerplate.

Opting for an empty alt attribute here in lieu of setting role attribute to presentation based on recommendations from Web Hypertext Application Technology Working Group and the Web Accessibility Initiative.

The Worldwide Web Consortium provides an example for the use of role="presentation":

An image that is in a container with the img role and where the full text alternative is available and is marked up with aria-labelledby and (if needed) aria-describedby;

Where, in this instance, we don't meet that threshold without the "full text alternative" in this specific implementation.

Related Issue

18

Motivation and Context

"The Lighthouse output results in a "Links do not have a discernible name" accessibility error which reduces the score" — @dave-fink

"By default, all icons would be purely decorative (most cases), but projects could opt-in to individually override icons they know should be descriptive either in their blocks or in the auto-blocking function." — @ramboz

How Has This Been Tested?

/test/decorate/decorateIcon.test.html

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have signed the Adobe Open Source CLA.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

adobe / aem-lib

feat: add alt text support to decorateIcon #26