From 4944e3c374839d48f5de973db3a037de26df908e Mon Sep 17 00:00:00 2001
From: Jef van Schendel <mail@jefvanschendel.nl>
Date: Sun, 12 Feb 2012 19:33:05 +0100
Subject: [PATCH] Make media thumbnail gallery a list instead of a table

---
 mediagoblin/static/css/base.css               |  1 +
 .../mediagoblin/utils/object_gallery.html     | 36 ++++++++-----------
 2 files changed, 16 insertions(+), 21 deletions(-)

diff --git a/mediagoblin/static/css/base.css b/mediagoblin/static/css/base.css
index 0447d8e5..165fdcf7 100644
--- a/mediagoblin/static/css/base.css
+++ b/mediagoblin/static/css/base.css
@@ -381,6 +381,7 @@ textarea#comment_content {
   margin: 0px 4px 10px 4px;
   text-align: center;
   font-size: 0.875em;
+  list-style: none;
 }
 
 .media_thumbnail a {
diff --git a/mediagoblin/templates/mediagoblin/utils/object_gallery.html b/mediagoblin/templates/mediagoblin/utils/object_gallery.html
index 81506a84..6b5988fb 100644
--- a/mediagoblin/templates/mediagoblin/utils/object_gallery.html
+++ b/mediagoblin/templates/mediagoblin/utils/object_gallery.html
@@ -19,29 +19,23 @@
 {% from "mediagoblin/utils/pagination.html" import render_pagination %}
 
 {% macro media_grid(request, media_entries, col_number=5) %}
-  <table class="thumb_gallery">
+  <ul class="thumb_gallery">
     {% for row in gridify_cursor(media_entries, col_number) %}
-      <tr class="thumb_row
-                 {%- if loop.first %} thumb_row_first
-                 {%- elif loop.last %} thumb_row_last{% endif %}">
-        {% for entry in row %}
-          {% set entry_url = entry.url_for_self(request.urlgen) %}
-          <td class="media_thumbnail thumb_entry
-                     {%- if loop.first %} thumb_entry_first
-                     {%- elif loop.last %} thumb_entry_last{% endif %}">
-            <a href="{{ entry_url }}">
-              <img src="{{ request.app.public_store.file_url(
-                             entry.media_files['thumb']) }}" />
-            </a>
-            {% if entry.title %}
-              <br />
-              <a href="{{ entry_url }}">{{ entry.title }}</a>
-            {% endif %}
-          </td>
-        {% endfor %}
-      </tr>
+      {% for entry in row %}
+        {% set entry_url = entry.url_for_self(request.urlgen) %}
+        <li class="media_thumbnail">
+          <a href="{{ entry_url }}">
+            <img src="{{ request.app.public_store.file_url(
+                           entry.media_files['thumb']) }}" />
+          </a>
+          {% if entry.title %}
+            <br />
+            <a href="{{ entry_url }}">{{ entry.title }}</a>
+          {% endif %}
+        </li>
+      {% endfor %}
     {% endfor %}
-  </table>
+  </ul>
 {%- endmacro %}
 
 {#
-- 
2.25.1