Skip to content

feat(forms): add blur validation and focus first invalid field on submit#4146

Open
Innagl wants to merge 1 commit into
developfrom
feature/form-validation-blur-focus
Open

feat(forms): add blur validation and focus first invalid field on submit#4146
Innagl wants to merge 1 commit into
developfrom
feature/form-validation-blur-focus

Conversation

@Innagl

@Innagl Innagl commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

Describe your changes

Wire useFormValidation blur handlers and submit-time focus across project, project risk, vendor risk, and vendor risk modal forms while keeping submit enabled and blocking save until validation passes.

Write your issue number after "Fixes "

Fixes #4145

Please ensure all items are checked off before requesting a review:

  • I deployed the code locally.
  • I have performed a self-review of my code.
  • I have included the issue # in the PR.
  • I have labelled the PR correctly.
  • The issue I am working on is assigned to me.
  • I have avoided using hardcoded values to ensure scalability and maintain consistency across the application.
  • I have ensured that font sizes, color choices, and other UI elements are referenced from the theme.
  • My pull request is focused and addresses a single, specific feature.
  • If there are UI changes, I have attached a screenshot or video to this PR.
Screenshot 2026-06-19 at 1 07 48 PM Screenshot 2026-06-19 at 1 07 31 PM

Wire useFormValidation blur handlers and submit-time focus across project,
project risk, vendor risk, and vendor risk modal forms while keeping submit
enabled and blocking save until validation passes.

Co-authored-by: Cursor <cursoragent@cursor.com>
@Innagl Innagl added this to the 2.4 milestone Jun 19, 2026
@Innagl Innagl self-assigned this Jun 19, 2026
@Innagl Innagl added enhancement New feature or request frontend Frontend related tasks/issues labels Jun 19, 2026
@Innagl

Innagl commented Jun 19, 2026

Copy link
Copy Markdown
Contributor Author

@MuhammadKhalilzadeh Specs (#4019 - 5. Improve form-validation UX across risk management, vendor management, and use cases) requires changes only for Vendor management, Risk management and Use cases. But I believe we should apply the same approach to the rest of the forms as well

@MuhammadKhalilzadeh

Copy link
Copy Markdown
Collaborator

@MuhammadKhalilzadeh Specs (#4019 - 5. Improve form-validation UX across risk management, vendor management, and use cases) requires changes only for Vendor management, Risk management and Use cases. But I believe we should apply the same approach to the rest of the forms as well

Thank you for the dedication @Innagl
Let's apply it wherever you see it's needed

@MuhammadKhalilzadeh

Copy link
Copy Markdown
Collaborator

Are we done on this PR @Innagl ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request frontend Frontend related tasks/issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improve form validation UX: blur validation and focus first error on submit

2 participants