Benny Joo
Benny Joo
> @siriwatknp Could you update the PR? I think I will try to add a codemod for transforming TextField to Input and then we can merge this PR. Done. I...
> @siriwatknp I haven't. That'd be great if you could add it. Done
@siriwatknp I think I covered all cases. Happy to get the codemod reviewed at this point.
Here is a list of system properties that can be used directly on the component as a prop: https://mui.com/system/properties/ It doesn't include `backgroundColor`. Please use `sx` prop for this property.
> But shouldn't the typings also have reflected the same? The typings reflect the same, and isn't that why you got a TS complaint when using `backgroundColor` directly?
My bad. In fact, `background-color` should be supported too. The [documentation](https://mui.com/system/properties/#properties-reference-table) says the following > Note that this table only lists custom properties, all other regular CSS properties and selectors...
I will look into this today.
@SanNic20 I am aware that you reported multiple issues with Masonry component, and I appreciate it. Still, I will reduce the scope of this issue to **"`ResizeObserver loop limit exceeded`...
I think that's a valid concern. According to this (https://stackoverflow.com/a/50387233), we can ignore the error. The functionality doesn't get affected by the error.
MUI X DataGrid went with the same solution though. Ref: https://github.com/mui/mui-x/pull/8744 Should we proceed with merging the PR? @mnajdova