From 7afa15bff1eead376e56cc959c3152b06d2dc40a Mon Sep 17 00:00:00 2001
From: Jean-Romain Garnier <jean-romain.garnier@supelec.fr>
Date: Fri, 15 Jan 2021 19:39:05 +0100
Subject: [PATCH 1/3] fix: Subtable alternating color in subscription list

---
 cotisations/templates/cotisations/aff_cotisations.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/cotisations/templates/cotisations/aff_cotisations.html b/cotisations/templates/cotisations/aff_cotisations.html
index e48a6e2b..05be4861 100644
--- a/cotisations/templates/cotisations/aff_cotisations.html
+++ b/cotisations/templates/cotisations/aff_cotisations.html
@@ -60,7 +60,7 @@ with this program; if not, write to the Free Software Foundation, Inc.,
         <tr>
             <td><a href="{% url 'users:profil' userid=facture.user.id %}">{{ facture.user }}</a></td>
             <td>
-                <table class="table-striped">
+                <table class="table-borderless">
                     {% for article in facture.name_detailed %}
                     <tr>
                         <td>

From 42ded799e4c55ca7d6ad46ca56ef694f79a539d1 Mon Sep 17 00:00:00 2001
From: Jean-Romain Garnier <jean-romain.garnier@supelec.fr>
Date: Fri, 15 Jan 2021 19:40:04 +0100
Subject: [PATCH 2/3] fix: Whitelist and ticket list responsiveness

---
 tickets/templates/tickets/aff_tickets.html |  2 +-
 users/templates/users/aff_whitelists.html  | 79 +++++++++++-----------
 2 files changed, 41 insertions(+), 40 deletions(-)

diff --git a/tickets/templates/tickets/aff_tickets.html b/tickets/templates/tickets/aff_tickets.html
index beaadc4f..caf82cf3 100644
--- a/tickets/templates/tickets/aff_tickets.html
+++ b/tickets/templates/tickets/aff_tickets.html
@@ -46,7 +46,7 @@ with this program; if not, write to the Free Software Foundation, Inc.,
 
 
 
-<div class="table-responsiv">
+<div class="table-responsive">
 
 	<table class="table">
     <thead>
diff --git a/users/templates/users/aff_whitelists.html b/users/templates/users/aff_whitelists.html
index fdfd0b76..40d55d28 100644
--- a/users/templates/users/aff_whitelists.html
+++ b/users/templates/users/aff_whitelists.html
@@ -30,43 +30,44 @@ with this program; if not, write to the Free Software Foundation, Inc.,
 {% load acl %}
 {% load logs_extra %}
 
-<table class="table table-striped">
-    <thead>
-        <tr>
-            {% trans "User" as tr_user %}
-            <th>{% include 'buttons/sort.html' with prefix='white' col="user" text=tr_user %}</th>
-            <th>{% trans "Reason" %}</th>
-            {% trans "Start date" as tr_start %}
-            <th>{% include 'buttons/sort.html' with prefix='white' col="start" text=tr_start %}</th>
-            {% trans "End date" as tr_end %}
-            <th>{% include 'buttons/sort.html' with prefix='white' col="end" text=tr_end %}</th>
-            <th></th>
-        </tr>
-    </thead>
-    {% for whitelist in white_list %}
-    {% if whitelist.is_active %}
-        <tr class="success">
-    {% else %}
-        <tr>
+<div class="table-responsive">
+    <table class="table table-striped">
+        <thead>
+            <tr>
+                {% trans "User" as tr_user %}
+                <th>{% include 'buttons/sort.html' with prefix='white' col="user" text=tr_user %}</th>
+                <th>{% trans "Reason" %}</th>
+                {% trans "Start date" as tr_start %}
+                <th>{% include 'buttons/sort.html' with prefix='white' col="start" text=tr_start %}</th>
+                {% trans "End date" as tr_end %}
+                <th>{% include 'buttons/sort.html' with prefix='white' col="end" text=tr_end %}</th>
+                <th></th>
+            </tr>
+        </thead>
+        {% for whitelist in white_list %}
+        {% if whitelist.is_active %}
+            <tr class="success">
+        {% else %}
+            <tr>
+        {% endif %}
+                <td><a href="{% url 'users:profil' whitelist.user.id%}" class="btn btn-primary btn-sm" role="button"><i class="fa fa-user"></i> {{ whitelist.user }}</a></td>
+                <td>{{ whitelist.raison }}</td>
+                <td>{{ whitelist.date_start }}</td>
+                <td>{{ whitelist.date_end }}</td>
+                <td class="text-right">
+                    {% can_delete whitelist %}
+                    {% include 'buttons/suppr.html' with href='users:del-whitelist' id=whitelist.id %}
+                    {% acl_end %}
+                    {% history_button whitelist %}
+                    {% can_edit whitelist %}
+                    {% include 'buttons/edit.html' with href='users:edit-whitelist' id=whitelist.id %}
+                    {% acl_end %}
+                </td>
+            </tr>
+            {% endfor %}
+    </table>
+
+    {% if white_list.paginator %}
+        {% include 'pagination.html' with list=white_list %}
     {% endif %}
-            <td><a href="{% url 'users:profil' whitelist.user.id%}" class="btn btn-primary btn-sm" role="button"><i class="fa fa-user"></i> {{ whitelist.user }}</a></td>
-            <td>{{ whitelist.raison }}</td>
-            <td>{{ whitelist.date_start }}</td>
-            <td>{{ whitelist.date_end }}</td>
-            <td class="text-right">
-                {% can_delete whitelist %}
-                {% include 'buttons/suppr.html' with href='users:del-whitelist' id=whitelist.id %}
-                {% acl_end %}
-                {% history_button whitelist %}
-                {% can_edit whitelist %}
-                {% include 'buttons/edit.html' with href='users:edit-whitelist' id=whitelist.id %}
-                {% acl_end %}
-            </td>
-        </tr>
-        {% endfor %}
-</table>
-
-{% if white_list.paginator %}
-    {% include 'pagination.html' with list=white_list %}
-{% endif %}
-
+</div>

From 3d748e9e42ea95bbb8defae3ffc133536cfdfb6e Mon Sep 17 00:00:00 2001
From: Jean-Romain Garnier <jean-romain.garnier@supelec.fr>
Date: Fri, 15 Jan 2021 19:51:11 +0100
Subject: [PATCH 3/3] fix: Use bootstrap class for fa-check and fa-times color
 instead of hardcoded value

---
 cotisations/templates/cotisations/aff_cost_estimate.html | 4 ++--
 re2o/templatetags/design.py                              | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/cotisations/templates/cotisations/aff_cost_estimate.html b/cotisations/templates/cotisations/aff_cost_estimate.html
index fe654744..269dc6af 100644
--- a/cotisations/templates/cotisations/aff_cost_estimate.html
+++ b/cotisations/templates/cotisations/aff_cost_estimate.html
@@ -73,9 +73,9 @@ with this program; if not, write to the Free Software Foundation, Inc.,
             <td>{{ estimate.id }}</td>
             <td>
                 {% if estimate.final_invoice %}
-                <a href="{% url 'cotisations:edit-custom-invoice' estimate.final_invoice.pk %}"><i style="color: #1ECA18;" class="fa fa-check"></i></a>
+                <a href="{% url 'cotisations:edit-custom-invoice' estimate.final_invoice.pk %}"><i class="fa fa-check text-success"></i></a>
                 {% else %}
-                <i style="color: #D10115;" class="fa fa-times"></i>
+                <i class="fa fa-times text-danger"></i>
                 {% endif %}
             </td>
             <td>
diff --git a/re2o/templatetags/design.py b/re2o/templatetags/design.py
index 488b1489..45739133 100644
--- a/re2o/templatetags/design.py
+++ b/re2o/templatetags/design.py
@@ -31,9 +31,9 @@ def tick(valeur, autoescape=False):
 
     if isinstance(valeur, bool):
         if valeur == True:
-            result = '<i style="color: #1ECA18;" class="fa fa-check"></i>'
+            result = '<i class="fa fa-check text-success"></i>'
         else:
-            result = '<i style="color: #D10115;" class="fa fa-times"></i>'
+            result = '<i class="fa fa-times text-danger"></i>'
         return mark_safe(result)
 
     else:  #  if the value is not a boolean, display it as if tick was not called