Skip to content

fix(docs): improve Docusaurus accessibility evidence - #1642

Open
Bill Berry (WilliamBerryiii) wants to merge 9 commits into
accessibility/viewer-interactionsfrom
accessibility/docusaurus
Open

Bill Berry (WilliamBerryiii) wants to merge 9 commits into
accessibility/viewer-interactionsfrom
accessibility/docusaurus

Conversation

@WilliamBerryiii

Copy link
Copy Markdown
Member

Pull Request

Description

Improve Docusaurus accessibility behavior and evidence quality. The branch repairs mobile navigation focus, search semantics, compact targets, adaptive rendering, and documentation structure; adds complete route, contrast, keyboard, Mermaid, and screen-reader evidence; and keeps unresolved contrast review separate from semantic method verdicts.

Closes #1637

Type of Change

  • 🐛 Bug fix (non-breaking change fixing an issue)
  • ✨ New feature (non-breaking change adding functionality)
  • 💥 Breaking change (fix or feature causing existing functionality to change)
  • 📚 Documentation update
  • 🏗️ Infrastructure change (Terraform/IaC)
  • ♻️ Refactoring (no functional changes)

Component(s) Affected

  • infrastructure/terraform/prerequisites/ - Azure subscription setup
  • infrastructure/terraform/ - Terraform infrastructure
  • infrastructure/setup/ - OSMO control plane / Helm
  • workflows/ - Training and evaluation workflows
  • training/ - Training pipelines and scripts
  • docs/ - Documentation

Testing Performed

  • Terraform plan reviewed (no unexpected changes)
  • Terraform apply tested in dev environment
  • Training scripts tested locally with Isaac Sim
  • OSMO workflow submitted successfully
  • Smoke tests passed (smoke_test_azure.py)

Targeted local validation:

  • npm --prefix docs/docusaurus test: typecheck, 68 Jest tests, and six Mermaid contract tests passed.
  • npm --prefix docs/docusaurus run lint:a11y: passed.
  • Canonical Chrome evidence on the integrated candidate: 159 applicable tests passed, 122 exact method cells passed, and all 175 route states completed with zero route errors; the separate 106-signature contrast gate remained fail-closed.
  • Pinned HVE closure: five cases, six execution journeys, seven method cells, and zero uncovered cells.
  • git diff --check origin/main...accessibility/docusaurus: passed after the two Docusaurus-only follow-up commits.

Documentation Impact

  • No documentation changes needed
  • Documentation updated in this PR
  • Documentation issue filed

Bug Fix Checklist

Complete this section for bug fix PRs. Skip for other contribution types.

  • Linked to issue being fixed
  • Regression test included, OR
  • Justification for no regression test:

Checklist

- repair navigation, search, breadcrumbs, tables, and adaptive layouts
- add governed labels and accessible diagram metadata
- update Docusaurus dependencies and configuration

♿ - Generated by Copilot
- add route, method, adaptive, contrast, and structure coverage
- share frontmatter and route discovery contracts
- retain exact reviewer evidence identity

♿ - Generated by Copilot
- keep semantic method cells independent from qualified contrast review
- preserve fail-closed contrast gating

♿ - Generated by Copilot
- add the pinned Docusaurus runtime profile
- correct native NVDA execution recipes

♿ - Generated by Copilot
# Conflicts:
#	docs/contributing/accessibility.md
#	docs/data-pipeline/chunking-compression-config.md
#	docs/docusaurus/package-lock.json
#	docs/docusaurus/package.json
#	docs/osmo-proxy.md
@github-actions

github-actions Bot commented Sep 22, 2026

Copy link
Copy Markdown
Contributor

Dependency Review

The following issues were found:

  • ✅ 0 vulnerable package(s)
  • ✅ 0 package(s) with incompatible licenses
  • ✅ 0 package(s) with invalid SPDX license definitions
  • ✅ 0 package(s) with unknown licenses.
  • ⚠️ 64 packages with OpenSSF Scorecard issues.

View full job summary

@codecov-commenter

Codecov Comments Bot (codecov-commenter) commented Sep 22, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.83%. Comparing base (241ff07) to head (bf09b82).

Additional details and impacted files
@@                        Coverage Diff                         @@
##           accessibility/viewer-interactions    #1642   +/-   ##
==================================================================
  Coverage                              87.83%   87.83%           
==================================================================
  Files                                    281      281           
  Lines                                  23508    23508           
  Branches                                3188     3188           
==================================================================
  Hits                                   20648    20648           
  Misses                                  2149     2149           
  Partials                                 711      711           
Flag Coverage Δ *Carryforward flag
go 100.00% <ø> (ø)
pester 86.75% <ø> (ø) Carriedforward from 2b651f7
pytest-data-pipeline 100.00% <ø> (ø) Carriedforward from 2b651f7
pytest-dataviewer 88.79% <ø> (ø)
pytest-dm-tools 100.00% <ø> (ø) Carriedforward from 2b651f7
pytest-evaluation 95.40% <ø> (ø)
pytest-fuzz 3.76% <ø> (ø)
pytest-inference 100.00% <ø> (ø) Carriedforward from 2b651f7
pytest-shared-ci 100.00% <ø> (ø) Carriedforward from 2b651f7
pytest-training 92.60% <ø> (ø) Carriedforward from 2b651f7
vitest 85.27% <ø> (ø) Carriedforward from 2b651f7
vitest-app 85.27% <ø> (ø) Carriedforward from 2b651f7
vitest-components 85.27% <ø> (ø) Carriedforward from 2b651f7
vitest-features 85.27% <ø> (ø) Carriedforward from 2b651f7
vitest-lib 85.27% <ø> (ø) Carriedforward from 2b651f7
vitest-state 85.27% <ø> (ø) Carriedforward from 2b651f7

*This pull request uses carry forward flags. Click here to find out more.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Comment thread docs/docusaurus/e2e/static-server.mjs Fixed
} from '../e2e/route-inventory'

const siteRoot = path.resolve(__dirname, '..')
const repositoryRoot = path.resolve(siteRoot, '..', '..')
@WilliamBerryiii
Bill Berry (WilliamBerryiii) changed the base branch from main to accessibility/viewer-interactions September 22, 2026 03:47
- move breadcrumb spacing into the existing global stylesheet
- remove the standalone headerless CSS module

♿ - Generated by Copilot
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.

Improve Docusaurus accessibility and evidence

3 participants