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 FilledInput to emotion #24634

Merged
merged 8 commits into from
Jan 27, 2021

Conversation

mnajdova
Copy link
Member

@mnajdova mnajdova commented Jan 26, 2021

Alternative of #24617 Migrates FilledInput to emotion and adds the components and componetnsProps props on the InpuBase component.

@mui-pr-bot
Copy link

mui-pr-bot commented Jan 26, 2021

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

Details of bundle changes

Generated by 🚫 dangerJS against 494d00c

@mnajdova mnajdova changed the title [TextField] Migrate InputBase to emotion [TextField] Migrate InputBase to emotion ver2 Jan 26, 2021
@mnajdova mnajdova changed the title [TextField] Migrate InputBase to emotion ver2 [TextField] Migrate InputBase to emotion Jan 26, 2021
@mnajdova mnajdova changed the title [TextField] Migrate InputBase to emotion [TextField] Migrate FilledInput to emotion Jan 26, 2021
@mnajdova
Copy link
Member Author

@oliviertassinari would be nice before merging if you can take a look on the solution for these argos differences that we had - https://www.argos-ci.com/mui-org/material-ui/builds/7319

Here is the commit 316c968

  • The first one is regarding using longhand props when merging the objects
  • The second one is about using classes vs JSS's $ syntax. Also maybe we should still keep the new component name with the classes together with the base, for example to have both MuiInputBase-sizeSmall and MuiFilledInput-sizeSmall. Now we have only the first.

Let me know what you think.

@oliviertassinari
Copy link
Member

@mnajdova OK, I'm on it.

@oliviertassinari oliviertassinari added the component: text field This is the name of the generic UI component, not the React module! label Jan 26, 2021
@oliviertassinari
Copy link
Member

@mnajdova I have fixed two visual regressions that were reported by Argos. For 1. I can't see the impact. For 2. I think that we can try without, and see if it fly.

@mnajdova mnajdova merged commit 4aca567 into mui:next Jan 27, 2021
xs9627 added a commit to xs9627/material-ui that referenced this pull request Jan 27, 2021
* next: (34 commits)
  [Tab] Migrate to emotion (mui#24651)
  [TextField] Migrate Input component to emotion (mui#24638)
  [ImageList] Migrate ImageListItemBar to emotion (mui#24632)
  [CircularProgress] Migrate to emotion (mui#24622)
  [DataTable] Add example in docs for data table (mui#24428)
  [Card] Migrate CardActionArea to emotion (mui#24636)
  [core] Fix `next` using stale pages (mui#24635)
  [List] Migrate ListItemIcon to emotion (mui#24630)
  [ClickAwayListener] Fix `children` and `onClickAway` types (mui#24565)
  [docs] Include in docs directive to silence `eslint` erroneous warning (mui#24644)
  [Fab] Migrate to emotion (mui#24618)
  [TextField] Migrate FilledInput to emotion (mui#24634)
  [Card] Migrate CardHeader to emotion (mui#24626)
  [Card] Migrate CardMedia to emotion (mui#24625)
  [ImageList] Migrate ImageListItem to emotion (mui#24619)
  [website] Add vision block (mui#24603)
  [docs] Add sorting section (mui#24637)
  [TextField] Prepare removal of labelWidth prop (mui#24595)
  [Dialog] Migrate DialogTitle to emotion (mui#24623)
  [ImageList] Migrate to emotion (mui#24615)
  ...
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