fix: allow sidebar to be shrunk when wider than viewport

[shoaib050326]

Summary

• Always accept sidebar resize when user is shrinking (nextWidth <= currentWidth), fixing a deadlock where the sidebar becomes permanently stuck when the window is restored to a smaller size after being maximized.

Root Cause

shouldAcceptWidth in AppSidebarLayout.tsx evaluated wrapper.clientWidth - nextWidth >= 640 for every drag step. When the sidebar width exceeds the viewport width (e.g. sidebar expanded while maximized, then window restored to small size), this expression is negative, blocking ALL resize attempts — even shrinking — creating a permanent deadlock.

Fix

-  shouldAcceptWidth: ({ nextWidth, wrapper }) =>
-    wrapper.clientWidth - nextWidth >= THREAD_MAIN_CONTENT_MIN_WIDTH,
+  shouldAcceptWidth: ({ currentWidth, nextWidth, wrapper }) =>
+    nextWidth <= currentWidth ||
+    wrapper.clientWidth - nextWidth >= THREAD_MAIN_CONTENT_MIN_WIDTH,

• currentWidth is already provided by the caller (sidebar.tsx:480) but was previously unused.
• Shrinking is always accepted. Expansion still enforces the 640px main-content minimum.

Risk

Low. One-line logic change in a single callback. Rollback is a one-line revert.

Closes #2339

---

Note

Low Risk
Low risk: a small change to the sidebar resize acceptance predicate that only relaxes constraints when shrinking; expansion behavior still enforces the minimum main-content width.

Overview
Prevents the thread sidebar from getting stuck at an oversized width after the window is resized smaller by updating AppSidebarLayout’s resizable.shouldAcceptWidth.

Shrinking (nextWidth <= currentWidth) is now always accepted, while expanding still requires enough remaining space to satisfy the THREAD_MAIN_CONTENT_MIN_WIDTH constraint.

^{Reviewed by Cursor Bugbot for commit 8d8c62b. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Fix sidebar shrinking when wider than the viewport

Previously, the sidebar resize logic in AppSidebarLayout.tsx only accepted a new width if the remaining main content met THREAD_MAIN_CONTENT_MIN_WIDTH, which blocked shrinking when the sidebar was wider than the viewport. The shouldAcceptWidth callback now also accepts any width decrease unconditionally, so shrinking always works regardless of viewport size.

^{Macroscope summarized 8d8c62b.}

[coderabbitai]

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: f2bf9943-4f0a-4d0f-967b-8de05a13d05a

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[macroscopeapp]

Approvability

Verdict: Approved

Straightforward UI bug fix that adds a condition to allow shrinking a sidebar that was previously too wide. The change is minimal, self-contained, and has clear intent with no security or infrastructure implications.

^{You can customize Macroscope's approvability policy. Learn more.}