Added `aria-label` to `alert` directive template #317

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Open

dcastro wants to merge 1 commit into yalabot:master from dcastro:alert-aria-label

dcastro commented Apr 25, 2017

We're using angular-foundation in our project and we noticed the "x" in an alert is read out as "times" by screen readers.

This PR adds an aria-label to the alert directive template for better accessibility.


          Added aria-label to alert directive template

23bd335

# for free to join this conversation on GitHub. Already have an account? # to comment

Labels

None yet