Skip to content

Add main landmark for WCAG 2.1 compliance#6

Merged
mgifford merged 4 commits intomainfrom
copilot/fix-accessibility-main-landmark
Feb 13, 2026
Merged

Add main landmark for WCAG 2.1 compliance#6
mgifford merged 4 commits intomainfrom
copilot/fix-accessibility-main-landmark

Conversation

Copy link
Contributor

Copilot AI commented Feb 13, 2026 •
edited
Loading

Document was missing a main landmark element, triggering axe rule violation landmark-one-main.

Changes

  • index.html: Wrapped slide content in <main> element. UI controls (progress bar, clock, status indicators) remain outside as they're not primary content.

  • scripts/check-main-landmark.sh: Added test to verify exactly one main landmark with matching tags. Integrated into test:all suite.

<body class="shower fade-in duration=30 warn=5 hidemouse">
  <div class="progress"></div>
  <div class="clock"></div>
  <section aria-live="assertive" aria-label="Slide mode status">...</section>

  <main>
    <!-- All slides here -->
  </main>
</body>

Visual verification

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • https://storage.googleapis.com/chromium-browser-snapshots/Linux_x64/869685/chrome-linux.zip
    • Triggering command: /home/REDACTED/work/_temp/ghcca-node/node/bin/node node install.js (http block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

This section details on the original issue you should resolve

<issue_title>Accessibility issue: Document should have one main landmark on /whisper-slides/</issue_title>
<issue_description>## What
An accessibility scan flagged the element <html lang="en-us"> on https://mgifford.github.io/whisper-slides/ because document should have one main landmark. Learn more about why this was flagged by visiting https://dequeuniversity.com/rules/axe/4.11/landmark-one-main?application=playwright.

To fix this, ensure the document has a main landmark.

Specifically:

Fix all of the following:

  • Document does not have a main landmark

Acceptance Criteria

  • The specific axe violation reported in this issue is no longer reproducible.
  • The fix MUST meet WCAG 2.1 guidelines OR the accessibility standards specified by the repository or organization.
  • A test SHOULD be added to ensure this specific axe violation does not regress.
  • This PR MUST NOT introduce any new accessibility issues or regressions.

</issue_description>

Comments on the Issue (you are @copilot in this section)

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI and others added 2 commits February 13, 2026 13:19
@mgifford
Add main landmark and test to fix accessibility issue
70aa6bc 
Co-authored-by: mgifford <116832+mgifford@users.noreply.github.com>
@mgifford
Improve main landmark test to check for matching tags
1e0f2e0 
Co-authored-by: mgifford <116832+mgifford@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix accessibility issue: Document should have one main landmark Add main landmark for WCAG 2.1 compliance Feb 13, 2026
Copilot AI requested a review from mgifford February 13, 2026 13:21
mgifford
mgifford approved these changes Feb 13, 2026
View reviewed changes
Copy link
Owner

@mgifford mgifford left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks fine. Not sure it's needed, but sure.

@mgifford mgifford marked this pull request as ready for review February 13, 2026 16:08
@mgifford mgifford merged commit 94f379a into main Feb 13, 2026
1 check passed
@mgifford mgifford deleted the copilot/fix-accessibility-main-landmark branch February 13, 2026 16:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@mgifford mgifford mgifford approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@mgifford mgifford

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Accessibility issue: Document should have one main landmark on /whisper-slides/

2 participants

@mgifford