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

PLANET-5211 Home page Accessibility: Navigation bar and "Skip to..." links. #88

Closed
planet-4 opened this issue Sep 21, 2020 · 1 comment
Assignees
Labels
Dev development tickets Feature Frontend tickets that mostly require js/css knowledge Moderate

Comments

@planet-4
Copy link

Maks sure navigation flow sounds good on a screen reader.

Tasks:

  • Hide "Skip to" links to the SR using aria-hidden.
  • Search: Both the input and the search icon are read as "Search". Make the input more descriptive using aria-label or hide tha search icon to the SR or add a label as "Press return/enter to search" or "click to search".
  • Countries list, make sure it works well, is browsable, and it has proper aria tags to deal with the transition (the big menu appearing).
  • Make sure new text is translatable.

 

Reporter: pcuadrad
Sections: Navigation

Potentially affected repositories: planet4-master-theme

@planet-4 planet-4 added A11y Dev development tickets Feature Frontend tickets that mostly require js/css knowledge Moderate labels Sep 21, 2020
@dpivo dpivo self-assigned this Oct 22, 2020
@comzeradd
Copy link
Member

Fixed at greenpeace/planet4-master-theme#1214

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Dev development tickets Feature Frontend tickets that mostly require js/css knowledge Moderate
Projects
None yet
Development

No branches or pull requests

3 participants