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

Tell users not to use conditional reveals to reveal text (for example contextual help) #1362

Closed
3 tasks done
36degrees opened this issue Sep 24, 2020 · 5 comments · Fixed by #1573
Closed
3 tasks done

Comments

@36degrees
Copy link
Contributor

36degrees commented Sep 24, 2020

What

Tell users not to use conditional reveals to reveal text (for example contextual help) – they should only be used to reveal related form fields. Any other use would need to be thoroughly research and tested.

This guidance should be added to the 'Conditionally revealing content' section for both the checkboxes and radios components.

Why

As part of an audit of GOV.UK, the conditional reveals on https://www.gov.uk/email-signup?link=/world/switzerland were flagged as failing WCAG 4.1.2.

Screenshot 2020-09-24 at 09 26 56

The page uses the conditional reveals in an unusual way, as what’s being revealed is effectively contextual help (like a hint) that would help the user to make the right choice. Whereas the way we currently intend the component to be used (as per the examples on https://design-system.service.gov.uk/components/radios/#conditionally-revealing-content) is to reveal additional form controls that ‘follow on’ from the radios.

It fails because changes to the expanded ‘state’ of the radio buttons are not always available to assistive tech, and so users may be unaware that something else has changed on the page as a result of selecting a radio. This is true regardless of what type of content is being revealed.

However, when used to reveal form controls, we think that the user is very likely to discover the revealed control as they navigate forwards from the radio button or checkbox. When used to reveal text, the text is not focusable and so is unlikely to be discovered.

We therefore want to discourage users from using conditional reveals in this way, whilst we investigate solutions to the broader issue.

Who needs to know about this

Content Designer

Done when

  • Drafted update to guidance
  • 2i'd
  • Published
@edwardhorsford
Copy link
Contributor

I've used this pattern before successfully - though I don't recall specifically testing this with AT.

Would a similar issue occur if it revealed a form control that was prefaced with a heading / paragraph of text?

@joelanman
Copy link
Contributor

yep, the user wouldnt know about the text, just the form control because they move to it

@36degrees
Copy link
Contributor Author

I think it'll depend on how they're navigating through the page. I don't think it's true to say they definitely wouldn't know about the text, but it's a lot more likely that they'd miss it.

@jbuller
Copy link

jbuller commented Sep 24, 2020

I think your are referring being in forms mode as opposed to browse mode of a screenreader.
In forms mode the elements that are not (programmatically associated with) fields may not are not read out.

@fofr
Copy link
Contributor

fofr commented Oct 1, 2020

Could the revealed text be made focusable, in the same way the error summary is focusable?
Is it possible for the revealed text to use aria-describedby?

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