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

[core] Improve support for extended props in theme #25934

Merged
merged 35 commits into from
Apr 26, 2021

Conversation

vicasas
Copy link
Member

@vicasas vicasas commented Apr 24, 2021

Added support custom for

  • Autocomplete -> size
  • Table -> size
  • ToggleButton -> size
  • ToggleButtonGroup -> size
  • Slider -> color
  • Pagination -> color, size
  • PaginationItem -> color, size
  • Fab -> color, size
  • SvgIcon -> color, size
  • Icon -> color, size
  • Chip -> color, size
  • InputBase -> color
  • OutlinedInput -> color
  • FilledInput -> color
  • Input -> color
  • Switch -> color, size
  • Checkbox -> color, size
  • Radio -> color, size

Closes #13875

@mui-pr-bot
Copy link

mui-pr-bot commented Apr 24, 2021

Details of bundle changes

@material-ui/core: parsed: +Infinity% , gzip: +Infinity%
@material-ui/lab: parsed: +Infinity% , gzip: +Infinity%
@material-ui/styles: parsed: +Infinity% , gzip: +Infinity%
@material-ui/system: parsed: +Infinity% , gzip: +Infinity%
@material-ui/unstyled: parsed: +Infinity% , gzip: +Infinity%
@material-ui/utils: parsed: +Infinity% , gzip: +Infinity%

Generated by 🚫 dangerJS against fedb770

@oliviertassinari oliviertassinari changed the title [ToggleButton][ToggleButtonGroup][Table][Autocomplete] Support custom sizes [core] Improve support for extended props in theme Apr 24, 2021
@oliviertassinari oliviertassinari added core Infrastructure work going on behind the scenes new feature New feature or request labels Apr 24, 2021
@vicasas
Copy link
Member Author

vicasas commented Apr 24, 2021

@oliviertassinari I'll keep uploading a few more commits.

@vicasas
Copy link
Member Author

vicasas commented Apr 24, 2021

I don't understand why the ci was broken in TableContext, TableCell since no changes have been made to them.

@vicasas
Copy link
Member Author

vicasas commented Apr 24, 2021

I don't understand why the ci was broken in TableContext, TableCell since no changes have been made to them.

I already understood, repairing...

@oliviertassinari I have the correction ready but before committing I wonder if we should better revert this change. 617802e

Update: I pushed the fix in this commit de467cc

@vicasas
Copy link
Member Author

vicasas commented Apr 24, 2021

I think there are 3 pending. I will continue later here or in another PR.

@vicasas
Copy link
Member Author

vicasas commented Apr 25, 2021

@oliviertassinari I think I need help on the last 3 missing components (Switch, Checkbox, Radio). For hours I have tried to deal with framer's mistakes without success.

@oliviertassinari
Copy link
Member

@vicasas Ok, it should be fixed now. It also seems that you handled all the cases, I have marked the issue as closed by this PR :)

@vicasas
Copy link
Member Author

vicasas commented Apr 25, 2021

@oliviertassinari Thanks!! I will finish the last two remaining Checkbox and Radio and we are ready.

@vicasas
Copy link
Member Author

vicasas commented Apr 25, 2021

With this last commit they would all be done. 💯🚀

@github-actions github-actions bot added PR: out-of-date The pull request has merge conflicts and can't be merged labels Apr 26, 2021
Copy link
Member

@mnajdova mnajdova left a comment

Choose a reason for hiding this comment

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

🚀 Thanks @vicasas

@github-actions github-actions bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged label Apr 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core Infrastructure work going on behind the scenes new feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[theme] Allow custom color variants
4 participants