From 1cf6adbc7fcfd9a3bbe2918f3dd9e551d0d10ac0 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Mon, 23 Feb 2026 12:26:45 +0000
Subject: [PATCH] feat(ui): always show Remove from DB button for DB-sourced
 settings with confirmation dialog

Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com>
---
 frontend/templates/settings.html | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/frontend/templates/settings.html b/frontend/templates/settings.html
index b35ccefe..30b3ffed 100644
--- a/frontend/templates/settings.html
+++ b/frontend/templates/settings.html
@@ -201,14 +201,14 @@
               </button>
               <button
                 type="button"
-                x-show="isDbOverride['{{ setting.key }}'] && formData['{{ setting.key }}'] === originalData['{{ setting.key }}']"
+                x-show="isDbOverride['{{ setting.key }}']"
                 x-transition
                 @click="revertSetting('{{ setting.key }}')"
                 :disabled="revertingKey === '{{ setting.key }}'"
                 class="px-3 py-1 text-sm bg-orange-500 text-white rounded-md hover:bg-orange-600 focus:outline-none focus:ring-2 focus:ring-offset-2 focus:ring-orange-400 disabled:opacity-50 disabled:cursor-not-allowed whitespace-nowrap"
                 title="Remove DB override and revert to environment variable or default"
               >
-                <span x-show="revertingKey !== '{{ setting.key }}'"><i class="fas fa-undo mr-1"></i>Back to ENV</span>
+                <span x-show="revertingKey !== '{{ setting.key }}'"><i class="fas fa-trash-alt mr-1"></i>Remove from DB</span>
                 <span x-show="revertingKey === '{{ setting.key }}'">Reverting…</span>
               </button>
             </div>
@@ -330,6 +330,10 @@ function settingsApp() {
     },
 
     async revertSetting(key) {
+      if (!confirm(`Remove '${key}' from the database?\n\nThe setting will revert to its environment variable or default value.`)) {
+        return;
+      }
+
       this.revertingKey = key;
       this.hideAlert();