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

Fix a bunch of accessibility issues! #53

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

emma-sg
Copy link

@emma-sg emma-sg commented Mar 24, 2018

  • Links now pass AAA WCAG for colour contrast! and they have underlines
    where they didn't before
  • Thin (maybe too thin) custom scrollbars are gone, letting the user's
    OS determine what's best (and not violating their scrollbar
    expectations either)
  • Added a title attribute to the twitter link down at the bottom, so
    assistive devices know what's goin' on
  • Put the textarea in the sign up form inside its label, like the rest
    of the form elements, so that it's clear what the label labels

- Links now pass AAA WCAG for colour contrast! and they have underlines
 where they didn't before
- Thin (maybe too thin) custom scrollbars are gone, letting the user's
 OS determine what's best (and not violating their scrollbar
 expectations either)
- Added a title attribute to the twitter link down at the bottom, so
 assistive devices know what's goin' on
- Put the textarea in the sign up form inside its label, like the rest
 of the form elements, so that it's clear what the label labels
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant