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][Select] Copy Select files from v5 #39188

Merged
merged 3 commits into from
Sep 28, 2023

Conversation

DiegoAndai
Copy link
Member

Part of #38972, #29345

This PR copies the files related to the Select component from the material package to the material-next package with minimal modification (e.g. import paths, just enough to pass CI) so it's easier to see the diffs of the following-up #38972 PRs.

There are some imports left from the material package that will be updated later:

@DiegoAndai DiegoAndai added component: select This is the name of the generic UI component, not the React module! v6.x labels Sep 27, 2023
@DiegoAndai DiegoAndai added this to the v6 milestone Sep 27, 2023
@DiegoAndai DiegoAndai self-assigned this Sep 27, 2023
@mui-bot
Copy link

mui-bot commented Sep 27, 2023

Netlify deploy preview

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

@mui/material-next: parsed: +22.27% , gzip: +19.10%

Bundle size report

Details of bundle changes (Toolpad)
Details of bundle changes

Generated by 🚫 dangerJS against d6cac35

Copy link
Member

@mj12albert mj12albert left a comment

Choose a reason for hiding this comment

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

🙌

@DiegoAndai DiegoAndai merged commit 0a3e8e4 into mui:master Sep 28, 2023
6 checks passed
@DiegoAndai DiegoAndai deleted the material-next-select-copy branch September 28, 2023 17:49
@DiegoAndai DiegoAndai added v7.x and removed v6.x labels Dec 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: select This is the name of the generic UI component, not the React module! v7.x
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants