Skip to content

fix(vscode): reduce chat prompt bottom spacing - #13603

Merged
marius-kilocode merged 3 commits into
mainfrom
golden-quill
Aug 31, 2026
Merged

fix(vscode): reduce chat prompt bottom spacing#13603
marius-kilocode merged 3 commits into
mainfrom
golden-quill

Conversation

@marius-kilocode

Copy link
Copy Markdown
Collaborator

What Problem This Solves

The empty space below the chat prompt grows with pane width, leaving an oversized bottom gap in Agent Manager.

Why This Change Was Made

Use the existing 6px minimum gutter for the bottom margin instead of the width-based gutter. Keep responsive side margins, readable width, top spacing, and session-dock behavior unchanged.

User Impact

Wide Agent Manager and editor chat panes have a compact bottom inset. Narrow sidebar chats and the New Worktree prompt retain their minimum spacing.

Evidence

  • All four Playwright prompt-spacing tests pass, including a new wide-pane check of the actual bottom gap.
  • Extension compile passed host/webview type checks, lint, and bundling. The Kilo marker guard passed.
  • Verified the real Agent Manager in isolated VS Code: at an 832px chat width, the bottom margin changed from 16.64px to 6px. Side spacing stayed unchanged.
  • Screenshots are cropped to the composer. The model selector is hidden for privacy in both captures; no model names or account data are included.

Before

Chat prompt before, with excess bottom spacing

After

Chat prompt after, with a compact six pixel bottom margin

Manual check: resize Agent Manager between narrow and wide layouts. Confirm the bottom inset stays small and the prompt controls remain inside the border.

@kilo-code-bot

kilo-code-bot Bot commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (21 files)
  • .changeset/tidy-prompt-bottom.md
  • packages/kilo-vscode/tests/chat-auto-scroll.spec.ts
  • packages/kilo-vscode/tests/prompt-spacing.spec.ts
  • packages/kilo-vscode/webview-ui/src/styles/chat-layout.css
  • 17 visual regression baselines under packages/kilo-docs/public/img/screenshot-tests/kilo-vscode/visual-regression/
Previous Review Summary (commit 287d073)

Current summary above is authoritative. Previous snapshots are kept for context only.

Previous review (commit 287d073)

Status: No Issues Found | Recommendation: Merge

Files Reviewed (3 files)
  • .changeset/tidy-prompt-bottom.md
  • packages/kilo-vscode/tests/prompt-spacing.spec.ts
  • packages/kilo-vscode/webview-ui/src/styles/chat-layout.css

Reviewed by grok-4.6 · Input: 47K · Output: 7.2K · Cached: 317.4K

Review guidance: REVIEW.md from base branch main

@marius-kilocode
marius-kilocode enabled auto-merge (squash) August 31, 2026 10:40
@WebReflection

Copy link
Copy Markdown
Contributor

@marius-kilocode I believe we need to update visual regressions/stories around this, otherwise looks good to me

@marius-kilocode
marius-kilocode merged commit 08757f7 into main Aug 31, 2026
26 checks passed
@marius-kilocode
marius-kilocode deleted the golden-quill branch August 31, 2026 11:17
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