Restore explicit control of spinner

Author: matt-bernhardt

app/assets/stylesheets/partials/_loading_spinner.scss

@@ -8,7 +8,7 @@
 }
 
 // Pagination overlay when loading
-[busy]:not([no-spinner]) {
+.spinner {
   position: relative;
   min-height: 400px;
   display: block;

app/javascript/loading_spinner.js

@@ -39,6 +39,9 @@ document.addEventListener('turbo:frame-render', function(event) {
       // console.log(`Updated tab input value to: ${queryParam}`);
     }
 
+    // Remove the spinner now that things are ready
+    document.getElementById('search-results').classList.remove('spinner');
+
     // Clear the pending action
     window.pendingFocusAction = null;
   };
@@ -59,6 +62,10 @@ document.addEventListener('click', function(event) {
     const clickedParams = new URLSearchParams(clickedElement.search);
     const newTab = clickedParams.get('tab');
 
+    // Throw the spinner on the search results immediately
+    document.getElementById('search-results').classList.add('spinner');
+
+    // Position the window at the top of the results
     window.scrollTo({ top: 0, behavior: 'smooth' });
 
     swapTabs(newTab);