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

[a11y] difficult to differentiate between focused and invalid states #851

Open
CITguy opened this issue Mar 16, 2021 · 1 comment
Open

Comments

@CITguy
Copy link
Contributor

CITguy commented Mar 16, 2021

Describe the Bug

Form inputs are inconsistently styled and have accessibility issues with differentiating between valid/invalid states.

Steps to Reproduce

Steps to reproduce the behavior:

  1. Install "Toggle Grayscale" Chrome extension (install link at CITguy/toggle-grayscale)
  2. Navigate to https://helixdesignsystem.github.io/helix-ui/components/text-input/
  3. Click "Required" checkbox under Control Options
  4. Click within Text Input in the example
  5. note the appearance of the input
  6. Click away from Text Input (leaving input blank/invalid)
  7. note the appearance of the input
  8. toggle the Toggle Grayscale extension ON (converting page to grayscale)
  9. repeat steps 2-7

Expected behavior

  1. Focus shadow should be consistent between Focus (valid) and Focus (invalid)
  2. Focus (invalid) should be distinguishable when compared to Focus (valid) when Toggle Grayscale is turned ON

Screenshots

Mode Focused (valid) Focused (invalid)
Color CleanShot 2021-03-16 at 10 25 11@2x CleanShot 2021-03-16 at 10 25 18@2x
Grayscale CleanShot 2021-03-16 at 10 25 43@2x CleanShot 2021-03-16 at 10 25 50@2x
  • "valid" + focus does not have shadow, while "invalid" + focus does have shadow
  • "valid" vs "invalid" underline color cannot be easily differentiated by users with color vision deficiencies

Environment

Please complete the following information:

  • Device: any
  • OS: any
  • Browser:
    • Chrome (because its the only browser that the Toggle Grayscale extension works on), but the design is equally broken in all browsers

Additional Context

WCAG § 1.4.1 Use of Color - Level A

@100stacks
Copy link
Member

hey @CITguy thanks for the feedback. The new specifications are still in flux, so this may change.

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

2 participants