migrate material ui to 4x

ShankzPatil commented 4 years ago

Layout [Grid] In order to support arbitrary spacing values and to remove the need to mentally count by 8, we are changing the spacing API:

/**

Defines the space between the type item component.
It can only be used on a type container component. */
- spacing: PropTypes.oneOf([0, 8, 16, 24, 32, 40]),
- spacing: PropTypes.oneOf([0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10]),

if you don't provide spacing by default value set by lib is 16. so it gives warning error.

vs-form commented 4 years ago

Hello @ShankzPatil . I will fix issues and update material-ui next time. Thanks.

ShankzPatil commented 4 years ago

What is ETA for the Material ui 4 because expansion panel is also replaced by accordion. so there are more changes required.

Issue with radio button is show stopper.
react-dom.development.js:89 Warning: Can't perform a React state update on an unmounted component. This is a no-op, but it indicates a memory leak in your application. To fix, cancel all subscriptions and asynchronous tasks in the componentWillUnmount method.

vs-form / vs-form-repo