-
-
Notifications
You must be signed in to change notification settings - Fork 32.3k
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
add missing "component" field to List props #21564
Conversation
Details of bundle changes.Comparing: 9e7c0c1...2c5b585 Details of page changes
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for your work!
However, this is not quite correct. You should use a similar approach to #20179.
This PR should also target the next
branch. We're only making critical bug fixes on master
currently.
@eps1lon thanks for the super quick response! Not sure I understand how to approach this. With
Is I will change the PR to point to |
This is also missing on the ButtonProps for the Button component. |
@dclark27 If you have a reproduction, opening a new issue would be more appropriate :) |
@peterkrieg do you think that you will have time to dedicate on the problem in the near future? If no, it's fine, we could close the pull request and reopen it once the timing is right :). |
Hi @oliviertassinari ! Thanks for responding. I'm not really sure what the requested changes are for this PR. Seems like at least a few components are missing the |
A global approach is used which is why the |
@peterkrieg I'm closing as has been inactive for a week. If you find the time to resume the work on this, please do, a fix to the problem would be appreciated :). |
Adds missing
component
field toListProps
. The referenced documentation for these props can be found here: https://material-ui.com/api/list/#propsFix: #17579