-
-
Notifications
You must be signed in to change notification settings - Fork 32.5k
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
[FormHelperText] Error styles should override disabled styles #13217
[FormHelperText] Error styles should override disabled styles #13217
Conversation
@matthewdordal Please could you follow the issue title / commit message format we use elsewhere? "[ComponentName] Imperative commit message" |
84788b8
to
56d1991
Compare
@mbrookes sorry that I missed the commit message format. I updated the commit message. Please let me know if there's any other things I should update. 😸 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's wait for the next minor v3.3.0. |
@oliviertassinari I agree with you that this is more of a bugfix, however I'm not sure how much sense it makes (the behavior, rather than the fix per se). How can you fix an error on a disabled input?! |
@mbrookes The error could be fixed by another component. Here is a case: Assume a user can use the |
@matthewdordal It's a great first pull request on Material-UI 👌🏻. Thank you for working on it! |
Material UI is a great project so just looking to help out.
According to the issue the error styles in the
FormHelperText
take precedence over the disabled styles. Please let me know if there is further changes that are required. If this is not an actual issue also feel free to close the PR.Before this PR here was the behavior:
Example Code:
Before this PR:
After this PR:
Closes #13210