Skip to content

fix(a11y): hide decorative SVGs from assistive tech#84

Open
adityakulraj wants to merge 1 commit intocomnam90:developfrom
adityakulraj:feature/adkunwar-hide-decorative-svgs
Open

fix(a11y): hide decorative SVGs from assistive tech#84
adityakulraj wants to merge 1 commit intocomnam90:developfrom
adityakulraj:feature/adkunwar-hide-decorative-svgs

Conversation

@adityakulraj
Copy link

Summary

  • add aria-hidden="true" to decorative inline SVGs in layouts/index.html
  • cover decorative icons in the header, controls, dialogs, quick links, report links, and generated service icon templates
  • keep the existing button labels and nearby text as the accessible names

Testing

  • git diff --check
  • verified there are no remaining <svg tags in layouts/index.html without aria-hidden

Closes #78

@adityakulraj
Copy link
Author

@comnam90 would appreciate your review on this one too when you have time. Thanks!

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR improves the accessibility of the Hugo-based single-page map UI (layouts/index.html) by marking decorative inline SVG icons as hidden from assistive technologies, reducing screen reader noise while preserving existing accessible names via surrounding text and aria-labels.

Changes:

  • Adds aria-hidden="true" to decorative inline SVGs across header controls, dialogs/panels, quick links, and report links.
  • Adds aria-hidden="true" to SVGs returned by getServiceIcon() (including the fallback icon template).

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.

a11y: add aria-hidden to decorative SVGs throughout index.html

2 participants