Skip to content

- Possible fix for Inline validation messages on field lose focus eve… #1233

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
May 27, 2025

Conversation

janadh
Copy link
Collaborator

@janadh janadh commented May 5, 2025

…nt #1219

  • We added event handlers for focus lost event (onBlur) and display the error message based on the state variables.

janadh and others added 3 commits May 5, 2025 18:26
#1219

- We added event handlers for focus lost event (onBlur) and display the error message based on the state variables.
- Always show feedback after first blur
@fongsean
Copy link
Collaborator

I've added the following changes:

  • Abstracted showFeedback state to a custom hook
  • Always show feedback after first blur

@fongsean
Copy link
Collaborator

This change should be in version alpha.57

@fongsean fongsean merged commit 5794cc4 into alpha May 27, 2025
5 of 6 checks passed
@fongsean fongsean deleted the alpha-jv-1219 branch May 27, 2025 01:49
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.

2 participants