From 442bdd2220857b5f8f970f9e807869b099583e57 Mon Sep 17 00:00:00 2001
From: Harvey Tindall <hrfee@protonmail.ch>
Date: Tue, 25 Nov 2025 15:04:24 +0000
Subject: [PATCH] settings: leave groups opened/closed on advanced settings
 toggle

---
 ts/modules/settings.ts | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/ts/modules/settings.ts b/ts/modules/settings.ts
index dee4d0f..2370a68 100644
--- a/ts/modules/settings.ts
+++ b/ts/modules/settings.ts
@@ -1208,17 +1208,25 @@ export class settingsList {
         })
     };
 
+
+    private _query: string;
     // FIXME: Fix searching groups
     // FIXME: Search "About" & "User profiles", pseudo-search "User profiles" for things like "Ombi", "Referrals", etc.
     search = (query: string) => {
         query = query.toLowerCase().trim();
         // Make sure a blank search is detected when there's just whitespace.
         if (query.replace(/\s+/g, "") == "") query = "";
+        const noChange = query == this._query;
 
         let firstVisibleSection = "";
        
         // Close and hide all groups to start with
         for (const groupButton of Object.values(this._groupButtons)) {
+            // Leave these opened/closed if the query didn't change
+            // (this is overridden anyway if an actual search is happening,
+            // so we'll only do it if the search is blank, implying something else
+            // changed like advanced settings being enabled).
+            if (noChange && query == "") continue;
             groupButton.openCloseWithoutAnimation(false);
             groupButton.hidden = !(groupButton.group.toLowerCase().includes(query) ||
                                   groupButton.name.toLowerCase().includes(query) ||
@@ -1355,6 +1363,9 @@ export class settingsList {
                 this._visibleSection = "";
             }
         }
+
+        // We can use this later to tell if we should leave groups expanded/closed as they were.
+        this._query = query;
     }
 }