Skip to content

Conversation

@dgageot
Copy link
Member

@dgageot dgageot commented Jan 30, 2026

Previously, pressing Enter on an elicitation dialog with invalid fields would silently do nothing, making users think the Enter key wasn't working.

Now the dialog:

  • Highlights invalid field labels in red
  • Shows descriptive error messages below each invalid field
  • Focuses the first field with an error
  • Clears errors when the user modifies a field

This provides clear feedback when validation fails, such as:

  • 'This field is required' for empty required fields
  • 'Must be a valid number' for invalid numeric input
  • 'Must be at least N' for minimum value violations
  • 'Must be a valid email address' for email format errors

Assisted-By: cagent

Previously, pressing Enter on an elicitation dialog with invalid fields
would silently do nothing, making users think the Enter key wasn't working.

Now the dialog:
- Highlights invalid field labels in red
- Shows descriptive error messages below each invalid field
- Focuses the first field with an error
- Clears errors when the user modifies a field

This provides clear feedback when validation fails, such as:
- 'This field is required' for empty required fields
- 'Must be a valid number' for invalid numeric input
- 'Must be at least N' for minimum value violations
- 'Must be a valid email address' for email format errors

Assisted-By: cagent
@dgageot dgageot requested a review from a team as a code owner January 30, 2026 12:30
@dgageot dgageot merged commit f5d8440 into docker:main Jan 30, 2026
5 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.

2 participants