Missing @mui/system dependency in @mui/core package

[rtritto]

With #28923 Popper was moved from @mui/material to @mui/core (change).
Popper still use @mui/system dependency that it was not added in package.json of @mui/core.

So I got Module not found: Can't resolve '@mui/system' error in @mui\core\Popper\index.js.

Versions:

@mui/material: from 5.0.5 to current 5.1.0

[South-Paw]

ETA?

I've got a couple of tickets outstanding on this issue & its a relatively straight forward resolution in @mui/core if I'm not mistaken?

[michaldudak]

The initial idea for the core package was not to depend on the system, but it seems we may not be able to avoid it (cc @mnajdova). I'll try to come up with another solution and will get back to you.

[mnajdova]

Agree, we should not have it as a dependency. We need to find alternative, maybe accept isRtl as a prop in the unstyled version and create a thin wrapper in the @mui/material package that would set it based on the theme.