CBHSQ / findtreatment

FindTreatment.gov, An 18F+SAMHSA Collaboration
https://findtreatment.gov/
Other
19 stars 12 forks source link

Run through Accessibility checklist #184

Closed Jkrzy closed 5 years ago

Jkrzy commented 5 years ago

Let's run through the 18F accessibility checklist

We'll make new issues for identified problems to track their remediation.


This checklist helps developers identify potential accessibility issues affecting their websites or applications. It's broken down into three sections of decreasing importance: A, B and C. Please check and address these issues in the order in which they appear.

For more detail on accessibility standards, please see WCAG2.0 AA

It is important to note, while B and C are noted as less critical, they are still required to be truly 508 compliant. This checklist should be used as a reference for development and is not a substitute for compliance checks by a section 508 coordinator.

A - Critical

B - Less Critical

C - Minor

Jkrzy commented 5 years ago

Ran through https://accessibility.18f.gov/keyboard/ and was able to successfully navigate to all areas and functionality of the site.

Jkrzy commented 5 years ago

We have limited use of tags on the site, all of which have associated alt attributes. We have no multimedia.

The SAMHSA logo SVG does not have a title attribute, may need to add one.

Jkrzy commented 5 years ago

We have no tables, flashing elements, or timeouts.

Jkrzy commented 5 years ago

lang is set in index.html

Jkrzy commented 5 years ago

We're using a single page title across all views rendered by React. We should remediate and provide unique and descriptive titles for each page. https://accessibility.18f.gov/page-titles/

TODO: #185

Jkrzy commented 5 years ago

The iframe in which google maps is rendered on the /results page does not have a title attribute. Can this be resolved on our end? https://accessibility.18f.gov/iframes/. The is also selectable via keyboard navigation but focus is hidden when selected.

Jkrzy commented 5 years ago

A few text color/contrast issues: https://accessibility.18f.gov/color/

Grey text on white background is 4:1 contrast ratio. Results page information is 1.5:1

Screen Shot 2019-08-05 at 4 45 32 PM Screen Shot 2019-08-05 at 4 45 36 PM Screen Shot 2019-08-05 at 4 48 36 PM
Jkrzy commented 5 years ago

Users should be able to tab once to select and move-past button links.

Most of the

Jkrzy commented 5 years ago

Check and remediate headings for each page.

The landing page has no <h1> and the footer has an out-of-order <h3> tag.

https://accessibility.18f.gov/headings/

Jkrzy commented 5 years ago

Improve site functionality when no CSS is present

When no CSS is present, keyboard navigation is broken. More than a few anchor tags are unreachable throughout the site and facility cards are unreachable on the results list page.

Jkrzy commented 5 years ago

For links which are not plainly and clearly described by their text, we should provide title or ARIA attributes.

The destination/functionality of links such as those on the site logo and content icons is unclear without the surrounding visual context.

https://accessibility.18f.gov/links/

Jkrzy commented 5 years ago

Superseded by new visual design work and accessibility assessments