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

[TextField] Migrate OutlinedInput to emotion #24688

Merged
merged 20 commits into from
Feb 17, 2021

Conversation

duganbrett
Copy link
Contributor

This PR migrates the OutlinedInput component to the new emotion format as a part of #24405.

@mui-pr-bot
Copy link

mui-pr-bot commented Jan 29, 2021

@material-ui/core: parsed: +0.08% , gzip: +0.08%

Details of bundle changes

Generated by 🚫 dangerJS against 59ea06d

@oliviertassinari oliviertassinari added the component: text field This is the name of the generic UI component, not the React module! label Jan 29, 2021
packages/material-ui/src/OutlinedInput/NotchedOutline.js Outdated Show resolved Hide resolved
packages/material-ui/src/OutlinedInput/NotchedOutline.js Outdated Show resolved Hide resolved
packages/material-ui/src/OutlinedInput/NotchedOutline.js Outdated Show resolved Hide resolved
packages/material-ui/src/OutlinedInput/OutlinedInput.js Outdated Show resolved Hide resolved
packages/material-ui/src/OutlinedInput/OutlinedInput.js Outdated Show resolved Hide resolved
packages/material-ui/src/OutlinedInput/OutlinedInput.js Outdated Show resolved Hide resolved
@@ -130,11 +173,7 @@ const OutlinedInput = React.forwardRef(function OutlinedInput(props, ref) {
}
/>
)}
classes={{
...classes,
root: clsx(classes.root, classes.underline),
Copy link
Member

Choose a reason for hiding this comment

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

You missed the underline classes in the useUtilityClasses above

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's been a while since I looked at this but I assumed this was here as an error. This shouldn't have an underline class.

@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged label Feb 15, 2021
@github-actions github-actions bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged label Feb 15, 2021
tchock added a commit to Bojagi/material-ui that referenced this pull request Feb 16, 2021
This is a copy of upstream PR mui#24688
tchock added a commit to Bojagi/material-ui that referenced this pull request Feb 16, 2021
This is a copy of upstream PR mui#24688
@oliviertassinari oliviertassinari self-assigned this Feb 16, 2021
tchock added a commit to Bojagi/material-ui that referenced this pull request Feb 16, 2021
This is a copy of upstream PR mui#24688
@oliviertassinari
Copy link
Member

@duganbrett Thanks!

@duganbrett duganbrett deleted the migrate-outlined-input branch February 18, 2021 19:30
tchock added a commit to Bojagi/material-ui that referenced this pull request Mar 10, 2021
This is a copy of upstream PR mui#24688
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!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants