Skip to content

Conversation

Ovgodd
Copy link
Collaborator

@Ovgodd Ovgodd commented Sep 17, 2025

Purpose

Improve accessibility by explicitly defining landmark roles in the HTML structure.
This helps users of assistive technologies navigate more efficiently and enhances
support across a wide range of browsers and screen readers.

issue : 878

Proposal

  • Add role="banner" to the element
  • Add role="main" to the element
  • Add role="contentinfo" to the element

Although HTML5 elements are recognized by modern tools, adding explicit role attributes ensures better support for older browsers and screen readers. This follows W3C recommendations and improves consistency across environments.

@Ovgodd Ovgodd requested a review from AntoLC September 17, 2025 06:19
@Ovgodd Ovgodd self-assigned this Sep 17, 2025
@Ovgodd Ovgodd force-pushed the fix/878-structure-landmarks branch from 82bd3d4 to 2aa54f2 Compare September 17, 2025 06:21
@Ovgodd Ovgodd marked this pull request as ready for review September 17, 2025 06:22
landmark roles help assistive tech users navigate quickly across the page

Signed-off-by: Cyril <c.gromoff@gmail.com>
@Ovgodd Ovgodd force-pushed the fix/878-structure-landmarks branch from 2aa54f2 to 1e39d17 Compare September 17, 2025 06:24
Copy link

github-actions bot commented Sep 17, 2025

Size Change: +66 B (0%)

Total Size: 3.66 MB

Filename Size Change
apps/impress/out/_next/static/38cc04fc/_buildManifest.js 0 B -888 B (removed) 🏆
apps/impress/out/_next/static/7f4f2d75/_buildManifest.js 886 B +886 B (new file) 🆕

compressed-size-action

@Ovgodd Ovgodd merged commit 1e39d17 into main Sep 17, 2025
27 of 29 checks passed
@Ovgodd Ovgodd deleted the fix/878-structure-landmarks branch September 17, 2025 10:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants