Skip to content
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

fix: disable validation of internal text-field #1052

Merged
merged 4 commits into from
Sep 21, 2023

Conversation

vursen
Copy link
Contributor

@vursen vursen commented Sep 21, 2023

Description

The PR disables the internal text-field's validation so that it doesn't override the propagated invalid state.

Fixes vaadin/flow-components#5409

Type of change

  • Bugfix

@vursen vursen force-pushed the fix/disable-validation-of-internal-text-field branch from 947404f to 7b2ef0b Compare September 21, 2023 06:50
@vursen
Copy link
Contributor Author

vursen commented Sep 21, 2023

The tests failed in iOS Simulator due to a timeout. I think it's not a big deal since they passed in all the other environments.

@vursen vursen merged commit a17b8ab into master Sep 21, 2023
2 of 3 checks passed
@vursen vursen deleted the fix/disable-validation-of-internal-text-field branch September 21, 2023 12:29
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.

Combobox with required-flag set: error not shown/disappears on focus loss
2 participants