From 8d329609eab7efeff0513df29f2557862c612e6f Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Fr=C3=A9d=C3=A9ric=20P=C3=A9ters?= <fpeters@entrouvert.com>
Date: Mon, 23 Dec 2019 13:32:51 +0100
Subject: [PATCH] manager: use list and icons for exception sources (#38689)

---
 chrono/manager/static/css/style.scss          |  7 ++++
 .../chrono/manager_import_exceptions.html     | 32 +++++--------------
 2 files changed, 15 insertions(+), 24 deletions(-)

diff --git a/chrono/manager/static/css/style.scss b/chrono/manager/static/css/style.scss
index 0839a1b3..19a7ef62 100644
--- a/chrono/manager/static/css/style.scss
+++ b/chrono/manager/static/css/style.scss
@@ -274,3 +274,10 @@ ul#id_weekdays {
 		width: 10em;
 	}
 }
+
+ul.objects-list.single-links li a.link-action-icon.refresh {
+	margin-right: 3em;
+	&::before {
+		content: "\f021"; /* refresh */
+	}
+}
diff --git a/chrono/manager/templates/chrono/manager_import_exceptions.html b/chrono/manager/templates/chrono/manager_import_exceptions.html
index dda92e38..c774344b 100644
--- a/chrono/manager/templates/chrono/manager_import_exceptions.html
+++ b/chrono/manager/templates/chrono/manager_import_exceptions.html
@@ -14,35 +14,19 @@
 
 <form method="post" enctype="multipart/form-data">
   {% if exception_sources %}
-  <table class="main">
-    <thead>
-      <tr>
-        <th>{% trans "Exceptions" %}</th>
-        <th></th>
-        <th></th>
-      </tr>
-    </thead>
-    <tbody>
+  <ul class="objects-list single-links">
       {% for object in exception_sources %}
-      <tr>
-        <td>
-          <span title="{{ object }}">
-            {% if object.ics_filename %}{{ object|truncatechars:50 }}{% else %}<a href="{{ object }}">{{ object|truncatechars:50 }}</a>{% endif %}
-          </span>
-        </td>
-        <td>
+        <li>
+          <a title="{{ object }}" {% if not object.ics_filename %}href="{{ object }}"{% endif %}>{% if object.ics_filename %}{{ object|truncatechars:50 }}{% else %}{{ object|truncatechars:50 }}{% endif %}</a>
           {% if object.ics_filename %}
-            <a rel="popup" href="{% url 'chrono-manager-time-period-exception-source-replace' object.pk %}">{% trans "replace" %}</a>
+            <a rel="popup" class="link-action-icon refresh" href="{% url 'chrono-manager-time-period-exception-source-replace' object.pk %}">{% trans "replace" %}</a>
           {% else %}
-            <a href="{% url 'chrono-manager-time-period-exception-source-refresh' object.pk %}">{% trans "refresh" %}</a>
+            <a class="link-action-icon refresh" href="{% url 'chrono-manager-time-period-exception-source-refresh' object.pk %}">{% trans "refresh" %}</a>
           {% endif %}
-        </td>
-        <td><a rel="popup" href="{% url 'chrono-manager-time-period-exception-source-delete' object.pk %}">{% trans "remove" %}</a></td>
-      </tr>
+          <a rel="popup" class="delete" href="{% url 'chrono-manager-time-period-exception-source-delete' object.pk %}">{% trans "remove" %}</a>
+        </li>
       {% endfor %}
-    </tbody>
-   </table>
-   <br />
+  </ul>
   {% endif %}
 
   <p class="notice">{% trans "To add new exceptions, you can upload a file or specify an address to a remote calendar." %}</p>