Kajabi / sage-lib

Sage Design System and Documentation
https://sage.kajabi.com
MIT License
20 stars 2 forks source link

[a11y] Audit all components for accessibility #818

Open monicawheeler opened 3 years ago

monicawheeler commented 3 years ago

Description

All components should be accessible. This issue is to review each component for accessibility.

Success criteria

TODO

Verify that all components are accessible. Document here.

If accessible, check the box. If not accessible, create a new issue that calls out that component and any details you think might be helpful, link it here, and check the box.

philschanely commented 2 years ago

@monicawheeler marking this with the EPIC tag as it may benefit from a dedicated milestone and place for tracking follow up tasks.

QuintonJason commented 2 years ago

@monicawheeler I created this Jira ticket SAGE-124

sdhull commented 2 years ago

While working with @dhstewart today, trying to write feature specs with capybara, we discovered a couple of accessibility issues with the searchable Dropdown component.

First, the button you click on to open it initially has no text or label. This makes it impossible to target with the (a11y-friendly) API of capybara (eg, click_on, which tries to find buttons & links by their text). The text is in an adjacent span.

image

Second (extremely minor), the search field had a label (can't find it in the docs sites) that had text but wasn't properly hooked up to the input with for/id. It did have targetable placeholder however, so that was good 😊