No bundle size changes (experimental)

Generated by 🚫 dangerJS against 0d2d95f

@eps1lon I think that we have tried to keep the CSS to its minimum in the visual regression tests so far, but there is likely more room for simplification 👍. I would add a note of caution: we added some CSS to create the right environment to reproduce bugs, these will still need to be here.

I think that we have tried to keep the CSS to its minimum in the visual regression tests so far, but there is likely more room for simplification 👍.

Hiding more stuff behind an abstraction (e.g. sx or Box) does not make a test simpler.

The style -> sx changes should be reverted. sx is unrelated to the tests.

I think it is not clear to me. Should we remove sx and use style in all tests including components being tested

-<Input sx={{ margin: 1 }} />
+<Input style={{ margin: 1 }} />

or only for Box components?

-<Box sx={{ margin: 1 }} />
+<div style={{ margin: 1 }} />

I think it is not clear to me. Should we remove sx and use style in all tests including components being tested

It's not about removing. It's about keeping it the way it is. We just want to migrate the parts that are using @material-ui/styles. All the rest should stay that way (e.g. style prop usage).

What is missing from this?

Using style over sx in the visual regression tests could make sense. It will be less refactoring in the future if we change the API. The style API is more reliable (assuming a 10 years window)