[Byntham]

Problem

In the desktop app, a two-finger trackpad swipe over the right panel's tab strip stalls whenever the cursor lands on the small gap between two tabs. Over a tab it scrolls fine, so the strip feels like it catches on the empty space between tabs.

Change

When the panel owns the desktop title bar, the tab bar is a window drag region (-webkit-app-region: drag). Each tab opts out with no-drag, but the 4px gaps between tabs do not. Electron does not deliver wheel events over a drag region, so the scroll stops there.

While the strip overflows, the tab strip now opts out of dragging as a whole. A strip that fits keeps its drag region, so the empty space after the tabs still moves the window. The change is one conditional class in RightPanelTabs.tsx. The web app has no drag regions and is unaffected.

Tradeoff: while tabs overflow, the bands above and below the tabs no longer drag the window. Wheel scrolling was already dead in those bands.

Scope and approval

A small fix for an obvious input bug in one component. It touches one line of markup and adds no setting or behavior beyond letting the existing strip scroll.

Verification

Real desktop app (Electron 44) built from each commit, on Xvfb with real X11 horizontal scroll input from xdotool, 19 tabs in an overflowing strip. Each clip flicks left 3 ticks with the cursor on a tab, then 3 ticks with the cursor on a gap.

	Cursor on a tab	Cursor on a gap
Before	strip moves 360px	strip moves 0px
After	strip moves 360px	strip moves 360px

Before

After

Frame sheets (2 fps)

Before	After

• With 2 tabs (no overflow), the strip computes app-region: none and the bar computes drag, so window dragging from the empty space is unchanged.
• vp run check:changed passes.
• No test added. This is Electron OS hit-testing, which a browser test cannot reproduce.
• Not checked: a real macOS trackpad (the report came from macOS; the mechanism is the same), and physically dragging the window, since Xvfb has no window manager.

Council review

Rounds: 1

Delegate	Round 1
Claude Fable 5.2	9/10
Claude Opus 5.5	9/10
GPT-6.1 Sol	10/10
Grok 4.7	10/10

Reviewed with sol-loop: 1 round with gpt-6.1-sol. Sol verdict: approved

Created with Claude Opus 5.5 in Claude Code (T3 Code).

🤖 Generated with Claude Code

[macroscopeapp]

Approvability

Verdict: Approved at 15b88d6

Macroscope's review found this PR approvable — The overflowing desktop tab strip now accepts scrolling across gaps between tabs rather than treating those gaps as window-drag regions. The change is confined to this UI interaction and leaves browser and non-overflow layouts unchanged.

Notes:

• All code in this push has already been reviewed. Approvability was decided on eligibility alone.

^{You can add or adjust custom eligibility rules. Learn more.}

[coderabbitai]

Walkthrough

The tab-list scroll area gains a no-drag class when the panel owns the desktop title bar and the tabs overflow.

Changes: Tab strip drag-region adjustment

Layer / File(s)	Summary
Conditional tab-list no-drag class apps/web/src/components/RightPanelTabs.tsx	When the panel owns the desktop title bar and the tabs overflow, the scroll area adds the no-drag class.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Bug fix

Suggested reviewers: juliusmarminge, maria-rcks

Merge Risk: Minimal · up to 7355c

This change lets trackpad scrolling work over gaps between right-panel tabs in the desktop app when the tabs overflow. It is a small, easily reversible styling change with no established defects, so merge risk is minimal.

Pre-merge checks: 4

Check name	Status	Explanation
Title check		The title describes the main change: trackpad scrolling over the right panel tab strip no longer stalls on gaps between tabs. It follows the conventional commit format and matches the raw summary, whi…
Description check		The description follows the template. It has Problem, Change, Scope and approval, and Verification sections. The Problem section explains the stall and the reproduction steps. The Change section expla…
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

Full details: Title check

Explanation

The title describes the main change: trackpad scrolling over the right panel tab strip no longer stalls on gaps between tabs. It follows the conventional commit format and matches the raw summary, which says the strip gets no-drag while it overflows so wheel events reach it. It is specific and concise, with no file lists or vague terms.

Full details: Description check

Explanation

