Skip to content

Agents - move the toggle details action - #333773

Merged
Ladislau Szomoru (lszomoru) merged 1 commit into
mainfrom
lszomoru/legitimate-porpoise
Sep 1, 2026
Merged

Agents - move the toggle details action#333773
Ladislau Szomoru (lszomoru) merged 1 commit into
mainfrom
lszomoru/legitimate-porpoise

Conversation

@lszomoru

Copy link
Copy Markdown
Member

No description provided.

Copilot AI balanced review requested due to automatic review settings September 1, 2026 14:47
@vs-code-engineering

Copy link
Copy Markdown
Contributor

📬 CODENOTIFY

The following users are being notified based on files changed in this PR:

Benjamin Christopher Simmonds (@benibenj)

Matched files:

  • src/vs/sessions/contrib/layout/browser/singlePane/singlePaneExistingSessionStrategy.ts
  • src/vs/sessions/contrib/layout/test/browser/desktopSessionLayoutController.test.ts

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

Review tier: Balanced
Findings: None

What changed in this PR

Moves the Agents Window’s Toggle Details action into the shared editor-title layout toolbar.

Changes:

  • Targets MenuId.EditorTitleLayout instead of the Sessions header menu.
  • Updates the focused menu-registration test.
File Description
singlePaneExistingSessionStrategy.ts Moves the Toggle Details menu contribution.
desktopSessionLayoutController.test.ts Verifies the new menu placement.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@lszomoru
Ladislau Szomoru (lszomoru) merged commit 4735247 into main Sep 1, 2026
41 checks passed
@lszomoru
Ladislau Szomoru (lszomoru) deleted the lszomoru/legitimate-porpoise branch September 1, 2026 15:00
@vs-code-engineering vs-code-engineering Bot added this to the 1.137.0 milestone Sep 1, 2026
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.

3 participants