Skip to content

chat - adjustments to the tools and server picker in input#272475

Merged
bpasero merged 5 commits intomainfrom
ben/identical-snipe
Oct 22, 2025
Merged

chat - adjustments to the tools and server picker in input#272475
bpasero merged 5 commits intomainfrom
ben/identical-snipe

Conversation

@bpasero
Copy link
Member

@bpasero bpasero commented Oct 21, 2025

@bpasero bpasero self-assigned this Oct 21, 2025
@bpasero bpasero added this to the On Deck milestone Oct 21, 2025
@bpasero bpasero marked this pull request as ready for review October 22, 2025 09:04
Copilot AI review requested due to automatic review settings October 22, 2025 09:04
@bpasero bpasero modified the milestones: On Deck, October 2025 Oct 22, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adjusts the positioning and visual presentation of MCP server and tool configuration actions in the chat input area. The changes move these actions from the ChatExecute menu to the ChatInput menu and update their visual styling from button-based indicators to smaller icon-based state indicators.

Key Changes:

  • Menu location changes from ChatExecute to ChatInput with updated order values
  • UI redesign replacing action buttons with absolute-positioned state indicators
  • CSS updates to support the new indicator-based design

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File Description
src/vs/workbench/contrib/mcp/browser/mcpCommands.ts Moves MCP server commands to ChatInput menu (order 101), replaces action button with state indicator, removes unused import
src/vs/workbench/contrib/chat/browser/media/chat.css Updates CSS for new state indicator positioning and removes button-based styling
src/vs/workbench/contrib/chat/browser/actions/chatToolActions.ts Moves configure tools action to ChatInput menu (order 100)

@bpasero bpasero merged commit d62af7e into main Oct 22, 2025
28 checks passed
@bpasero bpasero deleted the ben/identical-snipe branch October 22, 2025 16:00
@vs-code-engineering vs-code-engineering bot locked and limited conversation to collaborators Dec 6, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Half pill shaped warning for MCP unclear to me

3 participants