Fix sidebar thread panel weird states.#1497
Conversation
- Remove Collapsible wrapper around project thread list - Hide thread rows and overflow cleanly when the panel is not shown
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
- Remove redundant thread panel guard from show more/less controls - Keep thread list expansion state tied to project expansion only

What Changed
Collapsiblewrapper around each project’s thread list in the left sidebar.Why
The sidebar had a desync bug where the project caret could remain in the expanded state while the thread list was visually missing. This showed up most often after refresh and after project reordering triggered by message sends.
The root cause was that visibility depended on both state and a height-measured animated container. The chevron was driven by persisted project expansion state, but the rows were still gated by a separate collapsible layout path that could end up with stale height during hydration or reorder. Removing that wrapper makes visibility purely state-driven and keeps animation out of the correctness path, which is a more reliable long-term structure.
UI Changes
This changes sidebar interaction behavior, so before/after screenshots and a short video should be included for:
vid
Screen.Recording.2026-03-29.at.12.40.22.AM.mov
ps : don't mind the lag, it's coz of low memory on my device.
Checklist
Note
Low Risk
Low risk UI change, but it alters sidebar expand/collapse rendering behavior and could affect thread list visibility/animation edge cases.
Overview
Fixes inconsistent sidebar project thread visibility by removing the
Collapsible/CollapsibleContentwrapper around each project’s thread list and rendering thread rows directly based onshouldShowThreadPanel.Keeps the thread list container mounted (now with
overflow-hidden) so reordering/auto-animate remains intact while avoiding height-measurement desyncs that could leave the chevron expanded with no visible threads.Written by Cursor Bugbot for commit 4204457. This will update automatically on new commits. Configure here.
Note
Fix weird states in sidebar thread panel by replacing Collapsible with conditional rendering
Removes the
Collapsible/CollapsibleContentwrapper from the project thread panel in Sidebar.tsx. Thread rows now render conditionally based onshouldShowThreadPanel, while theSidebarMenuSubcontainer always renders withoverflow-hidden. The 'Show more'/'Show less' controls are no longer gated byCollapsibleContent, so they can appear even when the thread panel is hidden.Macroscope summarized 4204457.