Skip to content

Address feedback on default setting for simpleBrowser.useIntegratedBrowser#296776

Merged
jruales merged 2 commits intojruales/simplebrowser-integratedbrowser-default-truefrom
copilot/sub-pr-296775
Feb 22, 2026
Merged

Address feedback on default setting for simpleBrowser.useIntegratedBrowser#296776
jruales merged 2 commits intojruales/simplebrowser-integratedbrowser-default-truefrom
copilot/sub-pr-296775

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Feb 22, 2026

Aligns the hardcoded fallback value in extension.ts with the default: true set in package.json for simpleBrowser.useIntegratedBrowser.

  • extensions/simple-browser/src/extension.ts: Changed config.get<boolean>(useIntegratedBrowserSetting, false)config.get<boolean>(useIntegratedBrowserSetting, true) so the fallback matches the declared configuration default.

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

… true

Co-authored-by: jruales <1588988+jruales@users.noreply.github.com>
Copilot AI changed the title [WIP] Address feedback on default setting for simpleBrowser.useIntegratedBrowser Address feedback on default setting for simpleBrowser.useIntegratedBrowser Feb 22, 2026
@jruales jruales marked this pull request as ready for review February 22, 2026 01:09
@jruales jruales merged commit 72f99ae into jruales/simplebrowser-integratedbrowser-default-true Feb 22, 2026
2 of 3 checks passed
@jruales jruales deleted the copilot/sub-pr-296775 branch February 22, 2026 01:09
jruales added a commit that referenced this pull request Feb 23, 2026
* Update default value for useIntegratedBrowser setting

Changed default value of useIntegratedBrowser to true.

* Address feedback on default setting for `simpleBrowser.useIntegratedBrowser` (#296776)

* Initial plan

* Change default fallback for useIntegratedBrowserSetting from false to true

Co-authored-by: jruales <1588988+jruales@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: jruales <1588988+jruales@users.noreply.github.com>

* Remove "experimental" label from simpleBrowser.useIntegratedBrowser setting description (#296777)

* Initial plan

* Remove 'experimental' from useIntegratedBrowser setting description

Co-authored-by: jruales <1588988+jruales@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: jruales <1588988+jruales@users.noreply.github.com>

---------

Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
@vs-code-engineering vs-code-engineering Bot locked and limited conversation to collaborators Apr 8, 2026
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.

2 participants