The description follows the template. It has Problem, Change, Scope and approval, and Verification sections. The Problem section explains the stall and the reproduction steps. The Change section explains the drag-region cause and the fix, plus the tradeoff. The Scope section explains why no issue is linked and why the change is a fix. The Verification section gives before/after results, screenshots, a frame sheet, the commands run, and what was not checked. The description does not link a related issue, which the template asks for in the Scope section, but it does give a reason for omitting one. The template's agent-model line is present: the author names Claude Opus 5.5 in Claude Code.

Finishing Touches

Generate unit tests (beta)

• Create a new PR

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

[coderabbitai]

Additional comments: 1

apps/web/src/components/RightPanelTabs.tsx-1365-1365: 1

1365-1365: Major · Maintainability & Code Quality · Heavy lift

Unverified finding
Verification ran but could not confirm this finding. It is shown for review, not as a verified issue.

Add a regression test for gap scrolling.

Add a focused Electron test that scrolls over a tab gap when the strip overflows. Also check the transition back to a draggable strip when the tabs fit. The reported manual check will not catch a later regression.

As per coding guidelines, “Test behavior that can break,” including “the logic a bug fix changed.”

Source: Coding guidelines

Review info

Run configuration

• Source mode: Auto
• Repository: pingdotgg/t3code
• Branch: main
• File: .coderabbit.config.ts
• Commit: 5911884
• Layers: Repository TS config
• Review profile: CHILL
• Plan: Advanced
• Run ID: 79736bf4-b54f-4de8-b304-34e511b9597d

Commits

Reviewing files that changed from the base of the PR and between 5911884 and 15b88d6.

Files selected for processing: 1

• apps/web/src/components/RightPanelTabs.tsx

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 9 remain after this review.

[coderabbitai]

Additional comments: 1

apps/web/src/components/RightPanelTabs.tsx-1366-1366: 1

1366-1366: Minor · Maintainability & Code Quality · Quick win

Unverified finding
Verification ran but could not confirm this finding. It is shown for review, not as a verified issue.

Add a regression test for the overflow transition.

The PR adds no test for this state-dependent fix. A browser test can resize the tab list from fitting to overflowing and check that the desktop region changes to no-drag, then changes back when the tabs fit. This would fail without the new condition.

As per coding guidelines, “Logic, data, state, and protocol bugs get a failing test first.”

Source: Coding guidelines

Review info

Run configuration

• Source mode: Auto
• Repository: pingdotgg/t3code
• Branch: main
• File: .coderabbit.config.ts
• Commit: 73cbc6b
• Layers: Repository TS config
• Review profile: CHILL
• Plan: Advanced
• Run ID: 2ca175a4-4b47-4402-82c3-d58d9a7efbc9

Commits

Reviewing files that changed from the base of the PR and between 15b88d6 and 7355c94.

Files selected for processing: 1

• apps/web/src/components/RightPanelTabs.tsx

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 9 remain after this review.

[juliusmarminge]

Evidence

Electron 44.10.1 on Linux/Xvfb/Openbox, light theme, 1100×780 viewport, fresh isolated profiles and synthetic data on both commits. Open the right panel inline and add ten terminal tabs (980px content / 196px viewport). Reset scrollLeft to 400 before each trial, then send three native X11 left-wheel ticks over a tab (x=745) and two 4px gaps (x=698 and x=793), all at y=20. The overlay displays live measurements.

Before	After
73cbc6b1ed	7355c94c86
Native wheel recording	Native wheel recording
Both gaps: 400→400; zero renderer wheel events. Tab control: 400→160.	Both gaps and tab control: 400→40; three renderer wheel events each.

Setup script · Native wheel driver · Base measurements · Head measurements

Additional regression check: with one terminal tab and no overflow, dragging empty title-bar space moved the window from (0,0) to (120,80) on both commits. Before recording · After recording · Drag driver

Physical macOS trackpad gestures were not exercised; this proof uses native Linux/X11 horizontal wheel input.

Not shown:

• Physical macOS trackpad gestures were not exercised because this environment is a Linux VM; native X11 horizontal wheel input reproduced the reported Electron drag-region failure and demonstrated the fix.