generalize usage of the paginate_jinja macro

club/templates/club/club_sellings.jinja

@@ -53,6 +53,7 @@
       {% endfor %}
     </tbody>
   </table>
+    {# TODO: Replace this pagination by an Alpine pagination #}
   <script type="text/javascript">
     function formPagination(link){
       $("form").attr("action", link.href);

core/templates/core/base.jinja

@@ -11,6 +11,7 @@
       <link rel="stylesheet" href="{{ scss('core/markdown.scss') }}">
       <link rel="stylesheet" href="{{ scss('core/header.scss') }}">
       <link rel="stylesheet" href="{{ scss('core/navbar.scss') }}">
+      <link rel="stylesheet" href="{{ scss('core/pagination.scss') }}">
       <link rel="stylesheet" href="{{ static('vendored/select2/select2.min.css') }}">
 
       {% block jquery_css %}

core/templates/core/macros.jinja

@@ -157,6 +157,15 @@
 {% endmacro %}
 
 {% macro paginate_jinja(current_page, paginator) %}
+    {# Add pagination buttons for pages without Alpine.
+
+    This must be coupled with a view that handles pagination
+    with the Django Paginator object.
+
+    Parameters:
+        current_page (django.core.paginator.Page): the current page object
+        paginator (django.core.paginator.Paginator): the paginator object
+    #}
   <nav class="pagination">
     {% if current_page.has_previous() %}
       <a href="?page={{ current_page.previous_page_number() }}">
@@ -170,8 +179,8 @@
     {% for i in paginator.get_elided_page_range(current_page.number) %}
       {% if i == current_page.number %}
         <button class="active">{{ i }}</button>
-          {% elif i == paginator.ELLIPSIS %}
+      {% elif i == paginator.ELLIPSIS %}
-          <strong>{{ paginator.ELLIPSIS }}</strong>
+        <strong>{{ paginator.ELLIPSIS }}</strong>
       {% else %}
         <a href="?page={{ i }}">
           <button>{{ i }}</button>

counter/templates/counter/cash_summary_list.jinja

@@ -1,5 +1,5 @@
 {% extends "core/base.jinja" %}
-{% from 'core/macros.jinja' import user_profile_link, paginate %}
+{% from 'core/macros.jinja' import user_profile_link, paginate_jinja %}
 
 {% block title %}
   {% trans %}Cash register summary list{% endtrans %}
@@ -57,7 +57,7 @@
     </table>
     <br>
     {% if is_paginated %}
-      {{ paginate(page_obj, paginator) }}
+      {{ paginate_jinja(page_obj, paginator) }}
     {% endif %}
   {% else %}
     {% trans %}There is no cash register summary in this website.{% endtrans %}

counter/templates/counter/refilling_list.jinja

@@ -1,5 +1,5 @@
 {% extends "core/base.jinja" %}
-{% from 'core/macros.jinja' import paginate %}
+{% from "core/macros.jinja" import paginate_jinja %}
 
 {% block title %}
   {%- trans %}Reloads list{% endtrans %} -- {{ counter.name }}
@@ -28,7 +28,7 @@
       {%- endfor %}
     </table>
     {% if is_paginated %}
-      {{ paginate(page_obj, paginator) }}
+      {{ paginate_jinja(page_obj, paginator) }}
     {% endif %}
 {%- endblock %}
 

election/templates/election/election_list.jinja

@@ -1,5 +1,5 @@
 {% extends "core/base.jinja" %}
-{% from 'core/macros.jinja' import paginate %}
+{% from "core/macros.jinja" import paginate_jinja %}
 
 {% block title %}
   {%- trans %}Election list{% endtrans %}
@@ -46,7 +46,7 @@
     </section>
   {%- endfor %}
   {% if is_paginated %}
-    {{ paginate(page_obj, paginator) }}
+    {{ paginate_jinja(page_obj, paginator) }}
   {% endif %}
 {%- endblock %}
 

forum/templates/forum/topic.jinja

@@ -10,7 +10,6 @@
 
 {% block additional_css %}
   <link rel="stylesheet" href="{{ scss('forum/css/forum.scss') }}">
-  <link rel="stylesheet" href="{{ scss('core/pagination.scss') }}">
 {% endblock %}
 
 {% block content %}

matmat/templates/matmat/search_form.jinja

@@ -1,5 +1,5 @@
-{% from "core/macros.jinja" import user_mini_profile, paginate %}
 {% extends "core/base.jinja" %}
+{% from "core/macros.jinja" import user_mini_profile, paginate_jinja %}
 
 {% block title %}
   {% trans %}Search user{% endtrans %}
@@ -18,7 +18,9 @@
 
       {% endfor %}
     </div>
-    {{ paginate(page_obj, paginator) }}
+    {% if page_obj.has_other_pages() %}
+      {{ paginate_jinja(page_obj, paginator) }}
+    {% endif %}
     <hr>
   {% endif %}
   <h2>{% trans %}Search user{% endtrans %}</h2>

pedagogy/templates/pedagogy/guide.jinja

@@ -7,7 +7,6 @@
 
 {% block additional_css %}
   <link rel="stylesheet" href="{{ scss('pedagogy/css/pedagogy.scss') }}">
-  <link rel="stylesheet" href="{{ scss('core/pagination.scss') }}">
 {% endblock %}
 
 {% block head %}

rootplace/templates/rootplace/logs.jinja

@@ -1,5 +1,5 @@
 {% extends "core/base.jinja" %}
-{% from 'core/macros.jinja' import paginate %}
+{% from "core/macros.jinja" import paginate_jinja %}
 
 {% block title %}
   {% trans %}Operation logs{% endtrans %}
@@ -28,5 +28,5 @@
   </table>
 
   <br>
-  {{ paginate(page_obj, paginator) }}
+  {{ paginate_jinja(page_obj, paginator) }}
 {% endblock content %}

sas/templates/sas/album.jinja

@@ -3,7 +3,6 @@
 
 {%- block additional_css -%}
   <link rel="stylesheet" href="{{ scss('sas/css/album.scss') }}">
-  <link rel="stylesheet" href="{{ scss('core/pagination.scss') }}">
 {%- endblock -%}
 
 {% block title %}