Skip to content

TablePagination className not merged #17456

Description

@DavidHenri008

The TablePagination component does not use the clsx function to merge the classes.root and the className property.

  • The issue is present in the latest release.
  • I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯

This leads to classes.root being discarded and replaced by className when both are supplied.

Expected Behavior 🤔

The clsx application should probably be used like in other components like TableBody, TableCell, etc.

Steps to Reproduce 🕹

In the following example, you may look at the console html and see that the classes are missing on the MuiTablePagination-root div.
https://codesandbox.io/s/create-react-app-8ho0q

Your Environment 🌎

Tech Version
Material-UI v4.4.0
React 16.9.0
Chrome 76.0.3809

Metadata

Metadata

Assignees

No one assigned

    Labels

    good first issueGreat for first contributions. Enable to learn the contribution process.type: bugIt doesn't behave as expected.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions