Skip to content

[TextField] Fix missing standard variant classes in types#26115

Merged
siriwatknp merged 3 commits intomui:nextfrom
siriwatknp:form-standard-variant-classes-type
May 5, 2021
Merged

[TextField] Fix missing standard variant classes in types#26115
siriwatknp merged 3 commits intomui:nextfrom
siriwatknp:form-standard-variant-classes-type

Conversation

@siriwatknp
Copy link
Copy Markdown
Member

close #26114

@mui-pr-bot
Copy link
Copy Markdown

mui-pr-bot commented May 4, 2021

Details of bundle changes

Generated by 🚫 dangerJS against adc2199

Copy link
Copy Markdown
Member

@mnajdova mnajdova left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@mnajdova mnajdova changed the title [Typescript] add missing standard variant classes [core] add missing standard variant classes in types May 4, 2021
...styles[`position${capitalize(styleProps.position)}`],
...(styleProps.disablePointerEvents === true && styles.disablePointerEvents),
...(styleProps.variant === 'filled' && styles.filled),
...styles[styleProps.variant],
Copy link
Copy Markdown
Member Author

@siriwatknp siriwatknp May 4, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mnajdova I think this is simpler and also make it possible to style standard and outlined in theme.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agree

@oliviertassinari oliviertassinari changed the title [core] add missing standard variant classes in types [core] Fix missing standard variant classes in types May 4, 2021
@oliviertassinari oliviertassinari added the scope: text field Changes related to the text field. label May 4, 2021
@oliviertassinari oliviertassinari changed the title [core] Fix missing standard variant classes in types [TextField] Fix missing standard variant classes in types May 4, 2021
@siriwatknp siriwatknp merged commit 4a325e6 into mui:next May 5, 2021
@siriwatknp siriwatknp deleted the form-standard-variant-classes-type branch May 5, 2021 01:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope: text field Changes related to the text field. typescript

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Typescript] Form components missing standard key type in classes

4 participants