Skip to content

Added additional screen reader accessibility support to conform with the ARIA APG 1.1 Carousel Design pattern as recently changed with the W3C, plus fixed the breaking of keyboard and screen reader accessibility every time autoplay is set to true. - #3773

Open
accdc wants to merge 10 commits into
kenwheeler:masterfrom
accdc:master

Conversation

@accdc

@accdc accdc commented May 16, 2019

Copy link
Copy Markdown

No description provided.

@ahmadalfy

ahmadalfy commented Jul 16, 2019

Copy link
Copy Markdown
Collaborator

I am by no mean an expert on that so I will take your words for it. This is what you’re referring to?

If we no longer need this part:

// 'aria-label': (i + 1) + ' of ' + numDotGroups, /BG: Use of aria-label is breaking feedback for screen reader users as dots are incorrectly represented.

I would suggest we completely remove it instead of commenting it out. I would definitely love to see this integrated in 1.9.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants