-
-
Notifications
You must be signed in to change notification settings - Fork 4
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 material UI to version5 #21
Migrate material UI to version5 #21
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a first review pass with a couple of commets
63b95c3
to
2129735
Compare
Ok, The import was the problem, this is the bad side of using an automation tool. In addition, the theme needs to be adapted to this https://material-theme.com/docs/reference/color-palette/#material-themes and the migration needs to reflect the actual theme on production https://bruce.lnmetrics.info/ |
14c5825
to
bd4e348
Compare
In addition, the deprecation of the Metrics UI looks like not finished! |
6175942
to
1e20af0
Compare
Fixes #5.