Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

'slide' aria-label attribute added to <a> tags #1768

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

bjdevil21
Copy link

@bjdevil21 bjdevil21 commented Sep 16, 2019

Our accessibility people came back with this suggestion for screen readers:

"The pager (the dots under the slideshow when there is more than one slide) need labels (or ARIA labels). Currently a screed reader reads them as "List of 4 items. 1 group, 2. 2 group, 3..." etc. in VoiceOver. In JAWS sr says “link clickable 1, link clickable 2" It's not clear to the user what these links refer to.

This is the pager on https://newdesign-new-fitness.ws.asu.edu/:

  1. 1
  2. 2
  3. 3
  4. 4

You can either change the link wording:

  • Slide 1
  • or add an aria-label:
  • 1
  • to identify the link."

    =======================

    I didn't update the minified file (build process handles this?)

    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Labels
    None yet
    Projects
    None yet
    Development

    Successfully merging this pull request may close these issues.

    None yet

    1 participant