From e388e2e85a1fd9d5068f30674fc951e711e87258 Mon Sep 17 00:00:00 2001
From: zhom <2717306+zhom@users.noreply.github.com>
Date: Sun, 29 Mar 2026 15:47:53 +0400
Subject: [PATCH] refactor: don't allow portable build to be set as the default
 browser

---
 src/components/settings-dialog.tsx | 60 +++++++++++++++---------------
 1 file changed, 31 insertions(+), 29 deletions(-)

diff --git a/src/components/settings-dialog.tsx b/src/components/settings-dialog.tsx
index a30eb79..360c7dc 100644
--- a/src/components/settings-dialog.tsx
+++ b/src/components/settings-dialog.tsx
@@ -773,36 +773,38 @@ export function SettingsDialog({
             </p>
           </div>
 
-          {/* Default Browser Section */}
-          <div className="space-y-4">
-            <div className="flex justify-between items-center">
-              <Label className="text-base font-medium">Default Browser</Label>
-              <Badge variant={isDefaultBrowser ? "default" : "secondary"}>
-                {isDefaultBrowser ? "Active" : "Inactive"}
-              </Badge>
+          {/* Default Browser Section - hidden in portable mode */}
+          {!systemInfo?.portable && (
+            <div className="space-y-4">
+              <div className="flex justify-between items-center">
+                <Label className="text-base font-medium">Default Browser</Label>
+                <Badge variant={isDefaultBrowser ? "default" : "secondary"}>
+                  {isDefaultBrowser ? "Active" : "Inactive"}
+                </Badge>
+              </div>
+
+              <LoadingButton
+                isLoading={isSettingDefault}
+                onClick={() => {
+                  handleSetDefaultBrowser().catch((err: unknown) => {
+                    console.error(err);
+                  });
+                }}
+                disabled={isDefaultBrowser}
+                variant={isDefaultBrowser ? "outline" : "default"}
+                className="w-full"
+              >
+                {isDefaultBrowser
+                  ? "Already Default Browser"
+                  : "Set as Default Browser"}
+              </LoadingButton>
+
+              <p className="text-xs text-muted-foreground">
+                When set as default, Donut Browser will handle web links and
+                allow you to choose which profile to use.
+              </p>
             </div>
-
-            <LoadingButton
-              isLoading={isSettingDefault}
-              onClick={() => {
-                handleSetDefaultBrowser().catch((err: unknown) => {
-                  console.error(err);
-                });
-              }}
-              disabled={isDefaultBrowser}
-              variant={isDefaultBrowser ? "outline" : "default"}
-              className="w-full"
-            >
-              {isDefaultBrowser
-                ? "Already Default Browser"
-                : "Set as Default Browser"}
-            </LoadingButton>
-
-            <p className="text-xs text-muted-foreground">
-              When set as default, Donut Browser will handle web links and allow
-              you to choose which profile to use.
-            </p>
-          </div>
+          )}
 
           {/* Permissions Section - Only show on macOS */}
           {isMacOS && (