Closed jeremyckahn closed 9 months ago
vercel[bot]
commented
9 months ago The latest updates on your projects. Learn more about Vercel for Git ↗︎
| Name | Status | Preview | Comments | Updated (UTC) |
|---|---|---|---|---|
| farmhand | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Dec 20, 2023 4:40pm |
lstebner
commented
9 months ago
these tooltips are behaving a bit differently now. when they show up and i move the mouse to the next tool, the tooltip seems to get focus and i have to mouse all the way off, then back, to click a tool the tip is blocking. maybe we can move them to the left more so they never block the tools? on prod right now this doesn't seem to happen
What this PR does
This PR upgrades Farmhand from MUI 4 to MUI 5. Necessary code changes are made, as well as a handful of UI updates to support the MUI 5 component changes.
How this change can be validated
This is a sweeping change, so a wide-ranging smoke test would be best.
Questions or concerns about this change
I've tried to thoroughly test this and fix regressions, but it's possible I missed something. For a change this foundational, I think we need to accept that we will likely introduce minor regressions. We can fix them as we find them. Hopefully I didn't miss anything significant!
Additional information
I recommend hiding whitespace changes when reviewing this.
I followed the MUI migration guide for this upgrade: https://mui.com/material-ui/migration/migration-v4/
The majority of this PR is updating imports.
The most significant UI update is the design of the tabs. This is the old tabs component:
And this is the new one: