COVID19Tracking / website

The COVID Tracking Project website
https://covidtracking.com/
Apache License 2.0
499 stars 189 forks source link

Adjustments for accessibility (based on Voiceover) #790

Closed tealtan closed 4 years ago

tealtan commented 4 years ago

Running this through Voiceover I encountered these issues.

(We’ll want to check NVDA/JAWS separately, after fixing these issues first)

tealtan commented 4 years ago

Hero

Latest totals

CTA

Press List

Press Logos

Alt text for images

Blog

sarawb commented 4 years ago

Content for those Alt text items

CRDT Illustration of a face split into strips of different colored skintones on a light purple background.

JH Chart from the Johns Hopkins COVID-19 Testing Insights Initiative depicting daily total tests and daily positive tests using COVID Tracking Project data.

WH Image composite showing COVID Tracking Project data in use on a slide from the White House’s “Opening Up America Again” testing strategy presentation deck.

CES Screenshot of a COVID Exit Strategy chart that uses COVID Tracking Project data in its state-by-state comparison of interventions, testing, and outcomes.

kevee commented 4 years ago

@tealtan I used aria-hidden for the chart.

kevee commented 4 years ago

For the role="text", JSX was throwing an error that this constituted an "abstract" role. I can't find "text" in the list of ARIA roles. I made JSX skip these, but FYI.

kevee commented 4 years ago

I think we fixed all these issues, but a pass with NVDA/JAWS would be great.

tealtan commented 4 years ago

Awesome, thanks!

That role error is odd, but I think it’s fine for now.