Skip to content

feat: per-field validation feedback on Create Payment form#135

Merged
emdevelopa merged 1 commit intoemdevelopa:mainfrom
Majormaxx:feat/form-validation-feedback
Mar 29, 2026
Merged

feat: per-field validation feedback on Create Payment form#135
emdevelopa merged 1 commit intoemdevelopa:mainfrom
Majormaxx:feat/form-validation-feedback

Conversation

@Majormaxx
Copy link
Copy Markdown

Summary

  • Added amountError and recipientError state variables for per-field inline error display
  • Invalid fields get a red border (border-red-500/50) instead of the default white border
  • Inline <p role="alert"> messages appear directly below each invalid field with specific guidance
  • aria-invalid and aria-describedby attributes added for screen-reader accessibility
  • Errors are cleared as soon as the user starts correcting the field
  • Both fields are checked before early return so all errors appear at once

Closes #37

@drips-wave
Copy link
Copy Markdown

drips-wave bot commented Mar 26, 2026

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

@emdevelopa
Copy link
Copy Markdown
Owner

fix conflict

@emdevelopa
Copy link
Copy Markdown
Owner

Resolve Conflict

- Added amountError and recipientError state for inline error display
- Red border applied to invalid fields on submit
- Inline <p role="alert"> error messages appear below each invalid field
- aria-invalid and aria-describedby set for screen-reader accessibility
- Errors clear when user starts correcting the field
- Both fields validated before any early return (prevents partial submit)
@Majormaxx Majormaxx force-pushed the feat/form-validation-feedback branch from 362d902 to 5a08681 Compare March 28, 2026 22:38
@vercel
Copy link
Copy Markdown

vercel bot commented Mar 28, 2026

Someone is attempting to deploy a commit to the Emmanuel's projects Team on Vercel.

A member of the Team first needs to authorize it.

@emdevelopa emdevelopa merged commit eff98ec into emdevelopa:main Mar 29, 2026
2 of 3 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

Development

Successfully merging this pull request may close these issues.

[FE] Form Validation Feedback

2 participants