Repository navigation
Conversation
The ⌘ in the thread jump hints rendered in 10px SF Mono, where the glyph is too small to make out, and the badge floated over the card's middle, landing between the timestamp and the provider icon. The hint now renders as a Kbd in the row's status slot and takes up space there, so wider labels like Ctrl+1 shorten the project name instead of overlapping it. While it shows, the slot takes its status and hover controls out of view, the same way hover swaps in the row controls. The legacy sidebar keeps its inline badge with the readable font. Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
github-actions
Bot
added
vouch:unvouched
![macroscopeapp[bot]](https://avatars.githubusercontent.com/in/900172?s=60&v=4){kind=small}
ApprovabilityVerdict: Approved at Macroscope's review found this PR approvable — Sidebar jump hints become more readable and occupy the status slot instead of overlaying row content. The change is limited to temporary presentation, with keybindings and stored data unchanged. You can add or adjust custom eligibility rules. Learn more. |
|
No actionable comments were generated in the recent review.
 |
| Layer / File(s) | Summary |
|---|---|
Render hints in row status slots apps/web/src/components/sidebar/SidebarThreadRow.tsx |
Compact and card rows render jump hints in their status slots. The hint uses the Kbd component and is hidden while the snooze menu is open. The former placement after row content is removed. |
Estimated code review effort: 2 (Simple) | ~10 minutes
Change: Bug fix
Possibly related PRs
- pingdotgg/t3code#24825: This PR changes the same jump-hint visibility flow and
SidebarThreadRowintegration. - pingdotgg/t3code#25636: This PR changes the jump-hint visibility flow used by the sidebar rows.
Suggested labels: bugfix
Suggested reviewers: juliusmarminge, macroscopeapp
Merge Risk: 
3fcde
-
No actionable merge-blocking risk is established by this change.
Pre-merge checks: 
-
Check name Status Explanation Title check The title clearly identifies the sidebar jump-hint fix and follows the repository's conventional commit format. Description check The description includes complete Problem, Change, Scope and approval, and Verification sections. It explains the affected behavior, implementation, scope rationale, test results, screenshots, and rec… 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: Description check
-
Explanation
The description includes complete Problem, Change, Scope and approval, and Verification sections. It explains the affected behavior, implementation, scope rationale, test results, screenshots, and recording.
-
Finishing Touches
-
Generate unit tests (beta)
-
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Usage-based review receipt
- Mode: Continue automatically
- Reviewed files: 1
- Charged: $0.25
- View usage details
Note
This review was completed with usage-based billing: files reviewed beyond your plan's included limits are billed at $0.25/file. View usage-based billing.
|
Note Written by kept verification passed: blacksmith unverified: native macos/windows electron and its system fonts. these captures use linux chrome with real custom thread-jump bindings enabled for web; the default web bindings remain unchanged. no provider or transport code changed. uploaded video playback and seeking passed in chrome; the t3 preview player is unavailable on this vm. matching data at main |
Problem
Holding ⌘ on macOS shows thread jump hints, ⌘1 through ⌘9, in the sidebar. The ⌘ is too small to read, because 10px SF Mono draws it far smaller than the digit beside it. The badge also floats between the timestamp and the provider icon, so it lines up with neither.
To reproduce, open the desktop app on macOS 26.2 and hold ⌘ over the sidebar. The web app never shows these hints, since browsers keep ⌘1 through ⌘9 for switching tabs.
Change
Kbdcomponent. The model picker's ⌘1 through ⌘9 labels and the sidebar's command palette hint already use it. It renders the system font at 12px, where ⌘ is drawn full size. The sidebar badge was the only shortcut hint with its own monospace style.Ctrl+1labels on Windows and Linux and any remapped binding. The old badge needed an opaque pill because it sat on top of row content. In the slot it covers nothing, so the standardKbdfits without restyling.The one tradeoff is that while
⌘is held, the hint hides the row's status, such as "Working 2m", the same way hover does today. Hints only appear after a short hold, so quick⌘shortcuts never flash them. Releasing⌘brings the rows back to their normal state (timestamps or working statuses, and Settle/snooze on a hovered card).Scope and approval
This is a small fix for a visual bug in an existing feature, so I didn't open an issue first. Keybindings, contracts, and the rules for when hints appear are unchanged.
Verification
I ran the desktop dev build on macOS 26.2 against a copy of real data and captured everything through the Electron debug port.
ui-monospaceto 12px system font.Ctrl+1andCtrl+Shift+1labels, the slot grows to 43px and 78px. The project name or title truncates to make room, with no overlap at rest or hovered.vp linton both files and a webtsc --noEmitpass.Recording of hold ⌘, hover, and release, dark theme, 9 seconds
Idle → hold ⌘ and the hints replace the timestamps → hover card 1, then card 2, and the hints stay → release ⌘ and Settle and snooze return on the hovered card → move away. The capture doesn't include the pointer.
recording-hold-hover-release.mp4
Windows and Linux labels
Here's how it looks with the
Ctrl+1labels that Windows and Linux show. I swapped the label text in on macOS. The slot widens to fit, and the project name keeps its space.Done with Claude Opus 5.5 in Claude Code, via T3 Code.