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 updates #151

Open
GingerKiwi opened this issue Dec 27, 2023 · 3 comments
Open

A11y updates #151

GingerKiwi opened this issue Dec 27, 2023 · 3 comments

Comments

@GingerKiwi
Copy link

GingerKiwi commented Dec 27, 2023

components/footer /index.coffee

Line 37
[] Change

<a href="https://opencollective.com/torontojs" aria-label="OpenCollective">

to

<a href="https://opencollective.com/torontojs" aria-label="Open Collective">

Reason: Elements with visible text labels do not have matching accessible names

Main Page

[] 1. Change:

<img src="https://torontojs.com/guild_logo.a39081a5.png">

to:

<img src="https://torontojs.com/guild_logo.a39081a5.png" alt="logo for Guild.host">

  • Reason: alt text missing

[] 2. Change

<a href="/torontojs" role="link" class="css-175oi2r">

to

<a href="/torontojs" role="link" aria-label="link to TorontoJS on guild.host" class="css-175oi2r">
  • Reason: link does not have a discernible name.

[] 3. Navbar: Change background colour to darker

  • #991b1b and #7f1d1d passes WCAG AA for small text.

  • @madcampos is working on the new branding guide and likely has a brand colour that works.

  • Footer would be more accessible in a darker background colour. Maybe the almost black of the brand colours?

  • Reason: current colour does not have sufficient contrast.


Note: I haven't run manual testing with NVDA and JAWS yet. These are just some minor updates that make a major A11y impact

@kieran
Copy link
Collaborator

kieran commented Dec 27, 2023

did the first 2 - will wait for @madcampos' branding to change any colours

@madcampos
Copy link
Contributor

@GingerKiwi @kieran the Navbar color issue refers to the gray footer right?

image

@GingerKiwi
Copy link
Author

The Navbar issues refers to both the header and the footer navs.

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

No branches or pull requests

3 participants