Skip to content

fix(vscode): refine auto-merge split button - #13976

Merged
marius-kilocode merged 1 commit into
mainfrom
center-button-content-and-hover-animation
Sep 9, 2026
Merged

fix(vscode): refine auto-merge split button#13976
marius-kilocode merged 1 commit into
mainfrom
center-button-content-and-hover-animation

Conversation

@marius-kilocode

Copy link
Copy Markdown
Collaborator

What Problem This Solves

The Agent Manager auto-merge split button had an uneven action layout and a visually disconnected dropdown chevron. The chevron could also render a dark fill artifact instead of a clean stroke.

Why This Change Was Made

The primary merge action now fills the available footer column and centers its label. The dropdown trigger uses a consistent compact hit area, muted theme-aware colors, a seamless blue surface, and a dedicated right-segment hover state. The chevron is explicitly stroke-only, rotates when expanded, and respects reduced-motion preferences.

User Impact

Auto-merge controls now match the existing split-button interaction pattern while retaining the blue primary-action treatment. Hovering the right segment clearly identifies the dropdown without introducing a hard divider or an overly bright icon.

Evidence

Normal state:

Agent Manager auto-merge button in its normal state

Right-segment hover state:

Agent Manager auto-merge button with the dropdown segment hovered

Validation completed with bun run compile, the focused Agent Manager architecture test suite, and isolated VS Code visual checks for normal and right-segment hover states.

@kilo-code-bot

kilo-code-bot Bot commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (3 files)
  • .changeset/refine-pr-merge-split-button.md
  • packages/kilo-vscode/webview-ui/agent-manager/agent-manager.css
  • packages/kilo-vscode/webview-ui/agent-manager/pr/pr-panel.css

Reviewed by grok-4.6 · Input: 85.7K · Output: 14.7K · Cached: 1M

Review guidance: REVIEW.md from base branch main

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants