Details of bundle changes (experimental)

Generated by 🚫 dangerJS against b25f23b

@mnajdova One thing that I wonder after did some of the migration for internal components. It seems like the second one is simpler and gives the same result as the first one?

const Root = styled('div', {}, { skipSx: true })

const Root = styled('div')

Argos fail due to PickersCalendar is not migrate to emotion and css from withStyles has higher priority. will open another PR to migrate PickersCalendar first.

@mnajdova One thing that I wonder after did some of the migration for internal components. It seems like the second one is simpler and gives the same result as the first one?

const Root = styled('div', {}, { skipSx: true })

const Root = styled('div')

What do you mean result as the first one? The default value for skipSx is false: https://github.com/mui-org/material-ui/blob/next/packages/material-ui/src/styles/experimentalStyled.js#L81 Are you suggesting we change the default or?

@mnajdova One thing that I wonder after did some of the migration for internal components. It seems like the second one is simpler and gives the same result as the first one?

const Root = styled('div', {}, { skipSx: true })

const Root = styled('div')

What do you mean result as the first one? The default value for skipSx is false: https://github.com/mui-org/material-ui/blob/next/packages/material-ui/src/styles/experimentalStyled.js#L81 Are you suggesting we change the default or?

My bad, didn't explain it clearly. I mean why do we need to have skipSx option here, is it help on the performance if skipSx is provided?.

My bad, didn't explain it clearly. I mean why do we need to have skipSx option here, is it help on the performance if skipSx is provided?

We discussed it in one of the first pickers PR #25989 In my opinion we don't need to do it, but let's follow the convention it is at this moment, we can update before the beta version if we decide it's ok to have it on the internal components as well.