From b4018e320b8598737d30d21e7ff122ae30fa9fcf Mon Sep 17 00:00:00 2001 From: Kyle Cutler <67761731+kycutler@users.noreply.github.com> Date: Thu, 19 Mar 2026 14:56:56 -0700 Subject: [PATCH] Never use simple browser on desktop (#303312) * Never use simple browser on desktop * Update extensions/simple-browser/package.nls.json Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> --- extensions/simple-browser/package.json | 14 ++++++++------ extensions/simple-browser/package.nls.json | 3 +-- extensions/simple-browser/src/extension.ts | 7 ------- 3 files changed, 9 insertions(+), 15 deletions(-) diff --git a/extensions/simple-browser/package.json b/extensions/simple-browser/package.json index 59a0c8677fe..bf4dfcb250a 100644 --- a/extensions/simple-browser/package.json +++ b/extensions/simple-browser/package.json @@ -42,6 +42,14 @@ "category": "Simple Browser" } ], + "menus": { + "commandPalette": [ + { + "command": "simpleBrowser.show", + "when": "isWeb" + } + ] + }, "configuration": [ { "title": "Simple Browser", @@ -51,12 +59,6 @@ "default": true, "title": "Focus Lock Indicator Enabled", "description": "%configuration.focusLockIndicator.enabled.description%" - }, - "simpleBrowser.useIntegratedBrowser": { - "type": "boolean", - "default": true, - "markdownDescription": "%configuration.useIntegratedBrowser.description%", - "scope": "application" } } } diff --git a/extensions/simple-browser/package.nls.json b/extensions/simple-browser/package.nls.json index 0b88b068fbc..496dc28dfdd 100644 --- a/extensions/simple-browser/package.nls.json +++ b/extensions/simple-browser/package.nls.json @@ -1,6 +1,5 @@ { "displayName": "Simple Browser", "description": "A very basic built-in webview for displaying web content.", - "configuration.focusLockIndicator.enabled.description": "Enable/disable the floating indicator that shows when focused in the simple browser.", - "configuration.useIntegratedBrowser.description": "When enabled, the `simpleBrowser.show` command will open URLs in the integrated browser instead of the Simple Browser webview. **Note:** This setting is only available on desktop." + "configuration.focusLockIndicator.enabled.description": "Enable/disable the floating indicator that shows when focused in the simple browser." } diff --git a/extensions/simple-browser/src/extension.ts b/extensions/simple-browser/src/extension.ts index 75ee87d4da7..ddcdc52b42d 100644 --- a/extensions/simple-browser/src/extension.ts +++ b/extensions/simple-browser/src/extension.ts @@ -15,7 +15,6 @@ declare class URL { const openApiCommand = 'simpleBrowser.api.open'; const showCommand = 'simpleBrowser.show'; const integratedBrowserCommand = 'workbench.action.browser.open'; -const useIntegratedBrowserSetting = 'simpleBrowser.useIntegratedBrowser'; const enabledHosts = new Set([ 'localhost', @@ -37,12 +36,6 @@ const openerId = 'simpleBrowser.open'; * Checks if the integrated browser should be used instead of the simple browser */ async function shouldUseIntegratedBrowser(): Promise { - const config = vscode.workspace.getConfiguration(); - if (!config.get(useIntegratedBrowserSetting, true)) { - return false; - } - - // Verify that the integrated browser command is available const commands = await vscode.commands.getCommands(true); return commands.includes(integratedBrowserCommand); }