Skip to content

chore: add a11y step to ci for wcag aa guidelines#54

Merged
SethFalco merged 1 commit into
svg:mainfrom
SethFalco:pa11y
Feb 26, 2026
Merged

chore: add a11y step to ci for wcag aa guidelines#54
SethFalco merged 1 commit into
svg:mainfrom
SethFalco:pa11y

Conversation

@SethFalco
Copy link
Copy Markdown
Member

@SethFalco SethFalco commented Feb 26, 2026

Adds a new step to CI to enforce accessibility guidelines.

For now, I'm only going for AA, and I am ignoring the following errors:

  • Prism code blocks.
  • Something that I'm convinced is an error on the theme rather than our styles.

Changes

  • When optimizing an SVG in the playground, I changed the shade of green so that it's actually legible on light mode. Turns out before it was hard to read.
  • Fixed the keyboard shortcut colors in light mode as well. It was very difficult to read.

(Probably obvious that I'm not a big light mode fan.)

Chores

I also did some other chores just to align the project with conventions I apply across my other projects.

  • Renamed main.yml to ci.yml.
  • Omitted names from actions/jobs, I prefer to just see the file name/ID.
  • Updated actions and Node.js version.

@SethFalco SethFalco force-pushed the pa11y branch 2 times, most recently from 2bbad44 to f7fd4b9 Compare February 26, 2026 02:16
@SethFalco SethFalco merged commit 6013a0b into svg:main Feb 26, 2026
4 checks passed
@SethFalco SethFalco deleted the pa11y branch February 26, 2026 02:20
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

Successfully merging this pull request may close these issues.

1 participant