Skip to content

fix: add proper aria-live region for dynamic content updates - #112

Open
Douglas4tech wants to merge 4 commits into
int-money:mainfrom
Douglas4tech:arialive
Open

fix: add proper aria-live region for dynamic content updates#112
Douglas4tech wants to merge 4 commits into
int-money:mainfrom
Douglas4tech:arialive

Conversation

@Douglas4tech

Copy link
Copy Markdown

Summary

Closed: #82

I have fixed the accessibility issue in the waitlist form by adding a proper aria-live region to ensure screen reader users are notified of success and error states.

Type of Change

  • [ x] Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Chore or Refactoring

Checklist

  • [ x] I have read the CONTRIBUTING.md guidelines.
  • [x ] My commits follow the Conventional Commits specification.
  • [x ] My commits are atomic and represent singular logical changes.
  • [ x] I have self-reviewed my own code.
  • [ x] I have tested my changes locally and verified they pass linting and builds.
  • I have attached a screen recording (mandatory for UI changes).

Screen Recording / Screenshots

@drips-wave

drips-wave Bot commented Mar 28, 2026

Copy link
Copy Markdown

@Douglas4tech Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@Douglas4tech

Copy link
Copy Markdown
Author

@pragmaticAweds pls merge

@pragmaticAweds

pragmaticAweds commented Mar 29, 2026

Copy link
Copy Markdown
Contributor

Please attache a screen recording (mandatory for UI to avoid breaking)

Also, the CI is not passing

Do this for approval.

The aria-live region state was lost during the merge. Restored
the useState declaration for ariaMessage.
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.

fix: add proper aria-live region for dynamic content updates

3 participants