Skip to content

fix(web): dismiss mobile sidebar sheet after navigation#1549

Open
henrik092 wants to merge 1 commit intopingdotgg:mainfrom
henrik092:fix/mobile-sidebar-dismiss
Open

fix(web): dismiss mobile sidebar sheet after navigation#1549
henrik092 wants to merge 1 commit intopingdotgg:mainfrom
henrik092:fix/mobile-sidebar-dismiss

Conversation

@henrik092
Copy link
Copy Markdown

@henrik092 henrik092 commented Mar 29, 2026

On mobile, the sidebar renders as a slide-over sheet. After tapping a thread, creating a new thread, or navigating to settings, the sheet stays open and must be manually dismissed.

Add a closeMobileSidebar() helper that calls setOpenMobile(false) when isMobile is true, and invoke it after:

  • plain-click thread navigation (handleThreadClick)
  • programmatic thread navigation (navigateToThread)
  • new thread creation
  • settings navigation

Fixes #1258

What Changed

Why

UI Changes

Checklist

  • This PR is small and focused
  • I explained what changed and why
  • I included before/after screenshots for any UI changes
  • I included a video for animation/interaction changes

Note

Low Risk
Low risk UI behavior change limited to calling setOpenMobile(false) after user-initiated navigations; minimal impact outside mobile and no data/auth logic touched.

Overview
Ensures the mobile sidebar (sheet) auto-dismisses after navigation actions.

Sidebar now uses useSidebar() to add a closeMobileSidebar() helper and calls it after thread navigation (click and keyboard/programmatic), after creating a new thread from a project, and after navigating to Settings.

Written by Cursor Bugbot for commit be06b7b. This will update automatically on new commits. Configure here.

Note

Dismiss mobile sidebar sheet after navigation in Sidebar

On mobile, the sidebar sheet previously stayed open after navigating to a thread, creating a new thread, or opening Settings. A closeMobileSidebar callback (via useSidebar) is now called after each navigation action in Sidebar.tsx, so the sheet closes automatically on mobile.

Macroscope summarized be06b7b.

On mobile, the sidebar renders as a slide-over sheet. After tapping a
thread, creating a new thread, or navigating to settings, the sheet
stays open and must be manually dismissed.

Add a closeMobileSidebar() helper that calls setOpenMobile(false) when
isMobile is true, and invoke it after:
- plain-click thread navigation (handleThreadClick)
- programmatic thread navigation (navigateToThread)
- new thread creation
- settings navigation

Fixes pingdotgg#1258
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 29, 2026

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: ecad4f5c-1bdf-4362-9710-64e710806b98

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@github-actions github-actions bot added vouch:unvouched PR author is not yet trusted in the VOUCHED list. size:S 10-29 changed lines (additions + deletions). labels Mar 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S 10-29 changed lines (additions + deletions). vouch:unvouched PR author is not yet trusted in the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature]: Hide mobile sidebar after creating or selecting a thread

1 participant