Skip to content

Feat/aria live sync announcement#325

Open
mrunalC27 wants to merge 2 commits into
Priyanshu-byte-coder:mainfrom
mrunalC27:feat/aria-live-sync-announcement
Open

Feat/aria live sync announcement#325
mrunalC27 wants to merge 2 commits into
Priyanshu-byte-coder:mainfrom
mrunalC27:feat/aria-live-sync-announcement

Conversation

@mrunalC27
Copy link
Copy Markdown

@mrunalC27 mrunalC27 commented May 18, 2026

Summary

Added an accessible aria-live="polite" region in DashboardHeader to announce sync status updates for screen reader users after metrics refresh actions.

Closes #216

Type of Change

  • Bug fix
  • New feature
  • Documentation update
  • Refactor / code cleanup

Changes Made

  • Added a visually hidden live region using aria-live="polite" and role="status" in DashboardHeader
  • Updated announcement text after sync success and failure responses
  • Added timeout cleanup logic to clear announcements after 3 seconds
  • Prevented stale timeout issues on component unmount

How to Test

Steps for the reviewer to verify this works:

  1. Open the dashboard page

  2. Trigger the Sync button

  3. Verify that screen readers announce:

    • "Metrics refreshed successfully" on success
    • "Sync failed. Please try again." on failure
  4. Confirm the announcement clears automatically after 3 seconds

Screenshots (after UI change)

Screenshot 2026-05-18 194032

Checklist

  • Linked issue in summary
  • npm run lint passes locally
  • No TypeScript errors (npm run type-check)
  • Self-reviewed the diff
  • Added/updated tests if applicable

@vercel
Copy link
Copy Markdown

vercel Bot commented May 18, 2026

@mrunalC27 is attempting to deploy a commit to the PRIYANSHU DOSHI's projects Team on Vercel.

A member of the Team first needs to authorize it.

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.

[GOOD FIRST ISSUE] Add aria-live region for sync completion in DashboardHeader

1 participant