Skip to content

fix(vscode): preserve active editor pane - #12410

Merged
lambertjosh merged 2 commits into
mainfrom
investigate-pane-auto-resize-behavior
Jul 24, 2026
Merged

fix(vscode): preserve active editor pane#12410
lambertjosh merged 2 commits into
mainfrom
investigate-pane-auto-resize-behavior

Conversation

@lambertjosh

Copy link
Copy Markdown
Contributor

Summary

  • Open new Kilo chats in the currently selected editor group instead of creating and locking a group to the right.
  • Open and reveal Kilo Settings and Profile in the selected editor group rather than forcing column one.
  • Guard against reintroducing explicit group creation, locking, or text-editor-based column selection.

Why

The previous Open in Tab implementation inferred placement from visibleTextEditors, which excludes webview-only groups. In multi-pane layouts it could create one group, place the fresh Kilo chat in another, and leave the new group empty. Locking that group also caused later file opens to split around it. Settings separately forced column one, unexpectedly focusing and resizing an unrelated pane.

Using ViewColumn.Active preserves the user-selected pane and lets VS Code add each surface as a normal tab without rearranging the workspace.

@kilo-code-bot

kilo-code-bot Bot commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (4 files)
  • .changeset/steady-editor-tabs.md
  • packages/kilo-vscode/src/SettingsEditorProvider.ts
  • packages/kilo-vscode/src/extension.ts
  • packages/kilo-vscode/tests/unit/extension-arch.test.ts

Reviewed by claude-sonnet-5 · Input: 20 · Output: 4K · Cached: 411.2K

Review guidance: REVIEW.md from base branch main

@lambertjosh
lambertjosh merged commit d6b45d3 into main Jul 24, 2026
22 checks passed
@lambertjosh
lambertjosh deleted the investigate-pane-auto-resize-behavior branch July 24, 2026 23:22
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