-
Notifications
You must be signed in to change notification settings - Fork 70
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
Table: Error message styling for all Inputs #2721
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for moduswebcomponents ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
…t' into 2711---Input-Fire-valueError-event-on-error
@prashanthr6383 There are two scenarios with respect to validation.
Our form components already have the required support for consumer side validation so we will first focus on extending the same for the table component as a first step. This PR is related to the inbuild constraint validation that is not implemented in our form components yet, the proposed changes rely on the errorText that is passed by the consumer to emit the error event back to the consumer. With that said I'll move the PR to draft state for now until we decide on the desired behavior and what we are going to support. |
Description
Added new styles to the table inputs error message
note: for autocomplete duplicate error message is shown which is part of another issue #2720.
References
Fixes #2711, Fixes #2712, Fixes #2722, Fixes #2725, Fixes #2727
Type of change
How Has This Been Tested?
Checklist