Sign up
View stargazers13Sign up

feat(web): simplify sidebar drag destination cues#9750

merged2
sidebar/drag-sections
sidebar/drag-destination-cue
openedSep 4, 2026byJJulius Marminge
2 files changed
View on GitHubTwo-way sync enabledPR last imported 3d ago

Dragging between sections repeats the move instruction on the boundary while the dragged thread still shows its old pin. This optional follow-up to #9731 puts that feedback on the thread: → Pinned, → Active, or → Settled appears when crossing sections. The usual pin, status, and hover actions hide during dragging; reordering within a section shows no destination badge. Boundaries keep their section names and highlight.

The sortable layout, drop behavior, and movement animations are unchanged. This affects the shared web/desktop sidebar, including compact rows dragged out of Snoozed or Settled. It can be accepted or closed independently of the parent PR.

Verified with 207 focused sidebar tests, web typecheck, targeted lint, and an independent code review. React Doctor reported only existing sidebar complexity warnings. The recordings exercise pinning and unpinning on matching fixtures; the dark screenshots cover a compact Settled row moving toward Active.

Before and after — pinned thread moving into Active

Before — parentAfter — experiment
Before: boundary repeats Drop to unpin and dragged row retains the pinAfter: neutral boundary label and one Active destination badge on the dragged row

Before recording — pickup, unpin, then pin again:

https://gh-file-drop-api-prod-mi5fy3sowv63ufte.pinglabs.workers.dev/f/0c102f7a0712df0e/base-drag.mp4

After recording — the same sequence with the destination cue on the row:

https://gh-file-drop-api-prod-mi5fy3sowv63ufte.pinglabs.workers.dev/f/4c317de36b39b3fc/head-drag.mp4

Compact row in dark mode — Settled to Active

Before — parentAfter — experiment
Before: compact row drag with repeated boundary action textAfter: compact row carries the Active badge while the boundary keeps its label

The light-mode screenshots and recordings compare parent 33e70c44a3 with head 0186450688, using matching fixtures and viewport. They are cropped to the Pinned and Active sections. The existing compact-row evidence remains representative; these fixes do not change its presentation.

Prepared with Codex.

Note

Simplify sidebar drag destination cues and add accessible Move to badge

  • Removes contextual hint text from SidebarDragBoundary, SidebarSectionHeader, and SidebarThreadRow during drag operations
  • Adds an accessible hidden "Move to" prefix to the destination badge shown only on the actively dragged row when it crosses into a different section (pinned, active, or settled)
  • Hides pin indicators, status, and hover-action slots during a drag, replacing them with the destination badge in both slim and card layouts
  • Snoozed is no longer offered as a drag destination; same-section reordering shows no badge
  • Updates thread-sidebar.md to reflect the new drag-and-drop behavior

Macroscope summarized 85bf36e.

Note

Low Risk UI-only sidebar drag affordances; drop logic and server commands are unchanged per the PR scope.

Overview Sidebar drag-and-drop feedback moves from repeated boundary hints onto the dragged row, with boundaries kept as section names plus highlight only.

Row behavior: When a drag crosses into a different section (pinned, active, or settled—not snoozed), the active row shows a → Pinned / Active / Settled badge with an accessible Move to prefix. Same-section reorder shows no badge. During drag, the pin control, status/time slot, and hover actions are hidden in favor of that badge.

Boundaries & placeholders: SidebarDragBoundary, SidebarSectionHeader, and empty-section placeholders no longer take contextual hints (e.g. “Drop to pin”); labels are static section names. dropSection is typed to exclude snoozed.

Docs: thread-sidebar.md describes the new cues and that reordering within a section does not show a destination badge.

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

Ggithub-actionsBotSep 4, 2026
Synced from GitHub

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

ℹ️ No successful main baseline artifact is available yet. This run establishes the initial measurement.

ProviderMetricMain baselineThis PRImpactPR ceiling
CodexTotal thread wire—13.6 KiB—15.1 KiB✅
CodexThread snapshot wire—7.0 KiB—7.3 KiB✅
CodexLive turn WebSocket wire—6.5 KiB—7.8 KiB✅
CodexLive turn WebSocket decoded—57.0 KiB—66.4 KiB✅
CodexLive turn messages—8—21✅
ClaudeTotal thread wire—13.6 KiB—15.1 KiB✅
ClaudeThread snapshot wire—7.1 KiB—7.3 KiB✅
ClaudeLive turn WebSocket wire—6.5 KiB—7.8 KiB✅
ClaudeLive turn WebSocket decoded—57.8 KiB—66.4 KiB✅
ClaudeLive turn messages—8—21✅

Baseline: unavailable · PR result: 85bf36e · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

  • Codex decoded thread snapshot: 113.9 KiB
  • Claude decoded thread snapshot: 114.6 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

MmacroscopeappBotSep 4, 2026
Synced from GitHub

Approvability

Verdict: Approved at 896abc2

Macroscope's review found this PR approvable — This is a contained sidebar UX refinement that changes only drag-time destination cues and temporarily hides existing row controls; the underlying drag targets, drop actions, and persistence remain unchanged. Documentation is updated alongside the isolated UI change, with no schema, infrastructure, security, billing, or static-analysis impact.

No code changes detected at 85bf36e. Prior analysis still applies.

You can add or adjust custom eligibility rules. Learn more.

CcursorBotSep 6, 2026
Synced from GitHub

Bugbot is paused — on-demand spend limit reached

Bugbot uses usage-based billing for this team and has hit its on-demand spend limit.

A team admin can raise the spend limit in the Cursor dashboard, or wait for the next billing cycle to continue.

Added 2 commits

  • feat(web): simplify sidebar drag destination cues0b41f7b
  • fix(web): hide row actions while dragging85bf36e