Skip to content

refactor: [M3-6361] - MUI v5 Migration - Components > MenuItem#9295

Merged
carrillo-erik merged 5 commits intolinode:developfrom
carrillo-erik:refactor/M3-6361
Jun 29, 2023
Merged

refactor: [M3-6361] - MUI v5 Migration - Components > MenuItem#9295
carrillo-erik merged 5 commits intolinode:developfrom
carrillo-erik:refactor/M3-6361

Conversation

@carrillo-erik
Copy link
Copy Markdown
Contributor

Description 📝

MUI v5 migration for the MenuItem component.

Major Changes 🔄

  • Ran tss-react codemod on MenuItem
  • Replaced default export with named export, as a result the component is now imported as WrapperMenuItem

How to test 🧪

  1. Verify e2e tests pass
  2. Verify unit tests pass
  3. Inspect for any visual regressions

@cypress
Copy link
Copy Markdown

cypress Bot commented Jun 22, 2023

Passing run #4621 ↗︎

0 177 3 0 Flakiness 0

Details:

Merge branch 'develop' of https://github.com/linode/manager into refactor/M3-636...
Project: Cloud Manager E2E Commit: 2069119148
Status: Passed Duration: 13:55 💡
Started: Jun 29, 2023 5:27 AM Ended: Jun 29, 2023 5:41 AM

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

Comment thread packages/manager/src/components/MenuItem/MenuItem.tsx
Comment thread packages/manager/src/components/MenuItem/MenuItem.tsx Outdated
Comment thread packages/manager/src/components/MenuItem/MenuItem.tsx
Copy link
Copy Markdown
Contributor

@dwiley-akamai dwiley-akamai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The Create button dropdown items have a font size a bit smaller than prod currently:

Prod

Screenshot 2023-06-23 at 3 44 57 PM

This branch

Screenshot 2023-06-23 at 3 45 12 PM

In prod, editing a NodeBalancer config --> "Backend Nodes" --> the Mode dropdown has some interesting behavior. Clicking that dropdown places a white overlay on the rest of the app. That behavior isn't present on this branch, which is more aligned with other dropdowns throughout Cloud.

@carrillo-erik
Copy link
Copy Markdown
Contributor Author

I was able to identify the same discrepancies as you stated in your comment. However, I'm unsure if my changes introduced this visual defect as the MenuItem component I touched and the one found in both of the other components (NodeBalancerConfigNode & AddNewMenu) where defect is present are different.

@bnussman-akamai
Copy link
Copy Markdown
Member

bnussman-akamai commented Jun 26, 2023

@dwiley-akamai I refactored the "Add New" Menu to use MUI recently. This is why it might feel a bit different than normal. UX approved changes on it #9224 🎉

As for the NodeBalancer Mode dropdown. This one is unique because it uses MUI and not the React Select library. It might feel different because UX asked us to change the backdrop behavior in that same PR disucssed above https://github.com/linode/manager/pull/9224/files#diff-f767369a01cd40921dc0da8dac44cf6777e3c155f3680aa2990be0da93c2dc48R381

@bnussman-akamai bnussman-akamai added Approved Multiple approvals and ready to merge! and removed Add'tl Approval Needed Waiting on another approval! labels Jun 28, 2023
@carrillo-erik carrillo-erik merged commit bab130b into linode:develop Jun 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Approved Multiple approvals and ready to merge!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants