Skip to content

fix: announce form validation errors to screen readers - #660

Merged
ritaifeoluwa merged 2 commits into
SmartDropLabs:mainfrom
DooseWayo:fix/form-error-live-region
Oct 18, 2026
Merged

ritaifeoluwa merged 2 commits into
SmartDropLabs:mainfrom
DooseWayo:fix/form-error-live-region

Conversation

@DooseWayo

Copy link
Copy Markdown

Inline validation errors were conditionally mounted plain text, so screen readers never announced them (a live region must already exist before its content changes). Add a FieldError component that keeps a polite aria-live region mounted and wire it into the unlock, deposit (farm list and pool detail) and alerts API key fields, linking each input to its error via aria-describedby and isInvalid.

Closes #631
Closes #628
Closes #630
Closes #629

Inline validation errors were conditionally mounted plain text, so screen
readers never announced them (a live region must already exist before its
content changes). Add a FieldError component that keeps a polite aria-live
region mounted and wire it into the unlock, deposit (farm list and pool
detail) and alerts API key fields, linking each input to its error via
aria-describedby and isInvalid.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@drips-wave

drips-wave Bot commented Oct 17, 2026

Copy link
Copy Markdown

@DooseWayo 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

@netlify

netlify Bot commented Oct 17, 2026 •

Copy link
Copy Markdown

❌ Deploy Preview for spiffy-melomakarona-eb1e8a failed.

Name Link
🔨 Latest commit 2050177
🔍 Latest deploy log https://app.netlify.com/projects/spiffy-melomakarona-eb1e8a/deploys/6ad50526837e36348fe3e84f

@netlify

netlify Bot commented Oct 17, 2026 •

Copy link
Copy Markdown

❌ Deploy Preview for smart-drop failed.

Name Link
🔨 Latest commit 2050177
🔍 Latest deploy log https://app.netlify.com/projects/smart-drop/deploys/6ad50526f3e16fc48755007d

@ritaifeoluwa
ritaifeoluwa merged commit 8a931d8 into SmartDropLabs:main Oct 18, 2026
0 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants