@bobfrankston/rmfmail 1.2.248 → 1.2.251

package/client/app.js

@@ -2,7 +2,7 @@
  * mailx client entry point.
  * Wires together all UI components and WebSocket connection.
  */
-import { initFolderTree, refreshFolderTree, updateFolderCounts, setFolderSynced, getFolderSynced, setOutboxTotal, highlightFolder } from "./components/folder-tree.js";
+import { initFolderTree, refreshFolderTree, updateFolderCounts, setFolderSynced, getFolderSynced, setOutboxTotal, highlightFolder, setFolderFilter } from "./components/folder-tree.js";
 import { initMessageList, loadMessages, loadUnifiedInbox, loadSearchResults, reloadCurrentFolder, clearSearchMode, getSelectedMessages, markBodiesCached, getCurrentFocused, releaseFocus, removeMessagesAndReconcile, setRowFlagged, scrollFocusedIntoView, refreshPriorityIndex, revealMessage, getCurrentView, selectAllVisible, exitMultiSelect, getDateBasis, setDateBasis } from "./components/message-list.js";
 import { seenOf, flaggedOf, draftOf, setSeen, setFlagged } from "@bobfrankston/mailx-types";
 import { initTabs, setActiveView as setActiveTabView, openTab } from "./components/tabs.js";
@@ -2975,46 +2975,33 @@ document.getElementById("search-server-too")?.addEventListener("change", rerunAc
 document.getElementById("search-include-trash")?.addEventListener("change", rerunActiveSearch);
 // Message state handles move/delete — no manual event listener needed
 // ── Folder filter ──
+// The matching itself lives in folder-tree.ts (setFolderFilter), which filters
+// the folder DATA and re-renders. The old version here hid rendered rows,
+// which could only ever find folders that happened to be expanded — every
+// folder under a collapsed parent, and every folder of a collapsed account,
+// was invisible to it (Bob 2026-08-11: "folder search should not be case
+// sensitive" — the misses looked like a casing problem).
 const ftFilterInput = document.getElementById("ft-filter-input");
 if (ftFilterInput) {
-    ftFilterInput.addEventListener("input", () => {
-        const query = ftFilterInput.value.toLowerCase();
-        const tree = document.getElementById("folder-tree");
-        if (!tree)
-            return;
-        if (!query) {
-            // Clear filter — show everything
-            tree.querySelectorAll(".ft-filter-hidden").forEach(el => el.classList.remove("ft-filter-hidden"));
-            return;
-        }
-        // Hide all folders first, then show matches + their parent accounts
-        const folders = tree.querySelectorAll(".ft-folder");
-        const accounts = tree.querySelectorAll(".ft-account");
-        for (const acct of accounts)
-            acct.classList.add("ft-filter-hidden");
-        for (const f of folders)
-            f.classList.add("ft-filter-hidden");
-        for (const f of folders) {
-            const name = f.querySelector(".ft-folder-name")?.textContent?.toLowerCase() || "";
-            if (name.includes(query)) {
-                f.classList.remove("ft-filter-hidden");
-                // Show parent account
-                const acct = f.closest(".ft-account");
-                if (acct)
-                    acct.classList.remove("ft-filter-hidden");
-            }
-        }
-        // Also show unified inbox if it matches
-        const unified = tree.querySelector(".ft-unified");
-        if (unified) {
-            const text = unified.textContent?.toLowerCase() || "";
-            unified.classList.toggle("ft-filter-hidden", !text.includes(query));
-        }
-    });
+    // Re-render hits the folder API per account, so coalesce keystrokes.
+    let ftFilterTimer = null;
+    const FT_FILTER_DEBOUNCE_MS = 120;
+    const applyFolderFilter = (immediate) => {
+        if (ftFilterTimer) {
+            clearTimeout(ftFilterTimer);
+            ftFilterTimer = null;
+        }
+        const run = () => setFolderFilter(ftFilterInput.value);
+        if (immediate)
+            run();
+        else
+            ftFilterTimer = setTimeout(run, FT_FILTER_DEBOUNCE_MS);
+    };
+    ftFilterInput.addEventListener("input", () => applyFolderFilter(false));
     ftFilterInput.addEventListener("keydown", (e) => {
         if (e.key === "Escape") {
             ftFilterInput.value = "";
-            ftFilterInput.dispatchEvent(new Event("input"));
+            applyFolderFilter(true);
         }
     });
 }