Skip to content

Add a11y testing#585

Draft
TylerZeroMaster wants to merge 10 commits into
mainfrom
accessibility-audit
Draft

Add a11y testing#585
TylerZeroMaster wants to merge 10 commits into
mainfrom
accessibility-audit

Conversation

@TylerZeroMaster
Copy link
Copy Markdown
Contributor

No description provided.

.github/workflows/a11y.yml
  - Renamed job from matches to a11y_audit (clearer naming)
  - Simplified command to use new wrapper script ./script/test-a11y

  script/audit-a11y.js
  - Code formatting improvements (arrays split across lines for readability)
  - Renamed handleCssFile → buildTestBench (more descriptive)
  - Enhanced console output to include style name in success/failure messages
  - Added final message directing users to GitHub Actions summary tab
  - No functional changes, just cleaner code

  script/test-a11y (new file)
  - New bash wrapper that encapsulates the find command logic
  - Tests both *-pdf.css and webview-generic.css files
  - Uses bash best practices (set -euo pipefail)
  - Makes running accessibility tests simpler
Hopefully
Enables alphabetical sorting and better control over file processing order.
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