Skip to content

Fix Solved : Mobile navbar overlaps hero section content on smaller screens#312

Merged
mattqdev merged 3 commits into
physicshub:mainfrom
Vamshavardhan50:fix/mobile-nav-drawer
May 15, 2026
Merged

Fix Solved : Mobile navbar overlaps hero section content on smaller screens#312
mattqdev merged 3 commits into
physicshub:mainfrom
Vamshavardhan50:fix/mobile-nav-drawer

Conversation

@Vamshavardhan50

Copy link
Copy Markdown
Contributor

🔍 Description

  • Replaced the mobile dropdown menu with an off‑canvas drawer to stop hero overlap.
  • Added a dimmed backdrop behind the drawer and click‑to‑close behavior.
  • Added a close button (X icon) inside the drawer and Esc‑to‑close support.
  • Locked body scroll while the drawer is open.
  • Closed the drawer automatically when a nav link is clicked.
  • Added focus‑trap + focus‑restore for accessibility.
  • Added a staggered slide‑in animation for mobile nav items.

Closes #302

🎨 Visual Changes

  • Mobile screenshots (menu closed / menu open with backdrop)
image
  • Short screen recording showing open/close + Esc close
Screen.Recording.2026-05-15.115953.mp4

🧩 Additional Notes for Reviewers

  • Main UI changes:
    • app/(core)/components/Header.jsx/components/Header.jsx)
    • app/(core)/components/Nav.tsx/components/Nav.tsx)
    • app/(core)/styles/index.css/styles/index.css)
  • Tooling fixes to pass lint‑staged:
    • eslint.config.mjs — use FlatCompat for Next configs
    • pre-commit — removed deprecated Husky shim lines

@Vamshavardhan50 Vamshavardhan50 changed the title Fix ESLint flat config for Next.js Fix Solved : Mobile navbar overlaps hero section content on smaller screens May 15, 2026
@Vamshavardhan50
Vamshavardhan50 marked this pull request as draft May 15, 2026 06:39
@Vamshavardhan50
Vamshavardhan50 marked this pull request as ready for review May 15, 2026 06:40
@mattqdev

Copy link
Copy Markdown
Collaborator

Thank you for this! I really love it!

@mattqdev
mattqdev merged commit 7cfdad3 into physicshub:main May 15, 2026
2 checks passed
@physicshub

Copy link
Copy Markdown
Owner

🎉 This PR is included in version 3.28.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Mobile navbar overlaps hero section content on smaller screens

3 participants