-
-
Notifications
You must be signed in to change notification settings - Fork 32.2k
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
[zero][Badge] Identify zero runtime styled path #40555
Conversation
Netlify deploy previewhttps://deploy-preview-40555--material-ui.netlify.app/ Bundle size report |
c99cfba
to
0c18fee
Compare
b3c7d82
to
608c975
Compare
and replace emotion/system with zero runtime at build time.
608c975
to
b9e72dc
Compare
@@ -6,7 +6,7 @@ import { usePreviousProps } from '@mui/utils'; | |||
import { unstable_composeClasses as composeClasses } from '@mui/base/composeClasses'; | |||
import { useBadge } from '@mui/base/useBadge'; | |||
import { useSlotProps } from '@mui/base'; | |||
import styled from '../styles/styled'; | |||
import { styled } from '../zero-styled'; |
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.
Added alternate way to specify zero-runtime readiness without the need of a separate package. The only requirement now is that the import path should end with /zero-styled
. We can change this path as well.
The other approach seemed unnecessarily complex and required a side-effect file import.
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.
👏 Great job!
and replace emotion/system with zero runtime at build time.
To test locally, see the README of this repo.