Skip to content

fix(web): fix subagent row left border being cut off - #7207

Merged
maria-rcks merged 1 commit into
pingdotgg:mainfrom
sameerr03:fix/agent-row-left-edge
Aug 20, 2026
Merged

fix(web): fix subagent row left border being cut off#7207
maria-rcks merged 1 commit into
pingdotgg:mainfrom
sameerr03:fix/agent-row-left-edge

Conversation

@sameerr03

@sameerr03 sameerr03 commented Aug 16, 2026

Copy link
Copy Markdown
Contributor

What Changed

The subagent spawn summary no longer shifts outside the clipped chat timeline. Removing its redundant negative horizontal margin keeps the rounded left border visible while preserving the full-width row.

Why

The surrounding work-group layout already balances its negative margin with matching padding. The nested -mx-1 moved the subagent row four pixels past the timeline boundary, where horizontal overflow clipping hid its left edge.

UI Changes

Before After
Before: the subagent row's left edge is clipped After: the subagent row's complete rounded border is visible

Validation

  • vp fmt --check apps/web/src/components/chat/MessagesTimeline.tsx
  • vp lint apps/web/src/components/chat/MessagesTimeline.tsx
  • vp test run apps/web/src/components/chat/MessagesTimeline.test.tsx (20 tests)
  • git diff --check
  • Reproduced before the change and verified after hot reload in a real vp run dev thread

Checklist

  • This PR is small and focused
  • I explained what changed and why
  • I included before/after screenshots for the UI change
  • A video is not applicable because this is a static layout fix

Implemented with GPT-5.6-sol in the Codex harness.


Note

Low Risk
Single Tailwind class change on a chat UI row; no logic, API, or data impact.

Overview
Fixes a chat timeline layout bug where the subagent spawn summary row’s left rounded border was clipped.

The AgentSpawnCtaRow button no longer uses -mx-1. That extra negative horizontal margin stacked with the parent work-group’s -mx-1 / px-1 and pushed the row past the timeline’s overflow-x-clip boundary, hiding the left edge. The row still spans full width inside the group with the rest of its styling unchanged.

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

Note

Fix AgentSpawnCtaRow border visibility by removing negative horizontal margin

Removes the -mx-1 Tailwind class from the CTA button in MessagesTimeline.tsx, restoring the subagent row border visibility that was being clipped by the negative margin.

Macroscope summarized a75d43d.

@coderabbitai

coderabbitai Bot commented Aug 16, 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: f43748f2-16e6-4f46-837c-a844fad5d455

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:XS 0-9 changed lines (additions + deletions). labels Aug 16, 2026
@sameerr03 sameerr03 changed the title fix(web): keep subagent row border visible fix(web): fix subagent row left border being cut off Aug 16, 2026
@macroscopeapp

macroscopeapp Bot commented Aug 16, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved a75d43d

Single-line CSS change removing a negative margin class to fix border visibility. Pure styling fix with no runtime behavior impact.

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

@maria-rcks
maria-rcks enabled auto-merge (squash) August 20, 2026 20:35
@maria-rcks
maria-rcks merged commit 490f48e into pingdotgg:main Aug 20, 2026
18 checks passed
github-actions Bot added a commit to omarcresp/t3code-flake that referenced this pull request Aug 20, 2026
## What's Changed
* fix(web): retry failed thread bootstraps with a fresh id by @SunkenInTime in pingdotgg/t3code#7664
* fix(web): copy terminal selection instead of a blank clipboard by @sethwebster in pingdotgg/t3code#7678
* revert(web): restore sparse hourly usage breakdown by @t3-code[bot] in pingdotgg/t3code#7718
* fix(server): reconcile orphaned provider sessions by @t3-code[bot] in pingdotgg/t3code#7719
* fix(web): fix subagent row left border being cut off by @sameerr03 in pingdotgg/t3code#7207
* fix(web): unify composer control rounding by @kototok903 in pingdotgg/t3code#5957

## New Contributors
* @sethwebster made their first contribution in pingdotgg/t3code#7678
* @sameerr03 made their first contribution in pingdotgg/t3code#7207
* @kototok903 made their first contribution in pingdotgg/t3code#5957

**Full Changelog**: pingdotgg/t3code@v0.0.34-nightly.20260820.1142...v0.0.34-nightly.20260820.1146

Upstream release: https://github.com/pingdotgg/t3code/releases/tag/v0.0.34-nightly.20260820.1146
frankdavidcorona pushed a commit to frankdavidcorona/t3code that referenced this pull request Aug 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS 0-9 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.

2 participants