Skip to content
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

Migrate makeStyles to MUI v5 #330

Merged
merged 16 commits into from
Feb 2, 2024

Conversation

dklimpel
Copy link
Contributor

@dklimpel dklimpel commented Feb 2, 2023

Get rid of MUI v4.

The remaining import { makeStyles } from "@material-ui/core/styles"; have to be migrated with React Admin 3->4.

@@ -345,7 +345,6 @@ const LoginPage = ({ theme }) => {
type="submit"
color="primary"
disabled={loading || !supportPassAuth}
className={classes.button}
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have not seen any reason for this. No class was defined for this on top of this file.

@dklimpel dklimpel closed this Feb 2, 2023
@dklimpel dklimpel reopened this Feb 3, 2023
@dklimpel dklimpel marked this pull request as draft February 3, 2023 07:25
@dklimpel dklimpel marked this pull request as ready for review February 3, 2023 14:44
Comment on lines -26 to -37
export const ImportButton = ({ label, variant = "text" }) => {
return (
<ReactAdminButton
color="primary"
component="span"
variant={variant}
label={label}
>
<GetAppIcon style={{ transform: "rotate(180deg)", fontSize: "20" }} />
</ReactAdminButton>
);
};
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was not uses.

@dklimpel dklimpel mentioned this pull request Feb 4, 2023
@awesome-manuel awesome-manuel merged commit 58e02d6 into Awesome-Technologies:master Feb 2, 2024
1 check passed
@dklimpel dklimpel deleted the migrate-styles branch February 2, 2024 15:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants