-
Notifications
You must be signed in to change notification settings - Fork 931
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
Warning: Material-UI: theme.spacing.unit usage has been deprecated. #676
Comments
Material-UI v4 isn't supported right now, stay on 3. |
When it will be ready? |
@bbtimx Unfortunately, I don't have a good timeline for this at the moment, but it is on my radar, as a lot of people have been asking for it. There are more serious issues that require handling first, but I will turn to upgrading once those are resolved. |
@gabrielliwerant Would you be open to a PR for upgrading to Material 4? I just upgraded it locally and it wasn't too painful. It did require updating a number of deps, but all tests pass, spot check of examples work, and the docs still work. |
@PizzaBrandon I think I'd like to hold off for a little while longer, as I'm thinking that such an upgrade should come with a bump to the major release version, as it will be difficult to guarantee that there is no breakage. I'd like to squeeze in a few more minor releases before that point, but the plan is definitely to move to v4 at some point, so I won't forget. Or if you like, feel free to ping me about it again in a few weeks. ;) |
@gabrielliwerant I agree that this should definitely be treated as a breaking change that should be published as a new major. I would not expect installations using v3 to work correctly after this update. That said, if the update was maintained in a separate branch and published to npm under a |
And, of course, I offer to help maintain that branch at least until it makes it to the live version so it's not an extra burden for you. |
@PizzaBrandon understood, let me think about it. It might indeed be a strain on my already limited time. I also think a week or so would not be too long to wait to avoid the overhead of a separate |
MUI-datatables works like expected but throw an error message.
Expected Behavior
No error message.
Current Behavior
Error message:
Steps to Reproduce (for bugs)
Your Environment
The text was updated successfully, but these errors were encountered: