From 8462538123f95ca9eeb5592499655da06e7def89 Mon Sep 17 00:00:00 2001
From: Mathieu Jaumotte <mathieu.jaumotte@champs-libres.coop>
Date: Fri, 16 Jul 2021 09:51:15 +0200
Subject: [PATCH] adapt pagination to bootstrap, add a short pagination variant

---
 .../Resources/views/Pagination/long.html.twig | 38 ++++++++++---------
 .../views/Pagination/short.html.twig          | 25 ++++++++++++
 2 files changed, 45 insertions(+), 18 deletions(-)
 create mode 100644 src/Bundle/ChillMainBundle/Resources/views/Pagination/short.html.twig

diff --git a/src/Bundle/ChillMainBundle/Resources/views/Pagination/long.html.twig b/src/Bundle/ChillMainBundle/Resources/views/Pagination/long.html.twig
index 3166c187b..b0b787078 100644
--- a/src/Bundle/ChillMainBundle/Resources/views/Pagination/long.html.twig
+++ b/src/Bundle/ChillMainBundle/Resources/views/Pagination/long.html.twig
@@ -1,27 +1,29 @@
 {%- macro page(text, page, class = "") -%}
-<span class="link {{ class }}"><a href="{{ page.generateUrl() }}">{{ text }}</a></span>
+<li class="page-item {{ class }}"><a class="page-link" href="{{ page.generateUrl() }}">{{ text }}</a></li>
 {%- endmacro -%}
 
 {% import _self as m %}
 
 {% if paginator|length > 1 %}
-<div class="pagination long">
-    {%- if paginator.hasPreviousPage() -%}
-        {{ m.page('Previous'|trans, paginator.getPreviousPage() ) }}
-    {%- endif -%}
-    {% for i in range(10, 1) %}
-        {%- if paginator.hasPage(current - i) -%}
-        {{ m.page(current - i, paginator.getPage(current - i) ) }}
+<nav aria-label="Page navigation">
+    <ul class="pagination justify-content-center long">
+        {%- if paginator.hasPreviousPage() -%}
+            {{ m.page('Previous'|trans, paginator.getPreviousPage() ) }}
         {%- endif -%}
-    {%- endfor -%}
-    <span class="current link">{{ current }}</span>
-    {% for i in range(1,10) %}
-        {%- if paginator.hasPage(current + i) -%}
-        {{ m.page(current + i, paginator.getPage(current + i) ) }}
+        {% for i in range(10, 1) %}
+            {%- if paginator.hasPage(current - i) -%}
+            {{ m.page(current - i, paginator.getPage(current - i) ) }}
+            {%- endif -%}
+        {%- endfor -%}
+        <li class="active page-item"><span class="page-link">{{ current }}</span></li>
+        {% for i in range(1,10) %}
+            {%- if paginator.hasPage(current + i) -%}
+            {{ m.page(current + i, paginator.getPage(current + i) ) }}
+            {%- endif -%}
+        {%- endfor -%}
+        {%- if paginator.hasNextPage() -%}
+            {{ m.page('Next'|trans, paginator.getNextPage() ) }}
         {%- endif -%}
-    {%- endfor -%}
-    {%- if paginator.hasNextPage() -%}
-        {{ m.page('Next'|trans, paginator.getNextPage() ) }}
-    {%- endif -%}
-</div>
+    </ul>
+</nav>
 {% endif %}
diff --git a/src/Bundle/ChillMainBundle/Resources/views/Pagination/short.html.twig b/src/Bundle/ChillMainBundle/Resources/views/Pagination/short.html.twig
new file mode 100644
index 000000000..27bc0c75d
--- /dev/null
+++ b/src/Bundle/ChillMainBundle/Resources/views/Pagination/short.html.twig
@@ -0,0 +1,25 @@
+{%- macro page(text, page, class = "") -%}
+    <li class="page-item {{ class }}"><a class="page-link" href="{{ page.generateUrl() }}">{{ text }}</a></li>
+{%- endmacro -%}
+
+{% import _self as m %}
+
+{% if paginator|length > 1 %}
+    <nav aria-label="Page navigation">
+        <ul class="pagination pagination-sm justify-content-center short">
+
+            {% for i in range(10, 1) %}
+                {%- if paginator.hasPage(current - i) -%}
+                    {{ m.page(current - i, paginator.getPage(current - i) ) }}
+                {%- endif -%}
+            {%- endfor -%}
+            <li class="active page-item"><span class="page-link">{{ current }}</span></li>
+            {% for i in range(1,10) %}
+                {%- if paginator.hasPage(current + i) -%}
+                    {{ m.page(current + i, paginator.getPage(current + i) ) }}
+                {%- endif -%}
+            {%- endfor -%}
+
+        </ul>
+    </nav>
+{% endif %}