Skip to content

fix(web): thread details card gives titles room to read - #16746

Merged
t3dotgg merged 3 commits into
mainfrom
t3/details-card-readability
Oct 7, 2026
Merged

t3dotgg merged 3 commits into
mainfrom
t3/details-card-readability

Conversation

@t3dotgg

@t3dotgg t3dotgg commented Oct 7, 2026 •

Copy link
Copy Markdown
Member

The thread details card got too crowded. Lineage rows showed both a timer and the word "Running", and PR rows put the number, checks, a chevron, and an action next to the title. On a 280px card, titles were cut to a few characters.

Now the title gets the space:

  • Lineage: each row shows one item on the right. An agent shows only a short time ("45s", "1m", "1.5h", "14h"). The icon dot and the "N running" header already show the status. Forks, parents, queued and failed rows still show a status word, and "Failed" is red. The hover card keeps the exact time.
  • PR row: when the PR has checks, an action, or a watch, the row splits into two lines. Line 1 is the full-width title. Line 2 has #number, checks ("2 failed" or "1/3", no chevron), the watch eye, and the action (Resolve, Ready, Fix or Merge). The second line sits tight under its own title, with a gap before the next PR, so stacked PRs read as pairs. PRs with nothing to show stay on one line.
Before After
Lineage before Lineage after
PR rows before PR rows after

Watched PR: the eye sits on line 2 after the number and checks. Clicking it stops the watch.

Watched PR row

The screenshots come from a local dev server with real data: four running delegated subagents, and a thread linked to two open PRs that have conflicts.

Web and desktop only. Mobile has its own agents sheet and does not change.

Reviewed with sol-loop: 2 rounds with GPT-6.1 Sol on high.

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

🤖 Generated with Claude Code

t3dotgg and others added 2 commits October 6, 2026 23:32
Lineage rows show one trailing item with a compact time, and the PR row
moves its number, checks, watch, and action to a second line.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:L 100-499 changed lines (additions + deletions). labels Oct 7, 2026
@juliusmarminge juliusmarminge added the macroscope-review Opt PRs made by unvouched contributors in for Macroscope review. Vouched contributors auto-reviews label Oct 7, 2026
macroscopeapp[bot]
macroscopeapp Bot previously approved these changes Oct 7, 2026
@macroscopeapp

macroscopeapp Bot commented Oct 7, 2026 •

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved at 49e46cf

Macroscope's review found this PR approvable — This PR is a localized thread-details UI refinement: it gives pull-request titles more room and compacts related status information without changing the underlying actions, APIs, or data flow. Added formatting behavior is covered by focused tests, and no sensitive, deployment, default-setting, or static-analysis configuration is affected.

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

@coderabbitai

coderabbitai Bot commented Oct 7, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Path: .coderabbit.config.ts
  • Review profile: CHILL
  • Plan: Team
  • Run ID: fa233f3d-4e28-44de-8fd8-ec8399d76666
📥 Commits

Reviewing files that changed from the base of the PR and between a708981 and 49e46cf.

📒 Files selected for processing (1)
  • apps/web/src/components/chat/ThreadDetailsPrRow.tsx
🚧 Files skipped from review as they are similar to previous changes (1)
  • apps/web/src/components/chat/ThreadDetailsPrRow.tsx

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


📝 Walkthrough

Walkthrough

The changes add compact elapsed-time formatting for agent rows, revise pull request detail row layouts and controls, and display failure counts in the pull request checks trigger.

Changes

Agent elapsed-time display

Layer / File(s) Summary
Compact elapsed-time formatting
apps/web/src/components/chat/AgentElapsed.tsx, apps/web/src/components/chat/AgentElapsed.test.ts
Adds an optional compact format that displays seconds, whole minutes, or tenths of an hour. Tests cover values from 45 seconds to 14 hours.
Agent relationship row display
apps/web/src/components/chat/ThreadRelationshipsControl.tsx, apps/web/src/components/chat/ThreadRelationshipsControl.agents.test.tsx
Agent rows show compact elapsed time when available and the status is not failed or errored. Other eligible rows show a status label, with failed and errored labels styled as destructive.

Pull request details

Layer / File(s) Summary
Pull request detail row layout
apps/web/src/components/chat/ThreadDetailsPrRow.tsx, apps/web/src/components/chat/ThreadDetailsControl.tsx, apps/web/src/components/chat/threadDetailsPanelStyles.ts
Moves the pull request title to its own line and places eligible checks, the watch control, and an action on the second line. Adds meta-action styling and a primary tone.
Pull request check-count trigger
apps/web/src/components/pullRequest/PullRequestChecksPopover.tsx
Displays the number of failed or cancelled checks when the running-count display does not apply. Removes the chevron from the count variant.

