Skip to content

fix: add a burger menu for mobile navbar to stop overflow - #7

Merged
flvvius merged 1 commit into
mainfrom
bug-fix/mobile-navbar
Mar 1, 2026
Merged

fix: add a burger menu for mobile navbar to stop overflow#7
flvvius merged 1 commit into
mainfrom
bug-fix/mobile-navbar

Conversation

@flvvius

@flvvius flvvius commented Mar 1, 2026

Copy link
Copy Markdown
Owner

Summary by CodeRabbit

  • New Features
    • Responsive header that adapts navigation display based on screen size: full menu on desktop, hamburger menu on mobile devices.
    • Mobile navigation drawer slides in from the side when triggered, contains navigation links, and closes upon selection.
    • Improved mobile user experience with accessible and animated menu interactions.

@vercel

vercel Bot commented Mar 1, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
news-web Building Building Preview, Comment Mar 1, 2026 11:05am

@coderabbitai

coderabbitai Bot commented Mar 1, 2026

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 59f9eed and 35379a9.

📒 Files selected for processing (2)
  • apps/web/src/components/header.tsx
  • apps/web/src/components/ui/sheet.tsx

Walkthrough

Introduces a responsive header component with mobile navigation via a Sheet drawer component. The header displays desktop navigation on medium+ screens and provides a burger menu on smaller screens that triggers a sheet-based mobile menu. Adds new Sheet UI component library wrapping Radix UI primitives with configurable positioning and styling options.

Changes

Cohort / File(s) Summary
Header Responsiveness
apps/web/src/components/header.tsx
Adds useState for sheet state management, conditional desktop navigation visibility on medium+ screens, and mobile burger menu button that opens a Sheet drawer. Navigation links moved to module scope.
Sheet Component Library
apps/web/src/components/ui/sheet.tsx
New Sheet UI component set built on Radix UI primitives. Exposes 10 components (Sheet, SheetTrigger, SheetClose, SheetPortal, SheetOverlay, SheetContent, SheetHeader, SheetFooter, SheetTitle, SheetDescription) with support for side positioning and optional close button rendering.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: adding a Sheet-based burger menu for mobile navigation to address navbar overflow issues.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch bug-fix/mobile-navbar

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

@flvvius
flvvius merged commit 4654143 into main Mar 1, 2026
3 checks passed
@coderabbitai coderabbitai Bot mentioned this pull request May 10, 2026
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.

1 participant