mui / material-ui

Material UI: Ready-to-use foundational React components, free forever. It includes Material UI, which implements Google's Material Design.
https://mui.com/material-ui/
MIT License
91.86k stars 31.57k forks source link

[material-ui][Tooltip] deprecate `*Component` and `*Props` for v6 #42120

Open lhilgert9 opened 2 weeks ago

lhilgert9 commented 2 weeks ago

Part of #41281. @DiegoAndai

Question: Is it important that className of PopperProps and slotProps.popper are both applied, because a test exists in Tooltip.test.js?

I've also seen that #42107 is being worked on. We should see that the changes in these two PR's go together.

mui-bot commented 2 weeks ago

Netlify deploy preview

SpeedDialAction: parsed: +0.93% , gzip: +0.92% Tooltip: parsed: +1.14% , gzip: +1.03% packages/material-ui/material-ui.production.min.js: parsed: +0.05% , gzip: +0.07%

Bundle size report

Details of bundle changes (Toolpad) Details of bundle changes

Generated by :no_entry_sign: dangerJS against b7da0246b21b9b5ff6852874126df8204a5b17a3

lhilgert9 commented 4 days ago

CI fails because apparently the same popper element is not used in a rerender. I think that the error is in the useSlot hook. However, I have not found it yet. If anyone has an idea please let me know.