Skip to content

[TextField] Migrate FilledInput to emotion#24634

Merged
mnajdova merged 8 commits intomui:nextfrom
mnajdova:feat/filled-input-alternative
Jan 27, 2021
Merged

[TextField] Migrate FilledInput to emotion#24634
mnajdova merged 8 commits intomui:nextfrom
mnajdova:feat/filled-input-alternative

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 scope: text field Changes related to the text field. 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

scope: text field Changes related to the text field.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants