Skip to content
This repository has been archived by the owner on Aug 29, 2023. It is now read-only.

mdButton with mdIcon SVG throws aria-label warnings #1458

Closed
marcysutton opened this issue Feb 10, 2015 · 0 comments
Closed

mdButton with mdIcon SVG throws aria-label warnings #1458

marcysutton opened this issue Feb 10, 2015 · 0 comments
Assignees
Labels
a11y This issue is related to accessibility type: bug

Comments

@marcysutton
Copy link
Contributor

For accessibility purposes, mdIcon expects a label internally. In our mdButton demo, although mdIcon's parent element provides adequate labeling for assistive technologies, a warning is logged to the console. The correct behavior would be to only warn if there is no text available in the icon to copy to aria-label, nor a label on the parent button element.

@marcysutton marcysutton added type: bug a11y This issue is related to accessibility labels Feb 10, 2015
@marcysutton marcysutton changed the title mdButton with mdIcon throws aria-label warnings mdButton with mdIcon SVG throws aria-label warnings Feb 10, 2015
marcysutton pushed a commit that referenced this issue Feb 10, 2015
@ajoslin ajoslin added the in progress Mainly for in progress PRs, but may be used for issues that require multiple PRs label Feb 10, 2015
marcysutton pushed a commit that referenced this issue Feb 10, 2015
@ajoslin ajoslin removed the in progress Mainly for in progress PRs, but may be used for issues that require multiple PRs label Feb 10, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
a11y This issue is related to accessibility type: bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants