Skip to content

Fix landmark violation: Use aside elements for presentation controls#25

Draft
Copilot wants to merge 2 commits intomainfrom
copilot/fix-accessibility-landmarks-again
Draft

Fix landmark violation: Use aside elements for presentation controls#25
Copilot wants to merge 2 commits intomainfrom
copilot/fix-accessibility-landmarks-again

Conversation

Copy link

Copilot AI commented Feb 13, 2026

Axe flagged <div class="clock"> on /presentations/EAA-drupal4gov-eu26.html because all page content must be contained by landmarks per WCAG 2.1 (rule: region).

Changes

  • presentations/EAA-drupal4gov-eu26.html: Changed presentation controls from <div> to <aside> elements with aria-label attributes
  • presentations/AGENTS.md: Updated documentation to reflect correct pattern
<!-- Before -->
<div class="progress"></div>
<div class="clock"></div>

<!-- After -->
<aside class="progress" aria-label="Presentation progress"></aside>
<aside class="clock" aria-label="Presentation timer"></aside>

The <aside> element has implicit ARIA role complementary (a landmark), ensuring content is properly contained. CSS and JavaScript remain unaffected—both use class selectors.

Visual Confirmation

Presentation with fixed landmarks

Clock and progress controls render correctly with no functional regressions.

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: All page content should be contained by landmarks on /presentations/EAA-drupal4gov-eu26.html</issue_title>
<issue_description>## What
An accessibility scan flagged the element <div class="clock" aria-label="clock" style="--time-factor: 0.0008341666666666914;"> on https://ox.ca/presentations/EAA-drupal4gov-eu26.html because all page content should be contained by landmarks. Learn more about why this was flagged by visiting https://dequeuniversity.com/rules/axe/4.11/region?application=playwright.

To fix this, ensure all page content is contained by landmarks.

Specifically:

Fix any of the following:

  • Some page content is not contained by landmarks

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)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: mgifford <116832+mgifford@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix accessibility issue by adding landmarks to page content Fix landmark violation: Use aside elements for presentation controls Feb 13, 2026
Copilot AI requested a review from mgifford February 13, 2026 13:54
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.

Accessibility issue: All page content should be contained by landmarks on /presentations/EAA-drupal4gov-eu26.html

2 participants