From 57a8dd7c3f2a2e7ca5a107af38e7c8b2ac3c5eab Mon Sep 17 00:00:00 2001
From: ikkemaniac <ikkemaniac@localhost>
Date: Mon, 26 Sep 2022 23:31:45 +0200
Subject: [PATCH 1/2] fix: add pagination classes to nav section

---
 .../templates/django_tables2/bootstrap.html        | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/django_tables2/templates/django_tables2/bootstrap.html b/django_tables2/templates/django_tables2/bootstrap.html
index feaf220c..7a724fab 100644
--- a/django_tables2/templates/django_tables2/bootstrap.html
+++ b/django_tables2/templates/django_tables2/bootstrap.html
@@ -60,8 +60,8 @@
             <ul class="pagination">
             {% if table.page.has_previous %}
                 {% block pagination.previous %}
-                    <li class="previous">
-                        <a href="{% querystring table.prefixed_page_field=table.page.previous_page_number %}">
+                    <li class="page-item previous">
+                        <a class="page-link" href="{% querystring table.prefixed_page_field=table.page.previous_page_number %}">
                             <span aria-hidden="true">&laquo;</span>
                             {% trans 'previous' %}
                         </a>
@@ -71,11 +71,11 @@
             {% if table.page.has_previous or table.page.has_next %}
                 {% block pagination.range %}
                     {% for p in table.page|table_page_range:table.paginator %}
-                        <li {% if p == table.page.number %}class="active"{% endif %}>
+                        <li  class="page-item{% if p == table.page.number %} active{% endif %}">
                             {% if p == '...' %}
-                                <a href="#">{{ p }}</a>
+                                <a class="page-link" href="#">{{ p }}</a>
                             {% else %}
-                                <a href="{% querystring table.prefixed_page_field=p %}">
+                                <a class="page-link" href="{% querystring table.prefixed_page_field=p %}">
                                     {{ p }}
                                 </a>
                             {% endif %}
@@ -86,8 +86,8 @@
 
             {% if table.page.has_next %}
                 {% block pagination.next %}
-                <li class="next">
-                    <a href="{% querystring table.prefixed_page_field=table.page.next_page_number %}">
+                <li class="next page-item">
+                    <a class="page-link" href="{% querystring table.prefixed_page_field=table.page.next_page_number %}">
                         {% trans 'next' %}
                         <span aria-hidden="true">&raquo;</span>
                     </a>

From 519954f9cacd1895b621d395397473899e34fa6a Mon Sep 17 00:00:00 2001
From: ikkemaniac <ikkemaniac@localhost>
Date: Tue, 27 Sep 2022 00:54:20 +0200
Subject: [PATCH 2/2] fix: always show next and previous buttons to keep the
 nav bar consistent. just disable them appropriately.

---
 .../templates/django_tables2/bootstrap.html        | 14 +++++---------
 1 file changed, 5 insertions(+), 9 deletions(-)

diff --git a/django_tables2/templates/django_tables2/bootstrap.html b/django_tables2/templates/django_tables2/bootstrap.html
index 7a724fab..a3a712a6 100644
--- a/django_tables2/templates/django_tables2/bootstrap.html
+++ b/django_tables2/templates/django_tables2/bootstrap.html
@@ -58,16 +58,14 @@
         {% if table.page and table.paginator.num_pages > 1 %}
         <nav aria-label="Table navigation">
             <ul class="pagination">
-            {% if table.page.has_previous %}
                 {% block pagination.previous %}
-                    <li class="page-item previous">
-                        <a class="page-link" href="{% querystring table.prefixed_page_field=table.page.previous_page_number %}">
+                <li class="page-item previous{% if not table.page.has_previous %} disabled{% endif %}">
+                        <a class="page-link" {% if table.page.has_previous %}href="{% querystring table.prefixed_page_field=table.page.previous_page_number %}"{% endif %}>
                             <span aria-hidden="true">&laquo;</span>
                             {% trans 'previous' %}
                         </a>
                     </li>
                 {% endblock pagination.previous %}
-            {% endif %}
             {% if table.page.has_previous or table.page.has_next %}
                 {% block pagination.range %}
                     {% for p in table.page|table_page_range:table.paginator %}
@@ -84,16 +82,14 @@
                 {% endblock pagination.range %}
             {% endif %}
 
-            {% if table.page.has_next %}
-                {% block pagination.next %}
-                <li class="next page-item">
-                    <a class="page-link" href="{% querystring table.prefixed_page_field=table.page.next_page_number %}">
+            {% block pagination.next %}
+            <li class="next page-item{% if not table.page.has_next %} disabled{% endif %}">
+                    <a class="page-link" {% if table.page.has_next %}href="{% querystring table.prefixed_page_field=table.page.next_page_number %}"{% endif %}>
                         {% trans 'next' %}
                         <span aria-hidden="true">&raquo;</span>
                     </a>
                 </li>
                 {% endblock pagination.next %}
-            {% endif %}
             </ul>
         </nav>
         {% endif %}