From 826605242bf0d87919746f6c6d175f896ea5cf08 Mon Sep 17 00:00:00 2001
From: derv82 <derv82@gmail.com>
Date: Wed, 6 Dec 2017 02:20:17 -0800
Subject: [PATCH] Disable next/prev search result buttons when loading

---
 popup/search-results.js | 11 ++++++++---
 1 file changed, 8 insertions(+), 3 deletions(-)

diff --git a/popup/search-results.js b/popup/search-results.js
index f4a05d1d..656a3980 100755
--- a/popup/search-results.js
+++ b/popup/search-results.js
@@ -60,13 +60,18 @@
     function setLoading(isLoading) {
       if (loading !== isLoading) {
         loading = isLoading;
-        if (loading) {
+
+        render(); // Refresh elements that depend on `loading` state.
+
+        if (isLoading) {
+          // Show spinner
           $('#searchResults').appendChild(
             $create(
               '.lds-spinner',
               new Array(12).fill($create('div')).map(e => e.cloneNode()))
           );
         } else {
+          // Hide spinner
           $.remove('#searchResults > .lds-spinner');
         }
       }
@@ -82,7 +87,7 @@
         createSearchResultNode(resultToDisplay);
       });
 
-      $('#searchResultsNav-prev').disabled = (currentDisplayedPage <= 1);
+      $('#searchResultsNav-prev').disabled = (currentDisplayedPage <= 1 || loading);
       $('#searchResultsNav-currentPage').textContent = currentDisplayedPage;
 
       let totalResultsCount = processedResults.length;
@@ -91,7 +96,7 @@
         totalResultsCount += DISPLAYED_RESULTS_PER_PAGE;
       }
       const totalPageCount = Math.ceil(Math.max(1, totalResultsCount / DISPLAYED_RESULTS_PER_PAGE));
-      $('#searchResultsNav-next').disabled = (currentDisplayedPage >= totalPageCount);
+      $('#searchResultsNav-next').disabled = (currentDisplayedPage >= totalPageCount || loading);
       $('#searchResultsNav-totalPages').textContent = totalPageCount;
     }