-
Notifications
You must be signed in to change notification settings - Fork 16.2k
fix(navbar): some styling + components inconsistencies #36120
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(navbar): some styling + components inconsistencies #36120
Conversation
Code Review Agent Run #7492fcActionable Suggestions - 0Review Details
Bito Usage GuideCommands Type the following command in the pull request comment and save the comment.
Refer to the documentation for additional commands. Configuration This repository uses Documentation & Help |
There was a problem hiding this 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.
6211e74 to
f3fd76b
Compare
sadpandajoe
left a comment
There was a problem hiding this 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
f3fd76b to
f01d0b0
Compare
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 |
f01d0b0 to
9add887
Compare
sadpandajoe
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice!
(cherry picked from commit 9bff648)
(cherry picked from commit 9bff648)
SUMMARY
This PR modernizes the navigation menu implementation, and improves consistency between
Menu.tsxandRightMenu.tsxIt 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:
After:
TESTING INSTRUCTIONS
ADDITIONAL INFORMATION