Merge pull request #28 from unboxed/fix-filter-on-search

Change to only show results of selected filter - may want to relook at
unboxed · Jul 30, 2024 · ccccf73 · ccccf73
2 parents d32b79a + 82a2175
commit ccccf73
Showing 1 changed file with 16 additions and 4 deletions.
diff --git a/src/App.js b/src/App.js
@@ -391,16 +391,27 @@ function App () {
         }
       }
 
-
       var tr = document.getElementById("applicationTable").getElementsByTagName("tr");
+
       for (let i = 0; i < tr.length; i++) {
         var td = tr[i].getElementsByTagName("td")[1];
+        var tdStat = tr[i].getElementsByTagName("td")[4];
         if (td) {
-          if (toDisplay.includes(td.innerText)) {
-            tr[i].style.display = "";
+          if (document.getElementById("filterSelect").value === "None") {
+            if (toDisplay.includes(td.innerText)) {
+              tr[i].style.display = "";
+            }
+            else {
+              tr[i].style.display = "none";
+            }
           }
           else {
-            tr[i].style.display = "none";
+            if (toDisplay.includes(td.innerText) && document.getElementById("filterSelect").value === tdStat.innerText) {
+              tr[i].style.display = "";
+            }
+            else {
+              tr[i].style.display = "none";
+            }
           }
         }
       }
@@ -412,6 +423,7 @@ function App () {
     for (let i = 0; i < tr.length; i++) {
       tr[i].style.display = "";
     }
+    document.getElementById('filterSelect').selectedIndex = 0;
   }
 
   const bindSearchToEnter = () => {