[material-ui][Table] `slots` and `slotProps` documented properties are not exposed in package about material-ui

[siriwatknp]

For deep slots, what should be the usage? I'm thinking

Make sense to me for the time being but in v7, we should consider restructuring components to not have nested slots.

[siriwatknp]

cc @DiegoAndai should we have another umbrella to take care of the completeness of slots and slotProps as a follow-up from the deprecation? I can own this if you'd like.

[DiegoAndai]

Hey @quibaritaenperdresatrompe, thanks for the report! The button icon's slots are under the action slots, as they're slots of the TablePaginationActions component. So you can access them through, for example, MuiTablePagination.defaultProps.slots.actions.nextButtonIcon. Here's an example: https://stackblitz.com/edit/stackblitz-starters-5rgtos?file=src%2FApp.tsx.

I would mark this issue as expected by design.

For context, this was introduced in #39353.

cc @DiegoAndai should we have another umbrella to take care of the completeness of slots and slotProps as a follow-up from the deprecation? I can own this if you'd like.

@siriwatknp Yes, I think we should have an issue for that 🙌🏼 the goal would be to add any missing slots and slotProps that were not added due to a deprecation. Could you create that issue?

I'm also curious about what you think about the sidenote in #33797 (comment):

I realized that we made a mistake in that PR. It should be slotProps.actions.slotProps.firstButton.

For deep slots, what should be the usage? I'm thinking

• slots.<slot> to override one-level slots
• slotProps.<slot> to override one-level slot's props
• slotProps.<parentSlot>.slots.<childSlot> to override two-level slots
• slotProps.<parentSlot>.slotProps.<childSlot> to override two-level slot's props
• slotProps.<grandparentSlot>.slotProps.<parentSlot>.slots.<childSlot> to override three-level slots
• ...

I don't know if we have any more of these. If we go with that, then the current TablePagination implementation is incorrect as it shouldn't allow slots.actions.nextButtonIcon, the correct path should be slotProps.actions.slots.nextButtonIcon.

[DiegoAndai]

I think we should close this issue as expected and create a new one to remove nested slots in the future; what do you think?

[siriwatknp]

I think we should close this issue as expected and create a new one to remove nested slots in the future; what do you think?

Sounds good. The new issue could be an umbrella one that covers all existing components that contain nested slots.