From 761b72892d174ef6cf7f3e16e8e3a33efd16bc0c Mon Sep 17 00:00:00 2001
From: Hannes Rudolph <hrudolph@gmail.com>
Date: Fri, 23 Jan 2026 20:09:05 -0700
Subject: [PATCH] fix: display default values when settings are reset to
 undefined

- Use settingDefaults fallback for checkbox checked state
- Use settingDefaults fallback for conditional rendering
- Use settingDefaults fallback for Select value

This ensures that when a setting is reset to undefined (meaning 'use default'),
the UI displays the actual default value instead of showing undefined/empty state.
---
 webview-ui/src/components/settings/BrowserSettings.tsx | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/webview-ui/src/components/settings/BrowserSettings.tsx b/webview-ui/src/components/settings/BrowserSettings.tsx
index 6484b55da8..7853661f1a 100644
--- a/webview-ui/src/components/settings/BrowserSettings.tsx
+++ b/webview-ui/src/components/settings/BrowserSettings.tsx
@@ -120,7 +120,7 @@ export const BrowserSettings = ({
 					label={t("settings:browser.enable.label")}>
 					<div className="flex items-center gap-1">
 						<VSCodeCheckbox
-							checked={browserToolEnabled}
+							checked={browserToolEnabled ?? settingDefaults.browserToolEnabled}
 							onChange={(e: any) => setCachedStateField("browserToolEnabled", e.target.checked)}>
 							<span className="font-medium">{t("settings:browser.enable.label")}</span>
 						</VSCodeCheckbox>
@@ -141,7 +141,7 @@ export const BrowserSettings = ({
 					</div>
 				</SearchableSetting>
 
-				{browserToolEnabled && (
+				{(browserToolEnabled ?? settingDefaults.browserToolEnabled) && (
 					<div className="flex flex-col gap-3 pl-3 border-l-2 border-vscode-button-background">
 						<SearchableSetting
 							settingId="browser-viewport"
@@ -156,7 +156,7 @@ export const BrowserSettings = ({
 								/>
 							</div>
 							<Select
-								value={browserViewportSize}
+								value={browserViewportSize ?? settingDefaults.browserViewportSize}
 								onValueChange={(value) => setCachedStateField("browserViewportSize", value)}>
 								<SelectTrigger className="w-full">
 									<SelectValue placeholder={t("settings:common.select")} />