From e32d9b2925334a38ec785d1c571113ce54d19b3f Mon Sep 17 00:00:00 2001
From: Oliver Davies <oliver@oliverdavies.uk>
Date: Thu, 27 Oct 2016 07:08:45 +0100
Subject: [PATCH] Use a table for talks listing

---
 source/_partials/talks-table.html.twig        | 26 +++++++
 source/_partials/talks/listing-item.html.twig |  8 ---
 source/talks.html.twig                        | 70 +++++++------------
 3 files changed, 51 insertions(+), 53 deletions(-)
 create mode 100644 source/_partials/talks-table.html.twig
 delete mode 100644 source/_partials/talks/listing-item.html.twig

diff --git a/source/_partials/talks-table.html.twig b/source/_partials/talks-table.html.twig
new file mode 100644
index 00000000..e9ecabc7
--- /dev/null
+++ b/source/_partials/talks-table.html.twig
@@ -0,0 +1,26 @@
+<table class="table table-striped">
+    <thead>
+        <tr>
+            <th>Date</th>
+            <th>Talk</th>
+            <th>Event</th>
+        </tr>
+    </thead>
+    <tbody>
+        {% for talk in talks %}
+            <tr>
+                <td>
+                    {{ talk.date|date(site.default_date_format) }}
+                </td>
+
+                <td>
+                    <a href="{{ talk.url }}">
+                        {{ talk.title }}
+                    </a>
+                </td>
+
+                <td>{{ talk.event.name }}</td>
+            </tr>
+        {% endfor %}
+    </tbody>
+</table>
diff --git a/source/_partials/talks/listing-item.html.twig b/source/_partials/talks/listing-item.html.twig
deleted file mode 100644
index f97c1e69..00000000
--- a/source/_partials/talks/listing-item.html.twig
+++ /dev/null
@@ -1,8 +0,0 @@
-<li>
-    {{ talk.event.name }} -
-    {% if talk.fuzzy_date %}
-        {{ talk.title }}
-    {% else %}
-        <a href="{{ talk.url }}">{{ talk.title }}</a>
-    {% endif %}
-</li>
diff --git a/source/talks.html.twig b/source/talks.html.twig
index c8deedd2..ef1d48e5 100644
--- a/source/talks.html.twig
+++ b/source/talks.html.twig
@@ -12,53 +12,33 @@ use:
 
 <p>There is also information about events that I’ve attended and spoken at on my <a href="{{ site.lanyrd.url }}">Lanyrd</a> and <a href="{{ site.joindin.url }}">Joind.in</a> profiles.</p>
 
-{% set found = false %}
-{% set year = 0 %}
 {% set now = 'now'|date('U') %}
 
-{% for talk in data.talks|reverse if talk.date >= now %}
-    {% if loop.first %}
-        {% set found = true %}
+{% include "talks-table" with { talks: data.talks } %}
 
-        <h2>Upcoming Talks</h2>
-    {% endif %}
+<!-- <table class="table table-striped">
+    <thead>
+        <tr>
+            <th>Date</th>
+            <th>Talk</th>
+            <th>Event</th>
+        </tr>
+    </thead>
+    <tbody>
+        {% for talk in data.talks if talk.date < now %}
+            <tr>
+                <td>
+                    {{ talk.date|date(site.default_date_format) }}
+                </td>
 
-    {% if talk.date|date('Y') != year %}
-        {% set year = talk.date|date('Y') %}
+                <td>
+                    <a href="{{ talk.url }}">
+                        {{ talk.title }}
+                    </a>
+                </td>
 
-        </ul>
-        <h3>{{ year }}</h3>
-        <ul>
-    {% endif %}
-
-    {{ include('talks/listing-item') }}
-{% endfor %}
-
-{% if found %}
-    </ul>
-{% endif %}
-
-{% set found = false %}
-{% set year = 0 %}
-
-{% for talk in data.talks if talk.date < now %}
-    {% if loop.first %}
-        {% set found = true %}
-
-        <h2>Previous Talks</h2>
-    {% endif %}
-
-    {% if talk.date|date('Y') != year %}
-        {% set year = talk.date|date('Y') %}
-
-        </ul>
-        <h3>{{ year }}</h3>
-        <ul>
-    {% endif %}
-
-    {{ include('talks/listing-item') }}
-{% endfor %}
-
-{% if found %}
-    </ul>
-{% endif %}
+                <td>{{ talk.event.name }}</td>
+            </tr>
+        {% endfor %}
+    </tbody>
+</table> -->