Skip to content

Add main landmark to fix WCAG 2.1 landmark-one-main violation#19

Draft
Copilot wants to merge 3 commits intomainfrom
copilot/fix-accessibility-main-landmark
Draft

Add main landmark to fix WCAG 2.1 landmark-one-main violation#19
Copilot wants to merge 3 commits intomainfrom
copilot/fix-accessibility-main-landmark

Conversation

Copy link

Copilot AI commented Feb 13, 2026 •
edited
Loading

The root page was missing a main landmark, failing axe rule landmark-one-main (WCAG 2.1 Level A).

Changes

  • _layouts/default.html: Replace <section> with <main> for primary content wrapper
  • _layouts/home.html: Change <section class="home"> to <div class="home"> to avoid nested sectioning elements
  • assets/css/style.scss: Apply margin-bottom: 2rem to both section and main to preserve styling

Before/After

- <section>
+ <main>
    {{ content }}
- </section>
+ </main>

Page structure is now: <header> → <main> → <footer> with proper ARIA landmarks for assistive technology navigation.

Screenshot

No visual changes:

Page rendering after fix

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 -I y-3.2.0 -I _64-linux-gnu/ru-I -I ndor/bin/as -I kward -I as -I /x86_64-linux-gn-I /libexec/gcc/x86/usr/include/ruby-3.2.0/ruby/backward by-3.2.0 s-through=-lgcc kward s-through=-lc s-through=-lgcc (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 /</issue_title>
<issue_description>## What
An accessibility scan flagged the element <html lang="en-US" data-theme="light"> on https://ox.ca 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)

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 2 commits February 13, 2026 13:43
@mgifford
Add main landmark to fix accessibility issue
fa11a75 
Co-authored-by: mgifford <116832+mgifford@users.noreply.github.com>
@mgifford
Complete accessibility fix with main landmark
87fdfb9 
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 to fix WCAG 2.1 landmark-one-main violation Feb 13, 2026
Copilot AI requested a review from mgifford February 13, 2026 13:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mgifford mgifford Awaiting requested review from mgifford

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 /

2 participants

@mgifford