Skip to content

Make MCP Add Server button match plugins + button#302270

Merged
joshspicer merged 2 commits intomicrosoft:mainfrom
joshspicer:copilot/administrative-butterfly
Mar 16, 2026
Merged

Make MCP Add Server button match plugins + button#302270
joshspicer merged 2 commits intomicrosoft:mainfrom
joshspicer:copilot/administrative-butterfly

Conversation

@joshspicer
Copy link
Copy Markdown
Member

Summary

  • restyle the MCP "Add Server" button in AI Customization to match the plugins icon-only "+" button
  • switch to secondary icon-only button treatment with hover tooltip
  • keep the existing Add Configuration command wiring intact while adjusting browse-mode visibility handling

Testing

  • npm run compile-check-ts-native

Copilot AI review requested due to automatic review settings March 16, 2026 21:15
@joshspicer joshspicer enabled auto-merge (squash) March 16, 2026 21:16
@vs-code-engineering vs-code-engineering bot added this to the 1.112.0 milestone Mar 16, 2026
Copy link
Copy Markdown
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

Restyles the MCP “Add Server” control in the AI Customization MCP list to align with the icon-only “+” button pattern used elsewhere (e.g., plugins), while keeping the existing command wiring and adjusting browse-mode UI visibility.

Changes:

  • Switch “Add Server” from a text button to a secondary, icon-only button with hover tooltip.
  • Add managed hover tooltip wiring for the icon-only button.
  • Update browse-mode visibility logic to hide the button element directly.

You can also share your feedback on Copilot code review. Take the survey.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@joshspicer joshspicer merged commit a43982e into microsoft:main Mar 16, 2026
18 checks passed
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