Skip to content

fix(vscode): remove diff viewer corner artifacts - #11297

Merged
marius-kilocode merged 2 commits into
mainfrom
lime-supermarket
Jun 16, 2026
Merged

fix(vscode): remove diff viewer corner artifacts#11297
marius-kilocode merged 2 commits into
mainfrom
lime-supermarket

Conversation

@marius-kilocode

@marius-kilocode marius-kilocode commented Jun 16, 2026

Copy link
Copy Markdown
Collaborator

Virtualized diff rows are wrapped individually, which causes each accordion item to inherit first- and last-item corner radii. Those radii leave visible notches along file boundaries and the full-screen review divider.

Scope the accordion radius token to zero within inline and full-screen diff review surfaces. This keeps the file stack visually continuous while preserving rounded accordions elsewhere.

dyn-f82f2fa253abf5f26c5d43fd3b037c56 dyn-59b4ad50bc63661c24762054fbff8695

@kilo-code-bot

kilo-code-bot Bot commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (2 files)
  • .changeset/fix-diff-viewer-corners.md
  • packages/kilo-vscode/webview-ui/agent-manager/agent-manager.css
Previous Review Summary (commit 06da363)

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

Previous review (commit 06da363)

Status: No Issues Found | Recommendation: Merge

Files Reviewed (2 files)
  • .changeset/fix-diff-viewer-corners.md
  • packages/kilo-vscode/webview-ui/agent-manager/agent-manager.css

Reviewed by deepseek-v4-pro-20260423 · 133,298 tokens

Review guidance: REVIEW.md from base branch main

@marius-kilocode
marius-kilocode merged commit 33e26a1 into main Jun 16, 2026
21 checks passed
@marius-kilocode
marius-kilocode deleted the lime-supermarket branch June 16, 2026 10:34
t7tran pushed a commit to t7tran/kilocode that referenced this pull request Aug 14, 2026
fix(vscode): remove diff viewer corner artifacts
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