Skip to content

Conversation

@arvindm84
Copy link

Description

  • This PR fixes Mobile Menu with weird paddings #533 by removing the extra empty padding shown when the mobile menu is expanded with no navigation items, while keeping the actions (search bar, brightness toggle, GitHub icon) visible.

Validation

Before:
Before

After:
After

Related Issues

Fixes #533

Check List

  • I have read the Contributing Guidelines and made commit messages that follow the guideline.
  • I have run node --run test and all tests passed.
  • I have checked code formatting with node --run format & node --run lint.
  • I've covered new added functionality with unit tests if necessary.

@arvindm84 arvindm84 requested a review from a team as a code owner December 27, 2025 11:31
@vercel
Copy link

vercel bot commented Dec 27, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
api-docs-tooling Ready Ready Preview Dec 27, 2025 11:32am

@avivkeller
Copy link
Member

Hmm, if we are setting this element to not display, I wonder if I can just not render it if no nav items are provided. I'll look into that on the components repository

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Mobile Menu with weird paddings

2 participants