feat: main menu cloud sync

[amhsirak]

Summary by CodeRabbit

• Style
  • Standardized icon sizes (20px) across all tabs and action buttons for a more consistent look.
  • Updated tab icons, including introducing a Play icon where appropriate, all with explicit sizing.
  • Adjusted Documentation, Join Cloud, and Sponsor buttons to use uniformly sized start icons.
  • Minor visual polish improvements with no changes to navigation or behavior.
  • Results in cleaner, more balanced icon alignment and improved visual clarity.

[coderabbitai]

Walkthrough

Consolidates and prunes Material UI and icon imports in src/components/dashboard/MainMenu.tsx. Standardizes icon sizing (sx fontSize: 20) across tabs and buttons. Replaces/adjusts several tab and button icons (including adding PlayArrow, using Description for documentation). No changes to navigation or modal logic.

Changes

Cohort / File(s)	Summary
Main menu icon and import adjustments src/components/dashboard/MainMenu.tsx	Removed unused MUI imports (TextField, InputAdornment, IconButton). Replaced/updated icon imports (removed ContentCopy, added PlayArrow). Applied sx={{ fontSize: 20 }} to tab and button icons (AutoAwesome, VpnKey, Usb, CloudQueue, Description, Favorite, SlowMotionVideo, PlayArrow). No functional flow changes.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

• feat: oss community perks & sustainability #717 — Also edits MainMenu.tsx imports/icons and UI elements, overlapping with these icon and import changes.
• fix: match on hover menu tabs #755 — Touches the same imports and discount/copy UI that this PR removes, indicating direct conflict/sequence.
• feat: docs tab in main menu #642 — Changes the Documentation button to use the Description icon, matching this PR’s icon adjustment and sizing.

Suggested labels

Type: Enhancement, Scope: UI/UX

Poem

I trim my whiskers, neat and small,
Icons sized the same for all.
Tap-tap tabs, a tidy glow,
Play and docs now set to show.
No new routes—just spruced-up flair,
A rabbit’s touch of UI care. 🐇✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Title Check	⚠️ Warning	The pull request title indicates the addition of a “main menu cloud sync” feature, but the changeset only adjusts icon imports, sizes, and related UI elements without introducing any cloud sync functionality, making the title misleading.	Please update the title to accurately reflect the actual changes—such as “chore: standardize main menu icon sizing and imports”—or implement the described cloud sync feature if intended.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch sync-menu

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

src/components/dashboard/MainMenu.tsx (1)

26-47: Remove unused code.

The ossDiscountCode constant (line 26) and copyDiscountCode function (lines 40-47) are no longer used anywhere in the component. These should be removed to keep the codebase clean.

Apply this diff to remove the unused code:

-  const ossDiscountCode = "MAXUNOSS8";
-
   const handleChange = (event: React.SyntheticEvent, newValue: string) => {
     navigate(`/${newValue}`);
     handleChangeContent(newValue);
   };

   const handleRobotsClick = () => {
     if (location.pathname !== '/robots') {
       navigate('/robots');
       handleChangeContent('robots');
     }
   };
-
-  const copyDiscountCode = () => {
-    navigator.clipboard.writeText(ossDiscountCode).then(() => {
-      notify("success", "Discount code copied to clipboard!");
-    }).catch(err => {
-      console.error('Failed to copy text: ', err);
-      notify("error", "Failed to copy discount code.");
-    });
-  };

🧹 Nitpick comments (1)

src/components/dashboard/MainMenu.tsx (1)

102-141: Apply consistent icon sizing to modal buttons.

While the main menu buttons now have standardized icon sizing (sx={{ fontSize: 20 }}), the buttons inside the documentation modal (lines 114 and 124) lack this styling, creating a visual inconsistency. Apply the same sizing to all icon buttons for uniformity.

Apply this diff to ensure consistent icon sizing:

               <Stack spacing={2}>
                 <Button
                   href="https://docs.maxun.dev"
                   target="_blank"
                   rel="noopener noreferrer"
                   variant="outlined"
-                  startIcon={<Description />}
+                  startIcon={<Description sx={{ fontSize: 20 }} />}
                   fullWidth
                 >
                   Documentation
                 </Button>
                 <Button
                   href="https://www.youtube.com/@MaxunOSS/videos"
                   target="_blank"
                   rel="noopener noreferrer"
                   variant="outlined"
-                  startIcon={<SlowMotionVideo />}
+                  startIcon={<SlowMotionVideo sx={{ fontSize: 20 }} />}
                   fullWidth
                 >
                   Video Tutorials
                 </Button>
               </Stack>

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 95a4d3c and ef65fb3.

📒 Files selected for processing (1)

• src/components/dashboard/MainMenu.tsx (3 hunks)

🔇 Additional comments (2)

src/components/dashboard/MainMenu.tsx (2)

6-7: LGTM: Import cleanup is correct.

The removed imports (TextField, InputAdornment, IconButton, ContentCopy) are not used in the current code, and the added imports (PlayArrow, Description) are properly utilized in the tab and button icons.

---

92-95: LGTM: Consistent icon sizing across tabs.

The standardized fontSize: 20 styling across all tab icons creates visual consistency in the menu.