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

USWDS-Site - Bug: UnLighthouse.dev ARIA #2148

Open
2 tasks done
mgifford opened this issue Jun 22, 2023 · 4 comments
Open
2 tasks done

USWDS-Site - Bug: UnLighthouse.dev ARIA #2148

mgifford opened this issue Jun 22, 2023 · 4 comments
Labels
Affects: Accessibility 🟡 Relates to the accessibility of our components Type: Bug A problem in the code

Comments

@mgifford
Copy link
Contributor

Describe the bug

I ran https://unlighthouse.dev/ on the https://designsystem.digital.gov/ site and discovered a few accessibility problems. As I said 2147 , it is really quite good, seems that there are a few here:

Screenshot of ARIA issues

The troublesome domains seem to be under /together /patterns & /next

Steps to reproduce the bug

From the command line run:
% npx unlighthouse --site https://designsystem.digital.gov/

and sort the results by accessibility & ARIA.

Expected Behavior

There shouldn't be any ARIA errors.

Screenshots

No response

System setup

No response

Additional context

No response

Code of Conduct

@mgifford mgifford added Status: Triage We're triaging this issue and grooming if necessary Type: Bug A problem in the code labels Jun 22, 2023
@mejiaj mejiaj added Needs: Confirmation We need to confirm that this is an issue Affects: Accessibility 🟡 Relates to the accessibility of our components Needs: Refinement We need to give this issue more detail and removed Status: Triage We're triaging this issue and grooming if necessary labels Jun 22, 2023
@amyleadem amyleadem removed the Needs: Confirmation We need to confirm that this is an issue label Jul 28, 2023
@amyleadem
Copy link
Contributor

amyleadem commented Jul 28, 2023

Confirming that unlighthouse does flag accessibility errors on 12 of our pages. The tool scanned 156 pages. The lowest accessibility score it flagged was 92. The overall accessibility score for the site was 100.

image

These are all of the pages where an accessibility score of less than 100 was reported. The errors seem confined to /next, /together, and /patterns:

@amyleadem amyleadem removed the Needs: Refinement We need to give this issue more detail label Jul 28, 2023
@amyleadem
Copy link
Contributor

amyleadem commented Jul 28, 2023

This issue includes the color contrast errors found in #2147

@mejiaj
Copy link
Contributor

mejiaj commented Jul 31, 2023

@amyleadem @mgifford thanks for looking into this.

It's hard to tell what the scope of issues and possible fixes are. What are the most glaring errors and what types of fixes are we looking at? We should break this issue up if level of effort is high.

@amyleadem
Copy link
Contributor

@mejiaj @mgifford I have added some notes to my earlier comment with specifics on the flagged errors. It seems as there are only three errors shared across these pages. They all seem like fixes that require a relatively low level of effort.

  1. Color contrast issues on some /next pages (also flagged in USWDS-Site - Bug: UnLighthouse.dev Color Contrast Errors #2147)
  2. Invalid roles on citation notes
  3. Mismatched aria roles in component code samples on a couple patterns pages

@mejiaj mejiaj changed the title USWDS-Site - Bug: Bug: UnLighthouse.dev ARIA USWDS-Site - Bug: UnLighthouse.dev ARIA Jun 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Affects: Accessibility 🟡 Relates to the accessibility of our components Type: Bug A problem in the code
Projects
Status: Ready to Schedule
Development

No branches or pull requests

3 participants