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

Opening dialog messes up theme #48

Closed
DMLON opened this issue Feb 4, 2022 · 3 comments
Closed

Opening dialog messes up theme #48

DMLON opened this issue Feb 4, 2022 · 3 comments

Comments

@DMLON
Copy link

DMLON commented Feb 4, 2022

Hello, i'm trying to use this tool, but every time the confirm function is called, all of my website theme goes to default and i have to reload the website, any ideas what is going on?

EDIT: Cloning the repo and using the files directly works, i'm guessing it's an issue with the npm build using old material ui files (As far as i read this is an issue with material ui mui/material-ui#26452 )

@DMLON DMLON closed this as completed Feb 4, 2022
@DMLON DMLON reopened this Feb 4, 2022
@jonatanklosko
Copy link
Owner

Hey @DMLON, did you place ConfirmProvider inside ThemeProvider?

@DMLON
Copy link
Author

DMLON commented Feb 5, 2022

Yeah, tried it both inside and outside, had the same effect

@jonatanklosko
Copy link
Owner

Note that material-ui-confirm 3 is compatible with Material UI 5 and material-ui-confirm 2 with Material UI 4. Since you pointed out the Material UI issue, I don't think there's much we can do on this side, so I'm gonna close this, feel free to reopen if that's not the case :)

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

No branches or pull requests

2 participants