Remove redundant back-to-installed links from MCP/Plugin widgets

[joshspicer]

Summary

The global back arrow button (left of the search input) now handles exiting marketplace browse mode in MCP Servers and Plugins tabs, removing the redundant "← Back to installed servers/plugins" links that cluttered the UI.

Changes

• aiCustomizationManagementEditor.ts — createBackArrowButton and injectBackArrowIntoSearchRow now accept an optional click handler. For MCP/Plugin sections, the handler exits browse mode when active, otherwise navigates back to the overview.
• mcpListWidget.ts / pluginListWidget.ts — Removed the in-list backLink element, its DOM listeners, the backLink field, and the back-link height accounting in layout(). Added public isInBrowseMode() / exitBrowseMode() API for the editor to drive browse mode transitions.
• aiCustomizationManagement.css — Removed .mcp-back-link styles (24 lines).

Before / After

Before	After
Global back arrow + separate "← Back to installed" link	Global back arrow handles both actions

[github-actions]

Screenshot Changes

Base: 1b82a19e Current: e69dbdab

Changed (4)

chat/aiCustomizations/aiCustomizationManagementEditor/McpBrowseMode/Dark

Before	After

chat/aiCustomizations/aiCustomizationManagementEditor/PluginBrowseMode/Dark

Before	After

chat/aiCustomizations/aiCustomizationManagementEditor/McpBrowseMode/Light

Before	After

chat/aiCustomizations/aiCustomizationManagementEditor/PluginBrowseMode/Light

Before	After

[Copilot]

Pull request overview

This PR simplifies the MCP Servers and Plugins management UI by removing the redundant in-list “Back to installed …” links and reusing the global back arrow to exit marketplace browse mode when applicable.

Changes:

• Updated the management editor to allow the global back arrow to run a custom click handler for MCP/Plugin sections (exit browse mode if active, otherwise return to overview).
• Removed the in-widget back-link UI and related layout accounting from MCP/Plugin list widgets; added isInBrowseMode() / exitBrowseMode() APIs.
• Deleted now-unused CSS styles for the removed back-link element.

Show a summary per file

File	Description
src/vs/workbench/contrib/chat/browser/aiCustomization/aiCustomizationManagementEditor.ts	Adds optional click handler support for the global back arrow and wires it to exit browse mode in MCP/Plugins.
src/vs/workbench/contrib/chat/browser/aiCustomization/mcpListWidget.ts	Removes the in-list back link; exposes browse-mode query/exit APIs; adjusts list layout.
src/vs/workbench/contrib/chat/browser/aiCustomization/pluginListWidget.ts	Removes the in-list back link; exposes browse-mode query/exit APIs; adjusts list layout.
src/vs/workbench/contrib/chat/browser/aiCustomization/media/aiCustomizationManagement.css	Removes .mcp-back-link styling now that the element is gone.

Copilot's findings

Comments suppressed due to low confidence (1)

src/vs/workbench/contrib/chat/browser/aiCustomization/aiCustomizationManagementEditor.ts:787

• The back arrow button’s aria-label/hover text is always "Back to overview", but in MCP/Plugins the click handler can exit marketplace browse mode (back to installed) instead. This makes the accessible name/tooltip inaccurate when browse mode is active. Consider allowing the label/tooltip to be provided by the caller (or update it dynamically based on widget browse state) so it matches the actual action.

	private createBackArrowButton(onClick?: () => void): HTMLButtonElement {
		const button = $('button.section-back-arrow-button') as HTMLButtonElement;
		button.type = 'button';
		button.setAttribute('aria-label', localize('backToOverview', "Back to overview"));
		this.editorDisposables.add(this.hoverService.setupManagedHover(getDefaultHoverDelegate('element'), button, localize('backToOverviewTooltip', "Back to overview")));

• Files reviewed: 4/4 changed files
• Comments generated: 0