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

[material-next][TextField] Copy v5 TextField's inner components #39166

Merged

Conversation

mj12albert
Copy link
Member

@mj12albert mj12albert commented Sep 26, 2023

Part of #38374

This PR copies (with minimal changes) the remaining components that are required to compose the filled variant of TextField and Select:

@mui-bot
Copy link

mui-bot commented Sep 26, 2023

Netlify deploy preview

https://deploy-preview-39166--material-ui.netlify.app/

@mui/material-next: parsed: +3.08% , gzip: +2.34%

Bundle size report

Details of bundle changes (Toolpad)
Details of bundle changes

Generated by 🚫 dangerJS against 92fe002

@mj12albert mj12albert force-pushed the copy-material-next-input-helper-components branch 4 times, most recently from e97f9f8 to 234a869 Compare September 28, 2023 12:44
@mj12albert mj12albert force-pushed the copy-material-next-input-helper-components branch from 234a869 to 92fe002 Compare October 3, 2023 07:17
@mj12albert mj12albert added the component: text field This is the name of the generic UI component, not the React module! label Oct 3, 2023
@mj12albert mj12albert marked this pull request as ready for review October 3, 2023 10:14
@siriwatknp
Copy link
Member

@mj12albert Is this ready to go?

@mj12albert mj12albert merged commit d1cbdcb into mui:master Oct 4, 2023
20 checks passed
@mj12albert mj12albert deleted the copy-material-next-input-helper-components branch October 4, 2023 09:47
@mj12albert
Copy link
Member Author

I merged it ~ @siriwatknp

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: text field This is the name of the generic UI component, not the React module! design: material you v7.x
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants