Skip to content

fix: remove mobile nav label padding override#429

Merged
patcapulong merged 1 commit intomainfrom
bugfix/sidebar-icon-alignment
May 1, 2026
Merged

fix: remove mobile nav label padding override#429
patcapulong merged 1 commit intomainfrom
bugfix/sidebar-icon-alignment

Conversation

@patcapulong
Copy link
Copy Markdown
Contributor

Summary

  • Remove an overbroad mobile nav selector that applied bottom padding to nested .flex.flex-wrap elements.
  • Fix mobile sidebar nav item alignment by preventing label wrappers from receiving extra bottom spacing.

Test plan

  • Verified in local Mintlify preview at http://localhost:3001 by inspecting the mobile nav in responsive mode.

Made with Cursor

@vercel
Copy link
Copy Markdown

vercel Bot commented May 1, 2026

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

Project Deployment Actions Updated (UTC)
grid-flow-builder Ready Ready Preview, Comment May 1, 2026 1:22am

Request Review

@mintlify
Copy link
Copy Markdown
Contributor

mintlify Bot commented May 1, 2026

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
Grid 🟢 Ready View Preview May 1, 2026, 1:23 AM

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented May 1, 2026

Greptile Summary

This PR removes a single CSS rule (div#mobile-nav div#navigation-items .flex.flex-wrap) that was applying padding-bottom: 20px !important to all .flex.flex-wrap elements inside the mobile nav — including label wrappers that shouldn't receive extra bottom spacing. The removal is minimal and targeted; existing spacing rules for ul.sidebar-group and group containers (padding-bottom: 24px) continue to handle the intended layout.

Confidence Score: 5/5

Safe to merge — single rule removal with no logic errors or regressions identified.

The change is a clean 5-line CSS deletion of a known-overbroad selector. No new rules are introduced, no other selectors are affected, and existing spacing declarations for sidebar groups remain in place.

No files require special attention.

Important Files Changed

Filename Overview
mintlify/style.css Removes overbroad .flex.flex-wrap padding rule in mobile nav; surrounding spacing rules remain intact and unaffected.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["#mobile-nav #navigation-items"] --> B[".flex.flex-wrap elements"]
    A --> C["ul.sidebar-group / group containers"]

    B -- "BEFORE: padding-bottom 20px (overbroad)" --> D["label wrappers\nalso received padding ❌"]
    B -- "AFTER: no rule applied" --> E["label wrappers\nunaffected ✅"]

    C -- "padding-bottom: 24px (remains)" --> F["Sidebar group spacing\ncorrectly maintained ✅"]
Loading

Reviews (1): Last reviewed commit: "fix: remove mobile nav label padding ove..." | Re-trigger Greptile

@patcapulong patcapulong merged commit 26a2b20 into main May 1, 2026
8 checks passed
@patcapulong patcapulong deleted the bugfix/sidebar-icon-alignment branch May 1, 2026 02:03
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.

2 participants