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

SortSite flags removal of focus outline from disabled buttons #2127

Closed
36degrees opened this issue Jan 28, 2021 · 0 comments · Fixed by #2265
Closed

SortSite flags removal of focus outline from disabled buttons #2127

36degrees opened this issue Jan 28, 2021 · 0 comments · Fixed by #2265
Assignees
Labels
accessibility button 🕔 hours A well understood issue which we expect to take less than a day to resolve.
Milestone

Comments

@36degrees
Copy link
Contributor

Description of the issue

SortSite flags the way we remove the outline from the focus state of disabled button as a potential issue:

The CSS outline or border style on this element makes it difficult or impossible to see the dotted link focus outline.
Change the style to avoid obscuring the focus outline around focusable elements.
Using a border or outline style that obscures the focus ring causes problems for keyboard-only users in:

  • Chrome (obscures the focus indicator on links, buttons, dropdowns and range controls)
  • Firefox (obscures the focus indicator on links)
  • Internet Explorer (obscures the focus indicator on links, buttons and range controls)
  • Internet Explorer does not support the outline-offset: property, so using this does not reliably avoid focus outline overlap.

Line 375 WCAG 2.1 AA F78 Section 508 (2017) AA F78

375: .govuk-button--disabled:focus,.govuk-button[disabled="disabled"]:focus,.govuk-button[disabled]:focus{outline:none}

When the disabled attributed is used on a <button> the button should no longer be focusable, so I don't think this would be an issue.

However, if the govuk-button--disabled class is used on a link this could be problematic as the link might still be focusable but the focus indicator would not be visible.

@trang-erskine trang-erskine added 🕔 hours A well understood issue which we expect to take less than a day to resolve. and removed awaiting triage Needs triaging by team labels Feb 1, 2021
@36degrees 36degrees self-assigned this Jul 2, 2021
@36degrees 36degrees added this to the [NEXT] milestone Jul 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accessibility button 🕔 hours A well understood issue which we expect to take less than a day to resolve.
Projects
Development

Successfully merging a pull request may close this issue.

2 participants