Skip to content
This repository has been archived by the owner on Dec 23, 2017. It is now read-only.

Do an accessibility audit #415

Closed
noahmanger opened this issue Aug 4, 2015 · 3 comments
Closed

Do an accessibility audit #415

noahmanger opened this issue Aug 4, 2015 · 3 comments
Assignees

Comments

@noahmanger
Copy link
Contributor

We've been adding so much stuff, but the major components are more or less in place. We should do an audit of the accessibility of openFEC.

@noahmanger
Copy link
Contributor Author

I made a bunch of changes as noted here #723

Remaining task is to have Nick go through and double check my work.

@noahmanger
Copy link
Contributor Author

Remaining questions:

  • Do we need aria-controls or aria-expanded on the panel / rows ? - "it would be fine if the label was expand total receipts, expand total disbursements"
  • Do we need aria-controls to accordion buttons and toggles?
  • Where should the "Skip nav" button take the user? There's some disagreement on this.

@noahmanger
Copy link
Contributor Author

Added aria-controls to the toggles.

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

No branches or pull requests

2 participants