From daddbd984fe7a737ea3e949bc56390028aa94a1b Mon Sep 17 00:00:00 2001
From: Kateryna Stetsenko <itcreativeusa@gmail.com>
Date: Fri, 13 Dec 2024 22:05:48 -0800
Subject: [PATCH] [WV-861]Refactored pagination section for consistency: -
 Wrapped pagination links in a form with class pagination-form. - Added
 onclick=displayLoadingBanner to pagination links to show the loading banner
 during navigation.

Refactored pagination section for consistency:
- Wrapped pagination links in a `<form>` with class `pagination-form`.
- Added `onclick="displayLoadingBanner()"` to pagination links to show the loading banner during navigation.
---
 templates/politician/politician_list.html | 33 ++++++++++++-----------
 1 file changed, 17 insertions(+), 16 deletions(-)

diff --git a/templates/politician/politician_list.html b/templates/politician/politician_list.html
index f29f3567e..d3e9e2d51 100644
--- a/templates/politician/politician_list.html
+++ b/templates/politician/politician_list.html
@@ -75,6 +75,19 @@ <h1>Politicians</h1>
     {% endif %}
     <input type="text" name="politician_search" id="politician_search_id" value="{{ politician_search }}" />
     <input id="politician_search_submit" type="submit" value="Search for Politician" />
+    <!-- Pagination Section -->
+    <form method="get" action="" class="pagination-form">
+    <div class="float-right">
+        {% if previous_page_url %}
+            <a href="{{ previous_page_url }}" onclick="displayLoadingBanner()">Previous page</a> |
+        {% endif %}
+        Page {{ current_page_number|add:1 }}
+        {% if next_page_url %}
+            | <a href="{{ next_page_url }}" onclick="displayLoadingBanner()">Next page</a>
+        {% endif %}
+    </div>
+    </form>
+    <!-- End of Pagination Section -->
     &nbsp;&nbsp;
     <label for="show_politicians_with_email_id">
       <input type="checkbox" name="show_politicians_with_email" id="show_politicians_with_email_id" value="1"
@@ -121,18 +134,6 @@ <h1>Politicians</h1>
              {% if exclude_politician_analysis_done %}checked{% endif %} /> Exclude Analyzed
     </label>
 </form>
-<!-- Pagination Section -->
-{% if not hide_pagination %}
-<span class="float-right">
-    {% if previous_page_url %}
-        <a href="{{ previous_page_url }}">Previous page</a> |
-    {% endif %}
-    Page {{ current_page_number|add:1 }}
-    {% if next_page_url %}
-        | <a href="{{ next_page_url }}">Next page</a>
-    {% endif %}
-</span>
-{% endif %}
 
 {% if politician_list %}
     <table class="table">
@@ -260,15 +261,15 @@ <h1>Politicians</h1>
         </tr>
     {% endfor %}
     </table>
-    <!-- Pagination Section -->
+   <!-- Pagination Section -->
 {% if not hide_pagination %}
 <span class="float-right">
     {% if previous_page_url %}
-        <a href="{{ previous_page_url }}">Previous page</a> |
+        <a href="{{ previous_page_url }}" onclick="displayLoadingBanner()">Previous page</a> |
     {% endif %}
     Page {{ current_page_number|add:1 }}
     {% if next_page_url %}
-        | <a href="{{ next_page_url }}">Next page</a>
+        | <a href="{{ next_page_url }}" onclick="displayLoadingBanner()">Next page</a>
     {% endif %}
 </span>
 {% endif %}
@@ -288,7 +289,7 @@ <h1>Politicians</h1>
 
     function clearSearch() {
         displayLoadingBanner();
-        $('#politician_search_id').value = '';
+        $('#politician_search_id').val('');
         $('form[name="state_code_form"]').submit();
     }