Skip to content

Agents - move more actions to the right menu - #333962

Merged
Ladislau Szomoru (lszomoru) merged 1 commit into
mainfrom
lszomoru/residential-pike
Sep 2, 2026
Merged

Agents - move more actions to the right menu#333962
Ladislau Szomoru (lszomoru) merged 1 commit into
mainfrom
lszomoru/residential-pike

Conversation

@lszomoru

Copy link
Copy Markdown
Member

No description provided.

Copilot AI balanced review requested due to automatic review settings September 2, 2026 09:51

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

🟢 Approval recommended

The menu relocation is consistent, focused, and covered by updated tests and accessibility guidance.

Review tier: Balanced
Findings: None

What changed in this PR

Moves Agents Window diff and review actions into the right-side editor header menu.

Changes:

  • Relocates code review and diff actions.
  • Updates menu grouping, tests, comments, and accessibility help.
File Description
multiDiffEditor/​browser/​actions.ts Updates menu-location comments.
codeReview/​test/​browser/​codeReviewService.test.ts Tests code review’s new location.
codeReview/​browser/​codeReview.contributions.ts Moves code review to header actions.
changes/​test/​browser/​changesViewActions.test.ts Updates menu and accessibility expectations.
changes/​browser/​sessionsChangesAccessibilityHelp.ts Updates accessible menu guidance.
changes/​browser/​changesViewActions.ts Moves diff actions into header overflow.

💡 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 1b7f001 into main Sep 2, 2026
41 checks passed
@lszomoru
Ladislau Szomoru (lszomoru) deleted the lszomoru/residential-pike branch September 2, 2026 10:25
@vs-code-engineering vs-code-engineering Bot added this to the 1.137.0 milestone Sep 2, 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