Skip to content

fix(vscode): remove duplicate sidebar border - #12836

Merged
Drixled merged 1 commit into
mainfrom
Drixled/burrfish
Aug 4, 2026
Merged

fix(vscode): remove duplicate sidebar border#12836
Drixled merged 1 commit into
mainfrom
Drixled/burrfish

Conversation

@Drixled

@Drixled Drixled commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Summary

Remove the webview-provided right border from the Kilo Code container so VS Code is the sole owner of the sidebar divider. This avoids the doubled vertical edge while retaining the full-height container sizing.

Visual comparison

view

@kilo-code-bot

kilo-code-bot Bot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (2 files)
  • packages/kilo-vscode/src/KiloProvider.ts
  • .changeset/clean-vscode-sidebar-edge.md

The removed inline styles are safe: display: flex; flex-direction: column is already provided by the webview's own chat-layout.css, the duplicate height: 100% was shadowed by height: 100vh in the same rule, and dropping the webview's border-right correctly leaves VS Code as the sole owner of the sidebar divider. Full-height sizing is retained via height: 100vh, and the changeset is present with a user-facing description.


Reviewed by kimi-k3 · Input: 70.7K · Output: 5.2K · Cached: 265K

Review guidance: REVIEW.md from base branch main

@Drixled Drixled closed this Aug 4, 2026
@Drixled
Drixled deleted the Drixled/burrfish branch August 4, 2026 01:03
@Drixled
Drixled restored the Drixled/burrfish branch August 4, 2026 01:03
@Drixled Drixled reopened this Aug 4, 2026
@Drixled
Drixled requested a review from marius-kilocode August 4, 2026 01:04
@Drixled
Drixled merged commit fab0e96 into main Aug 4, 2026
46 checks passed
@Drixled
Drixled deleted the Drixled/burrfish branch August 4, 2026 05:47
LudwigSolutionsAI pushed a commit to LudwigSolutionsAI/kilocode that referenced this pull request Aug 6, 2026
t7tran pushed a commit to t7tran/kilocode that referenced this pull request Aug 14, 2026
fix(vscode): remove duplicate sidebar border
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