From e3ea0e2028b53d6359957da8ef7d2850a0ba5926 Mon Sep 17 00:00:00 2001
From: Joshua Laymon <joshlaymon@icloud.com>
Date: Sun, 24 Aug 2025 00:23:53 +0000
Subject: [PATCH] Update web/templates/search.html

---
 web/templates/search.html | 74 +++++++--------------------------------
 1 file changed, 12 insertions(+), 62 deletions(-)

diff --git a/web/templates/search.html b/web/templates/search.html
index 68c23ca..3e23ab3 100644
--- a/web/templates/search.html
+++ b/web/templates/search.html
@@ -401,83 +401,33 @@
   }
 </style>
 
+<!-- Save q + selected fields at the moment the search form submits -->
 <script>
 (function(){
-  // Try to find your search form; fallback to any GET form on the page
-  const form = document.querySelector('form.search-form') || document.querySelector('form[method="get"]');
-  const qInput = form && form.querySelector('input[name="q"]');
+  const form = document.querySelector('form.search-form');
+  if (!form) return;
 
-  // 1) Parse selected fields from the URL (most reliable, matches what server saw)
-  function fieldsFromURL(urlSearch) {
-    const params = new URLSearchParams(urlSearch || location.search);
-    const out = [];
-    // sel[field]=on → capture "field"
-    for (const [k, v] of params.entries()) {
-      if (k.startsWith('sel[') && k.endsWith(']') && (v === 'on' || v === 'true' || v === '1')) {
-        out.push(k.slice(4, -1));
-      }
-    }
-    return out;
-  }
+  const qInput = form.querySelector('input[name="q"]');
 
-  // 2) Parse from checkboxes as a fallback (in case query string isn’t available)
-  function fieldsFromCheckboxes(scope=document) {
+  function getSelectedFields(scope=form){
     const out = [];
     scope.querySelectorAll('input[type="checkbox"][name^="sel["]').forEach(cb => {
-      if (cb.checked) out.push(cb.name.slice(4, -1));
+      if (cb.checked) out.push(cb.name.slice(4, -1)); // sel[illustration] -> illustration
     });
-    return out;
+    return out.length ? out : ["subject"]; // backend default
   }
 
-  function currentQuery() {
-    return (qInput && qInput.value || '').trim();
-  }
-
-  function save(q, fields) {
-    // Default to ["subject"] if nothing explicitly selected (mirrors backend)
-    if (!fields || !fields.length) fields = ["subject"];
+  form.addEventListener('submit', () => {
+    const q = (qInput && qInput.value || '').trim();
+    const fields = getSelectedFields();
     try {
-      localStorage.setItem('lastSearchQ', q || '');
+      localStorage.setItem('lastSearchQ', q);
       localStorage.setItem('lastSearchFields', JSON.stringify(fields));
     } catch(_) {}
-  }
-
-  // Save on form submit
-  if (form) {
-    form.addEventListener('submit', () => {
-      const q = currentQuery();
-      // Prefer what’s headed to the server (checkboxes), but also merge URL if present
-      const f1 = fieldsFromCheckboxes(form);
-      const f2 = fieldsFromURL(); // in case the page was reloaded with selections
-      const fields = Array.from(new Set([...f1, ...f2]));
-      save(q, fields);
-    });
-  }
-
-  // Save when clicking a result link (user may not submit again before clicking)
-  document.addEventListener('click', (e) => {
-    const a = e.target.closest('a');
-    if (!a) return;
-    // Heuristic: links to entries look like /entry/<id> (adjust if your pattern differs)
-    const href = a.getAttribute('href') || '';
-    if (!/\/entry\/\d+/.test(href)) return;
-
-    const q = currentQuery();
-    // Most reliable: read from current URL (exactly what server saw for these results)
-    const fields = fieldsFromURL(location.search);
-    const fallback = fieldsFromCheckboxes(document);
-    const merged = Array.from(new Set([...(fields||[]), ...(fallback||[])]));
-    save(q, merged);
-  });
-
-  // Also save once on page load (covers pressing Enter without clicking results first)
-  document.addEventListener('DOMContentLoaded', () => {
-    const q = currentQuery();
-    const fields = fieldsFromURL(location.search);
-    if (q && fields.length) save(q, fields);
   });
 })();
 </script>
 
 
+
 {% endblock %}
\ No newline at end of file