Skip to content

Conversation

@gabotorresruiz
Copy link
Contributor

SUMMARY

This PR modernizes the navigation menu implementation, and improves consistency between Menu.tsx and RightMenu.tsx

It also fixes spacing and tab inconsistencies that occurred when the navbar height changed (e.g., due to logo size or applied margins).

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

Before:

before

After:

after

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

  • Has associated issue:
  • Required feature flags:
  • Changes UI
  • Includes DB Migration (follow approval process in SIP-59)
    • Migration is atomic, supports rollback & is backwards-compatible
    • Confirm DB migration upgrade and downgrade tested
    • Runtime estimates and downtime expectations provided
  • Introduces new feature or API
  • Removes existing feature or API

@bito-code-review
Copy link
Contributor

bito-code-review bot commented Nov 14, 2025

Code Review Agent Run #7492fc

Actionable Suggestions - 0
Review Details
  • Files reviewed - 3 · Commit Range: 6211e74..6211e74
    • superset-frontend/src/features/home/Menu.tsx
    • superset-frontend/src/features/home/RightMenu.tsx
    • superset-frontend/src/hooks/useThemeMenuItems.tsx
  • Files skipped - 0
  • Tools
    • Whispers (Secret Scanner) - ✔︎ Successful
    • Detect-secrets (Secret Scanner) - ✔︎ Successful

Bito Usage Guide

Commands

Type the following command in the pull request comment and save the comment.

  • /review - Manually triggers a full AI review.

  • /pause - Pauses automatic reviews on this pull request.

  • /resume - Resumes automatic reviews.

  • /resolve - Marks all Bito-posted review comments as resolved.

  • /abort - Cancels all in-progress reviews.

Refer to the documentation for additional commands.

Configuration

This repository uses Default Agent You can customize the agent settings here or contact your Bito workspace admin at [email protected].

Documentation & Help

AI Code Review powered by Bito Logo

@dosubot dosubot bot added the change:frontend Requires changing the frontend label Nov 14, 2025
Copy link

@korbit-ai korbit-ai bot left a comment

Choose a reason for hiding this comment

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

I've completed my review and didn't find any issues.

Files scanned
File Path Reviewed
superset-frontend/src/hooks/useThemeMenuItems.tsx
superset-frontend/src/features/home/Menu.tsx
superset-frontend/src/features/home/RightMenu.tsx

Explore our documentation to understand the languages and file types we support and the files we ignore.

Check out our docs on how you can make Korbit work best for you and your team.

Loving Korbit!? Share us on LinkedIn Reddit and X

@gabotorresruiz gabotorresruiz force-pushed the fix/navbar-inconsistencies branch from 6211e74 to f3fd76b Compare November 14, 2025 17:50
Copy link
Member

@sadpandajoe sadpandajoe left a comment

Choose a reason for hiding this comment

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

Small nit, but not a blocker

@gabotorresruiz gabotorresruiz force-pushed the fix/navbar-inconsistencies branch from f3fd76b to f01d0b0 Compare November 15, 2025 00:45
@gabotorresruiz
Copy link
Contributor Author

gabotorresruiz commented Nov 15, 2025

Small nit, but not a blocker

Hey @sadpandajoe can you please check this out --> f01d0b0

I think that's the best I can do to have that offset in one place and have it reference from all the menu popups from the navbar

@gabotorresruiz gabotorresruiz force-pushed the fix/navbar-inconsistencies branch from f01d0b0 to 9add887 Compare November 15, 2025 01:08
Copy link
Member

@sadpandajoe sadpandajoe left a comment

Choose a reason for hiding this comment

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

Nice!

@sadpandajoe sadpandajoe merged commit 9bff648 into apache:master Nov 17, 2025
75 of 76 checks passed
sadpandajoe pushed a commit that referenced this pull request Nov 17, 2025
sadpandajoe pushed a commit that referenced this pull request Nov 17, 2025
@sadpandajoe sadpandajoe added the v6.0 Label added by the release manager to track PRs to be included in the 6.0 branch label Nov 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

change:frontend Requires changing the frontend size/L v6.0 Label added by the release manager to track PRs to be included in the 6.0 branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants