Skip to content

feat(a11y): add ARIA labels to navigation and polite announcements to…#20

Merged
rajshekharbind merged 2 commits intoOPCODE-Open-Spring-Fest:mainfrom
ayushhh101:feat/a11y-nav-and-notifications
Oct 16, 2025
Merged

feat(a11y): add ARIA labels to navigation and polite announcements to…#20
rajshekharbind merged 2 commits intoOPCODE-Open-Spring-Fest:mainfrom
ayushhh101:feat/a11y-nav-and-notifications

Conversation

@ayushhh101
Copy link
Contributor

… notifications

Description

  • Add aria-label="Open menu" to the navigation hamburger button.
  • Add role="navigation" and aria-label="Primary" to the sidebar .
  • Add aria-live="polite" to the notifications list wrapper for screen-reader announcements.
  • No visual changes; improves keyboard/SR accessibility and landmark semantics.

Changed files:

  1. src/components/Navigation.jsx
  2. src/components/NotificationList.jsx

Semver Changes

  • Patch (bug fix, no new features)
  • Minor (new features, no breaking changes)
  • Major (breaking changes)

Issues

Closes #19

Checklist

@ayushhh101
Copy link
Contributor Author

Hi @rajshekharbind — the label checker is blocking this PR. Could you please add the required labels?
Type: Easy
Semver: Minor
PR:Accept
After labels are applied, the workflow should pass (or I can re-run the checks). Thanks!

@rajshekharbind rajshekharbind merged commit 6f6eab4 into OPCODE-Open-Spring-Fest:main Oct 16, 2025
3 of 5 checks passed
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.

Enhance accessibility: ARIA for navigation and notifications

2 participants