Priority: ⬇️ Low

Estimated code review effort: 3 (Moderate) | ~20 minutes

Change: Bug fix

Merge Risk: ⚪ Minimal · up to 49e46

Pull request rows retain a readable title and show available controls; no issue identified here needs resolution before merging.

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title clearly summarizes the main change: giving titles more room in the thread details card.
Description check ✅ Passed The description explains the problem and changes, states the platform scope, and includes before-and-after screenshots and the authoring model and harness. It does not state focused test results or pr…
✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Comment @coderabbitai help to get the list of available commands.

@github-actions

github-actions Bot commented Oct 7, 2026 •

Copy link
Copy Markdown
Contributor

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

ℹ️ The exact PR base did not have a successful artifact. Baseline uses the latest successful main measurement shown below.

Provider Metric Main baseline This PR Impact PR ceiling
Codex Total thread wire 4.9 KiB 5.0 KiB +23 B (+0.5%) 6.8 KiB ✅
Codex Thread snapshot wire 3.8 KiB 3.8 KiB 0 B (0.0%) 4.9 KiB ✅
Codex Live turn WebSocket wire 1.2 KiB 1.2 KiB +23 B (+1.9%) 2.0 KiB ✅
Codex Live turn WebSocket decoded 20.8 KiB 20.9 KiB +41 B (+0.2%) 29.3 KiB ✅
Codex Live turn messages 1 2 +1 (+100.0%) 8 ✅
Claude Total thread wire 5.0 KiB 5.0 KiB −41 B (−0.8%) 6.8 KiB ✅
Claude Thread snapshot wire 3.8 KiB 3.8 KiB 0 B (0.0%) 4.9 KiB ✅
Claude Live turn WebSocket wire 1.2 KiB 1.2 KiB −41 B (−3.3%) 2.0 KiB ✅
Claude Live turn WebSocket decoded 21.2 KiB 21.2 KiB −41 B (−0.2%) 29.3 KiB ✅
Claude Live turn messages 2 1 −1 (−50.0%) 8 ✅

Baseline: 250e052 · PR result: 49e46cf · 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: 108.5 KiB
  • Claude decoded thread snapshot: 108.8 KiB

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

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@macroscopeapp
macroscopeapp Bot dismissed their stale review October 7, 2026 06:44

Dismissing prior approval to re-evaluate 49e46cf

@t3dotgg
t3dotgg merged commit f60367b into main Oct 7, 2026
32 of 33 checks passed
@t3dotgg
t3dotgg deleted the t3/details-card-readability branch October 7, 2026 06:49
github-actions Bot added a commit to omarcresp/t3code-flake that referenced this pull request Oct 7, 2026
## What's Changed
* docs: connect Claude Code, Codex, ChatGPT and bots over MCP by @juliusmarminge in pingdotgg/t3code#16741
* fix(web): thread details card gives titles room to read by @t3dotgg in pingdotgg/t3code#16746
* fix(mcp): agent HTML pages stop painting slab backgrounds by @t3dotgg in pingdotgg/t3code#16752
* fix: composer picks up new project skills without a server restart by @juliusmarminge in pingdotgg/t3code#16750
* feat(server): run a project action when a worktree thread settles by @t3dotgg in pingdotgg/t3code#16290
* feat(web): old Claude threads compact on send instead of stacking notices by @t3dotgg in pingdotgg/t3code#16631
* fix(server): settled threads stop polling their pull requests by @t3dotgg in pingdotgg/t3code#16762
* fix(server): stop storing tool image bytes no client reads by @derektrimm in pingdotgg/t3code#16652
* fix(server): status refresh no longer pegs CPU in repos with thousands of untracked files by @t3dotgg in pingdotgg/t3code#16771
* perf(server): background branch lookups share one GitHub query per sweep by @t3dotgg in pingdotgg/t3code#16760
* fix(server): threads settle as soon as a client sees their PR merge by @t3dotgg in pingdotgg/t3code#16761
* feat(server,web,mobile): agents see snooze state and link to threads by @t3dotgg in pingdotgg/t3code#16782


**Full Changelog**: pingdotgg/t3code@v0.0.46-nightly.20261007.2761...v0.0.46-nightly.20261007.2774

Upstream release: https://github.com/pingdotgg/t3code/releases/tag/v0.0.46-nightly.20261007.2774
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

macroscope-review Opt PRs made by unvouched contributors in for Macroscope review. Vouched contributors auto-reviews size:L 100-499 changed lines (additions + deletions). vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants