From 291f160a0085ab2f1cb6f6c42593debe1568bb35 Mon Sep 17 00:00:00 2001
From: Antonio Terceiro <terceiro@debian.org>
Date: Sun, 20 Jun 2021 17:05:31 -0300
Subject: [PATCH] talks: list talks in a table

If multiple languages are configured, also display the language in an
extra column.
---
 wafer/talks/templates/wafer.talks/talks.html | 93 ++++++++++++--------
 wafer/talks/views.py                         |  4 +
 2 files changed, 61 insertions(+), 36 deletions(-)

diff --git a/wafer/talks/templates/wafer.talks/talks.html b/wafer/talks/templates/wafer.talks/talks.html
index e1966165..f8410b51 100644
--- a/wafer/talks/templates/wafer.talks/talks.html
+++ b/wafer/talks/templates/wafer.talks/talks.html
@@ -5,44 +5,65 @@
 {% block content %}
 <section class="wafer wafer-talks">
   {% regroup talk_list by talk_type.name as grouped_talks %}
-  {% for type, talks in grouped_talks %}
-    {% if type %}
-      <h1>{{ type }}</h1>
-    {% else %}
-      <h1>{% trans 'Talks' %}</h1>
-    {% endif %}
-    <div class="wafer list">
-      {% for talk in talks %}
-        <div>
-          {% if not talk.cancelled and not talk.accepted and talk.is_late_submission %}
-            <span class="badge badge-warning" title="{% trans 'Late submission' %}">L</span>
-          {% endif %}
-          {% if talk.submitted %}
-            <span class="badge badge-info" title="{% trans 'Submitted' %}">{{ talk.status }}</span>
-          {% elif talk.under_consideration %}
-            <span class="badge badge-info" title="{% trans 'Under consideration' %}">{{ talk.status }}</span>
-          {% elif talk.reject %}
-            <span class="badge badge-danger" title="{% trans 'Not accepted' %}">{{ talk.status }}</span>
-          {% elif talk.cancelled %}
-            <span class="badge badge-warning" title="{% trans 'Talk Cancelled' %}">{{ talk.status }}</span>
-          {% elif talk.provisional %}
-            <span class="badge badge-success" title="{% trans 'Provisionally Accepted' %}">{{ talk.status }}</span>
-          {% endif %}
-          {% reviewed_badge user talk %}
-          {% if talk.withdrawn %}
-            <del><a href="{{ talk.get_absolute_url }}">{{ talk.title }}</a></del>
-          {% else %}
-            <a href="{{ talk.get_absolute_url }}">{{ talk.title }}</a>
-          {% endif %}
-          {% blocktrans trimmed with authors=talk.get_authors_display_name %}
-            by <span class="author">{{ authors }}</span>
-          {% endblocktrans %}
-        </div>
+  <div class="table-responsive">
+    <table class='table table-striped'>
+      {% for type, talks in grouped_talks %}
+        <thead class='thead-dark'>
+          <tr>
+            <th>
+              {% if type %}
+                {{ type }}
+              {% else %}
+                {% trans "Talk" %}
+              {% endif %}
+            </th>
+            {% if languages %}<th>{% trans "Language" %}</th>{% endif %}
+            <th>{% trans "Speakers" %}</th>
+          </tr>
+        </thead>
+        {% for talk in talks %}
+          <tr>
+            <td>
+              {% if not talk.cancelled and not talk.accepted and talk.is_late_submission %}
+                <span class="badge badge-warning" title="{% trans 'Late submission' %}">L</span>
+              {% endif %}
+              {% if talk.submitted %}
+                <span class="badge badge-info" title="{% trans 'Submitted' %}">{{ talk.status }}</span>
+              {% elif talk.under_consideration %}
+                <span class="badge badge-info" title="{% trans 'Under consideration' %}">{{ talk.status }}</span>
+              {% elif talk.reject %}
+                <span class="badge badge-danger" title="{% trans 'Not accepted' %}">{{ talk.status }}</span>
+              {% elif talk.cancelled %}
+                <span class="badge badge-warning" title="{% trans 'Talk Cancelled' %}">{{ talk.status }}</span>
+              {% elif talk.provisional %}
+                <span class="badge badge-success" title="{% trans 'Provisionally Accepted' %}">{{ talk.status }}</span>
+              {% endif %}
+              {% reviewed_badge user talk %}
+              {% if talk.withdrawn %}
+                <del><a href="{{ talk.get_absolute_url }}">{{ talk.title }}</a></del>
+              {% else %}
+                <a href="{{ talk.get_absolute_url }}">{{ talk.title }}</a>
+              {% endif %}
+            </td>
+            {% if languages %}
+              <td>{{ talk.get_language_display }}</td>
+            {% endif %}
+            <td>
+              {% with authors=talk.get_authors_display_name %}
+                <span class="author">{{ authors }}</span>
+              {% endwith %}
+            </td>
+          </tr>
+        {% endfor %}
       {% empty %}
-        <p>{% trans 'No talks accepted yet.' %}</p>
+        <tr>
+          <th colspan="{% if languages %}3{% else %}2{% endif %}">
+            {% trans 'No talks accepted yet.' %}
+          </th>
+        </tr>
       {% endfor %}
-    </div>
-  {% endfor %}
+    </table>
+  </div>
 </section>
 {% if is_paginated %}
   <section class="wafer wafer-pagination">
diff --git a/wafer/talks/views.py b/wafer/talks/views.py
index b3f0bff1..219ae239 100644
--- a/wafer/talks/views.py
+++ b/wafer/talks/views.py
@@ -57,6 +57,10 @@ def get_queryset(self):
             "talk_type", "corresponding_author", "authors", "authors__userprofile"
         )
 
+    def get_context_data(self, **kwargs):
+        context = super().get_context_data(**kwargs)
+        context["languages"] = Talk.LANGUAGES
+        return context
 
 
 class TalkView(BuildableDetailView):