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

Add support for Material-UI v5 beta #418

Merged
merged 12 commits into from
Aug 21, 2021
Merged

Add support for Material-UI v5 beta #418

merged 12 commits into from
Aug 21, 2021

Conversation

joemaffei
Copy link
Contributor

@joemaffei joemaffei commented Aug 16, 2021

This is a refactor of the existing styling using the conventions recommended by the Material-UI team for upgrading to v5.

Fixes #412

@benediktdertinger
Copy link

@iamhosseindhv is there any chance this can merged the upcoming days?

@iamhosseindhv iamhosseindhv changed the title Upgrade to Material-UI v5 Add support for Material-UI v5.0 Aug 18, 2021
@iamhosseindhv
Copy link
Owner

iamhosseindhv commented Aug 18, 2021

Thanks for your contribution. I'm reviewing this PR.
I've changed the base branch to next since master is currently compatible with v4 (MUI latest). next branch is supposed to be compatible with MUI v5 since it's still in beta.

@iamhosseindhv iamhosseindhv changed the base branch from master to next August 18, 2021 22:18
@rickstaa

This comment has been minimized.

@rickstaa
Copy link

rickstaa commented Aug 19, 2021

@iamhosseindhv Seems to work fine with material-ui v5 (i.e. ts and js versions) when the package-lock.json and package.json conflicts are resolved by removing the package-lock.json file and recreating it using the npm install command. @joemaffei Thanks again for creating this pull request.

onpaws added a commit to splitgraph/splitgraph.com that referenced this pull request Aug 19, 2021
- intended to support the notistack alpha release that claims MUIv5 beta support
- at the time of writing, latest MUI release is 5.0.0-beta.4
- if CI doesn't pass, another option is here: iamhosseindhv/notistack#418
@joemaffei
Copy link
Contributor Author

@iamhosseindhv conflicts resolved

@iamhosseindhv iamhosseindhv changed the title Add support for Material-UI v5.0 Add support for Material-UI v5 beta Aug 21, 2021
@iamhosseindhv iamhosseindhv merged commit e09bff2 into iamhosseindhv:next Aug 21, 2021
@iamhosseindhv
Copy link
Owner

Thanks for your contribution all specially @joemaffei @onpaws ❤️. You can now use this by npm i notistack@next. Please open a new issue if you're having any issues.

iamhosseindhv added a commit that referenced this pull request Sep 26, 2021
* Closes #334 #333 - Fix collapse keys

* 1.0.3-alpha.0

* 1.0.4-alpha.0

* 1.0.6-alpha.0

* Fixes #355 - Update MUI peer dependency to next

* 1.0.6-next.0

* Use root classKey for Collapse

* 1.0.6-next.1

* Add support for Material-UI v5 beta (#418)

* upgrade to material-ui v5; modify code as recommended by the material-ui docs

* upgrade to material-ui v5; modify code as recommended by the material-ui docs

* add missing anchor classes to SnackbarItem; fix child class declarations

* update dependencies; run build

* package-lock update

* Allow passing classes from props

* Adapt new MUI breakpoints

* Bind this

* Mark emotion peerDeps as optional

* 1.0.6-next.2

Co-authored-by: Hossein Dehnokhalaji <[email protected]>

* Support @mui/material package naming (#423)

* Support Material-UI 5.0.0-rc.0

* Bump version to 1.0.6-next.3

* Remove @mui/styles dependency

* Prepare next to be published as latest (#436)

* Add tag for people on mui-v4

* Update simple example

* Update simple example

* Update redux example

* Update mobx example

* Minor

* Update next --> latest in example codes

* Update docs

* Update CHANGELOG

* Update MUI v5 peerDependencies

* Update version to v2.0.2

* Update docs.json

Co-authored-by: Joe Maffei <[email protected]>
Co-authored-by: David Ramos <[email protected]>
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.

Material-UI 5.0 support
4 participants