Skip to content

fix: restore scrolling after navigation#481

Merged
OXeu merged 3 commits into
mainfrom
fix/restore-scroll-after-navigation
Mar 8, 2026
Merged

fix: restore scrolling after navigation#481
OXeu merged 3 commits into
mainfrom
fix/restore-scroll-after-navigation

Conversation

@OXeu
Copy link
Copy Markdown
Collaborator

@OXeu OXeu commented Mar 8, 2026

Summary

  • move mobile menu scroll locking into a controlled effect
  • restore body scrolling when the menu closes, the route changes, or the component unmounts
  • add a regression test for navigation while the mobile menu is open

Testing

  • cd client && bun run check
  • cd client && bun run test src/components/site-header/primitives/tests/menu.test.tsx

@chatgpt-codex-connector
Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 8, 2026

@OXeu OXeu merged commit 0278244 into main Mar 8, 2026
7 checks passed
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 8, 2026

🧹 Preview deployment has been cleaned up.

The temporary preview environment for this PR has been deleted.

@OXeu OXeu deleted the fix/restore-scroll-after-navigation branch March 12, 2026 17:55
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