Skip to content

fix(web): allow collapsing the settled threads tail - #6380

Open
marcoamt wants to merge 4 commits into
pingdotgg:mainfrom
marcoamt:fix/sidebar-settled-threads-show-less
Open

fix(web): allow collapsing the settled threads tail#6380
marcoamt wants to merge 4 commits into
pingdotgg:mainfrom
marcoamt:fix/sidebar-settled-threads-show-less

Conversation

@marcoamt

@marcoamt marcoamt commented Aug 12, 2026

Copy link
Copy Markdown

What Changed

  • Added a way to collapse the settled threads list back after expanding it with "Show N more".
  • A small undo icon next to "Show N more" collapses back to the initial list; once fully expanded, the button itself becomes "Show less".

Why

The settled threads list shows the first 10 threads with a "Show 25 more" button, but there was no way to collapse it back once expanded. This adds a small, reversible affordance consistent with the existing un-settle icon pattern, without adding an extra row to the sidebar.

UI Changes

Before: the settled tail paging row only offers "Show 25 more" — no way back once expanded.

file-fc6047777d4b87a1cb4ac75153c25f0e

After: hovering the paging row reveals a rollback icon aligned with the thread rows' un-settle affordance, letting you collapse back to the initial page.

file-6ae786bc4fddce28920bd2a31e56fbdd

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

Implemented with Claude Sonnet 5 in Claude Code.


Note

Low Risk
Sidebar-only UI and local settledVisibleCount state; no API, auth, or data changes.

Overview
Adds Show less for the settled threads tail so expanding with Show N more is reversible, without an extra sidebar row.

Paging logic now tracks the active route thread via settledRouteThreadIndex, and uses settledFloorVisibleCount / settledIsExpanded so collapse only appears when it would actually hide rows (including when a deep-linked thread stays pinned visible). The paging row shows Show more while hidden threads remain; when fully expanded it becomes Show less, and while more pages exist a hover undo icon on the same row collapses back to the initial page—aligned with settled rows’ un-settle affordance.

Reviewed by Cursor Bugbot for commit 77178e0. Bugbot is set up for automated code reviews on this repo. Configure here.

Note

Allow collapsing the settled threads list in the sidebar

  • Adds a 'Show less' button to the settled threads section in Sidebar.tsx that resets the visible count back to the initial page size.
  • While partially expanded (more items hidden), a secondary 'Show less' control appears on hover/focus at the right side of the paging row.
  • The collapse action is suppressed when the currently routed thread is beyond the initial page and collapsing would be a no-op due to that thread being pinned visible.

Macroscope summarized 77178e0.

Expanding "Show 25 more" had no way back; the paging row now offers a
hover rollback while pages remain and collapses to "Show less" once
fully expanded.
@coderabbitai

coderabbitai Bot commented Aug 12, 2026

Copy link
Copy Markdown

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 Plus

Run ID: 25d2b8bd-173d-4075-b9e6-86a0d50589e3

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

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.

@github-actions github-actions Bot added vouch:unvouched PR author is not yet trusted in the VOUCHED list. size:M 30-99 changed lines (additions + deletions). labels Aug 12, 2026
Comment thread apps/web/src/components/Sidebar.tsx Outdated
Comment thread apps/web/src/components/Sidebar.tsx Outdated
@macroscopeapp

macroscopeapp Bot commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved 77178e0

This PR adds a 'Show less' button to collapse expanded settled threads in the sidebar - a minor UI enhancement complementing the existing 'Show more' feature. The changes are self-contained within the Sidebar component with no impact beyond the UI affordance.

You can customize Macroscope's approvability policy. Learn more.

settledVisibleCount only resets on a project-scope change, so un-settling,
deleting, or reclassifying threads could shrink the settled list back to (or
below) the initial page while leaving a stale, orphaned "Show less" row.
macroscopeapp[bot]
macroscopeapp Bot previously approved these changes Aug 12, 2026

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 74d92d3. Configure here.

Comment thread apps/web/src/components/Sidebar.tsx Outdated
… no-op

The open-thread pin can already put the tail at the same row count that
collapsing to the initial page would produce, so basing "expanded" on
settledVisibleCount alone showed a Show less control that changed nothing
when clicked.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@macroscopeapp
macroscopeapp Bot dismissed their stale review August 12, 2026 22:28

Dismissing prior approval to re-evaluate 77178e0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M 30-99 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.

1 participant