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

Details of bundle changes

Generated by 🚫 dangerJS against 354ff22

@mnajdova On a side note, we think that we should remove MenuList > ListItem in v5 because the abstraction makes too little sense.

@mnajdova On a side note, we think that we should remove MenuList > ListItem in v5 because the abstraction makes too little sense.

In that case should we hold off with these changes until the component is refactored?

In that case should we hold off with these changes until the component is refactored?

@mnajdova I have repurposed #21587 to keep track of this effort. I don't know when it will happen. I think that it would be more valuable to first remove JSS from the list of dependencies entirely. Having to implement the components/componentsProps makes us a step further in the unstyled direction we are pursuing

@mnajdova I have repurposed #21587 to keep track of this effort. I don't know when it will happen. I think that it would be more valuable to first remove JSS from the list of dependencies entirely. Having to implement the components/componentsProps makes us a step further in the unstyled direction we are pursuing

Makes sense! @xs9627 would you like to proceed with this? #24634 should be a good template for introducing components and componentsProps and using them in a component.

Not quite sure about the changes, just saw all tests passed for MenuItem and ListItem :)

@xs9627 Thanks :)