Skip to content

Upgrade to Next 12 and MUI 5 #199

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

Closed
wants to merge 37 commits into from
Closed

Upgrade to Next 12 and MUI 5 #199

wants to merge 37 commits into from

Conversation

cgbuen
Copy link
Collaborator

@cgbuen cgbuen commented Dec 1, 2021

No description provided.

cgbuen and others added 7 commits November 30, 2021 13:55
- Install mui 5.2
- Remove material-ui 4
- Replace peerDependencies
- Upgrade minimum node version (in preparation for next)
Note: consider readjusting sizing values if necessary
Note: consider fixing accordion test manually, as it errored during
codemod run
(For JSS => emotion migration)
(where codemods didn't work)
@cgbuen cgbuen added do not merge in progress Work is still in progress labels Dec 1, 2021
cgbuen and others added 4 commits December 20, 2021 13:20
- Ensure defaultClass.root is used over passed in classes.root for
  Carousel
- Update thumbnails Tab wrapper usage
- Fix spaces in selectors where necessary
- Migrate JSS selector code to styled for hovering => expand
- Fix expand media query
- Added 'material-ui-popup-state' to manage hover/popup menus for main navigation
- Updated NavTab to use the aforementioned library
@ali-hellani
Copy link

is this task still in progress ?

@andersonbrandon andersonbrandon removed do not merge in progress Work is still in progress labels Jan 24, 2022
@andersonbrandon andersonbrandon self-requested a review January 24, 2022 22:53
Copy link
Collaborator

@andersonbrandon andersonbrandon left a comment

Choose a reason for hiding this comment

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

v9.0.0-alpha.0

@andersonbrandon
Copy link
Collaborator

Going to fix the tests...

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.

4 participants