Skip to content

fix(vscode): improve history context menu sizing#10646

Merged
marius-kilocode merged 2 commits into
mainfrom
fix/vscode-history-context-menu
May 28, 2026
Merged

fix(vscode): improve history context menu sizing#10646
marius-kilocode merged 2 commits into
mainfrom
fix/vscode-history-context-menu

Conversation

@marius-kilocode
Copy link
Copy Markdown
Collaborator

@marius-kilocode marius-kilocode commented May 28, 2026

The local History actions menu is currently too compressed to comfortably scan and select its actions. Its rename, export, and delete entries inherit a compact menu treatment that makes this frequently used sidebar menu feel cramped and visually broken.

This gives only the local History session menu a readable minimum width, normal item spacing, and consistent hover and separator treatment. The adjustment is intentionally scoped to menu presentation and does not change session rename behavior or broaden the styling of unrelated context menus.

Before:
image
After:
image

Comment thread packages/kilo-vscode/webview-ui/src/styles/history.css Outdated
Comment thread packages/kilo-vscode/webview-ui/src/styles/history.css Outdated
@kilo-code-bot
Copy link
Copy Markdown
Contributor

kilo-code-bot Bot commented May 28, 2026

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Previously Flagged Issues — All Resolved

Both suggestions from the prior review were addressed in the latest commit:

File Issue Status
packages/kilo-vscode/webview-ui/src/styles/history.css Font size --font-size-small (11px) diverging from other context menus ✅ Fixed — line removed
packages/kilo-vscode/webview-ui/src/styles/history.css Redundant :hover pseudo-class alongside [data-highlighted] ✅ Fixed — :hover removed

The !important overrides on border and box-shadow were also cleaned up as a bonus.

Files Reviewed (3 files)
  • .changeset/history-context-menu-size.md - no issues
  • packages/kilo-vscode/webview-ui/src/components/history/SessionList.tsx - no issues
  • packages/kilo-vscode/webview-ui/src/styles/history.css - no issues

Reviewed by claude-4.6-sonnet-20260217 · 126,139 tokens

Review guidance: REVIEW.md from base branch main

@marius-kilocode marius-kilocode enabled auto-merge May 28, 2026 08:21
@marius-kilocode marius-kilocode merged commit aae8204 into main May 28, 2026
18 checks passed
@marius-kilocode marius-kilocode deleted the fix/vscode-history-context-menu branch May 28, 2026 08:32
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