Material ui v4

[Floriferous]

Here's the migration checklist: https://material-ui.com/guides/migration-v3/

It doesn't seem to me that this package uses almost any of the changed features. I fixed a couple tests, only the target had to change.

Also, instead of pinning material-ui I let it loose, they have followed semver very well with proper deprecations before any braking changes in their major versions. Since they regularly add new features in their minor versions, that will give users of this package a bit more flexibility.

Closes #542.

[codecov]

Codecov Report

Merging #543 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master   #543   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files         162    162           
  Lines        1491   1491           
=====================================
  Hits         1491   1491

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8b1a1a2...c9b3739. Read the comment docs.

[radekmie]

To make it usable with @material-ui/core@4, you need to change this line only. Top level package.json and package-lock.json are being used only in the tests and documentation (playground). Updating the above line and tests will be enough - we can switch to the newer version internally later.

[Floriferous]

Alright, I updated the package requirements, any reason to undo the top-level package.json version changes?

[radekmie]

Yeah, actually two: one, this change has almost completely rewritten the package-lock.json file (even not related to this update), two, as a rule, each change should be minimal. Also, this change added two not needed (and problematic in a monorepo) files, namely website/package-lock.json and packages/uniforms-material/package-lock.json.

[Floriferous]

Okay, just so I get this right, I undo all the changes (including tests, since the ones I fixed depend on root material-ui@4), except the package.json in uniforms-material/package.json ?

[radekmie]

Updating package dependencies is enough. If you’d like to add these tests too, then leave the top level dependencies updated but update the lock file as little as possible. I can do it later today.

[radekmie]

Done. I had to update enzyme and adapter too, because of enzymejs/enzyme#2146.