Skip to content

fix(web): remove negative margin from agent button - #6391

Closed
zacksmash wants to merge 2 commits into
pingdotgg:mainfrom
zacksmash:agent/remove-agent-button-negative-margin
Closed

fix(web): remove negative margin from agent button#6391
zacksmash wants to merge 2 commits into
pingdotgg:mainfrom
zacksmash:agent/remove-agent-button-negative-margin

Conversation

@zacksmash

@zacksmash zacksmash commented Aug 12, 2026

Copy link
Copy Markdown

Summary

  • remove the -mx-1 class from the agent spawn CTA button
  • keep the shared web and desktop renderer button aligned with the surrounding timeline content

Why

The negative horizontal margin extended the button beyond its containing content. Removing it preserves the existing internal padding and behavior while correcting the alignment.

User impact

The agent button now follows the container's horizontal bounds in both the web and desktop apps. Desktop bundles this web client, so no desktop-specific component change is needed. There is no interaction or behavior change.

Validation

  • git diff --check upstream/main...HEAD
  • pre-commit vp fmt

Generated by GPT-5.6 via Codex in T3 Code.

Note

Remove negative horizontal margin from agent spawn CTA button

Removes the -mx-1 Tailwind class from the AgentSpawnCtaRow button in MessagesTimeline.tsx, correcting its horizontal alignment within its container.

Macroscope summarized 17de002.


Note

Low Risk
Single-class CSS tweak with no logic or behavior changes.

Overview
Removes the -mx-1 Tailwind class from the AgentSpawnCtaRow button in MessagesTimeline.tsx so the agent spawn CTA no longer bleeds past the timeline content column horizontally.

Padding, click behavior, and status copy are unchanged; only layout alignment with sibling timeline rows is corrected for web and desktop (shared client).

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

@coderabbitai

coderabbitai Bot commented Aug 12, 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: 80276fe0-a2d4-4bde-a75b-8e1a3295b801

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 12, 2026
@zacksmash
zacksmash marked this pull request as ready for review August 12, 2026 22:50
@macroscopeapp

macroscopeapp Bot commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved 49b1643

This is a trivial CSS change removing a negative margin class (-mx-1) from a button. It affects only visual layout with no logic or runtime behavior changes.

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

@t3-code

t3-code Bot commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

closing as already addressed. Its exact one-line patch merged in #7207.

@t3-code t3-code Bot closed this Aug 20, 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.

1 participant