Skip to content

Fix mobile overlay not closing on navigation link clicks#1198

Draft
Copilot wants to merge 2 commits intovlk/refactor-navigationfrom
copilot/sub-pr-1195
Draft

Fix mobile overlay not closing on navigation link clicks#1198
Copilot wants to merge 2 commits intovlk/refactor-navigationfrom
copilot/sub-pr-1195

Conversation

Copy link
Contributor

Copilot AI commented Feb 18, 2026

The mobile overlay remained open after clicking navigation links because only the Playground link had an onClick handler. Navigation without closing the overlay creates poor UX.

Changes:

  • Added onClick=toggleMobileOverlay to in-app links (Blog, Community, Packages)
  • Added onClick=closeMobileOverlay to external links (X, Bluesky, GitHub, Forum)
// Before
<Link to=#"/blog" className={...}>
  {React.string("Blog")}
</Link>

// After
<Link to=#"/blog" className={...} onClick=toggleMobileOverlay>
  {React.string("Blog")}
</Link>

External links use closeMobileOverlay instead of toggleMobileOverlay since they navigate away from the app.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: jderochervlk <60623931+jderochervlk@users.noreply.github.com>
Copilot AI changed the title [WIP] Update navbar based on review feedback from PR #1195 Fix mobile overlay not closing on navigation link clicks Feb 18, 2026
Copilot AI requested a review from jderochervlk February 18, 2026 18:54
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

Comments