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

Conditionally revealing content with checkboxes not announcing collapsed/expanded state on macOS with VoiceOver #989

Closed
NickColley opened this issue Sep 7, 2018 · 3 comments

Comments

@NickColley
Copy link
Contributor

When using the conditionally revealing content component VoiceOver on macOS in Safari doesn't announce that it is collapsed or expanded, despite the use of aria-expanded and others. All other screen readers are fine with it. Weirdly, that only happens when using checkboxes, not radio buttons!

How would you expect it to work?

When on the trigger ("Citizen of a different country"), VoiceOver should say "collapsed" (like it does when using the same component with radio buttons). When triggering it, it should say "expanded".

How does it work currently?

It doesn't say whether it's collapsed or expanded.

Originally raised by @selfthinker on GOV.UK Elements: alphagov/govuk_elements#575

This is still an issue in GOV.UK Frontend's implementation and was noted as part of the assistive technology testing: #616

My worry with this is that we are adding aria attributes to elements that by the spec should not have them, so I'm not sure if this will be easy to fix or if browser vendors would even consider it a bug.

We should do a proper investigation and make sure we do our best to try to resolve this.

@36degrees
Copy link
Contributor

Is this a duplicate of #935?

@kellylee-gds kellylee-gds added this to Upcoming sprints in Design System Sprint Board via automation Sep 5, 2019
@kellylee-gds kellylee-gds moved this from Upcoming sprints to Sprint Backlog in Design System Sprint Board Sep 5, 2019
@NickColley NickColley moved this from Sprint Backlog to Needs review in Design System Sprint Board Sep 9, 2019
@NickColley NickColley removed this from Needs review in Design System Sprint Board Sep 11, 2019
@hannalaakso hannalaakso added the 🐛 bug Something isn't working the way it should (including incorrect wording in documentation) label Sep 24, 2019
@NickColley NickColley removed the 🐛 bug Something isn't working the way it should (including incorrect wording in documentation) label Dec 11, 2019
@NickColley
Copy link
Contributor Author

@36degrees we could consider consolidating them but they are different in that one is the JAWs behaviour one is the VoiceOver behaviour.

See #979 (comment) for investigation into aria attributes on conditional reveals.

@36degrees
Copy link
Contributor

Consolidating into #1972.

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

No branches or pull requests

4 participants