From 9b288613aeb4e5d32ffb9b57dd453c55eaa5eed6 Mon Sep 17 00:00:00 2001
From: Saurabh Patel <saurabh.finch@gmail.com>
Date: Fri, 5 Apr 2019 13:02:11 +0530
Subject: [PATCH] DEV: remove span from inside <tr> and move meta info to a td
 (#7324)

---
 app/views/list/list.erb | 44 ++++++++++++++++++++---------------------
 1 file changed, 21 insertions(+), 23 deletions(-)

diff --git a/app/views/list/list.erb b/app/views/list/list.erb
index 7e7088bac7a..d7deb94e517 100644
--- a/app/views/list/list.erb
+++ b/app/views/list/list.erb
@@ -53,32 +53,30 @@
       <meta itemprop='itemListOrder' content='http://schema.org/ItemListOrderDescending'>
       <% @list.topics.each_with_index do |t,i| %>
         <tr>
-          <span itemprop='itemListElement' itemscope itemtype='http://schema.org/ListItem'>
+          <td itemprop='itemListElement' itemscope itemtype='http://schema.org/ListItem'>
             <meta itemprop='position' content='<%= i %>'>
             <meta itemprop='url' content='<%= t.url %>'>
             <meta itemprop='name' content='<%= t.title %>'>
-            <td>
-              <a href='<%= t.url %>' class='title raw-link raw-topic-link'>
-                <span><%= t.title %></span>
-              </a>
-              <% if (!@category || @category.has_children?) && t.category %>
-                <div>
-                  <a href='<%= t.category.url %>'><span class='category-name'><%= t.category.name %></span></a>
-                </div>
-              <% end %>
-              <% if t.pinned_until && (t.pinned_until > Time.zone.now) && (t.pinned_globally || @list.category) && t.excerpt %>
-                <p class='excerpt'>
-                  <%= t.excerpt.html_safe %>
-                </p>
-              <% end %>
-            </td>
-            <td>
-              <span class='posts' title='<%= t 'posts' %>'><%= t.posts_count %></span>
-            </td>
-            <td>
-              <%= I18n.l(t.created_at, format: :date_only) %>
-            </td>
-          </span>
+            <a href='<%= t.url %>' class='title raw-link raw-topic-link'>
+              <span><%= t.title %></span>
+            </a>
+            <% if (!@category || @category.has_children?) && t.category %>
+              <div>
+                <a href='<%= t.category.url %>'><span class='category-name'><%= t.category.name %></span></a>
+              </div>
+            <% end %>
+            <% if t.pinned_until && (t.pinned_until > Time.zone.now) && (t.pinned_globally || @list.category) && t.excerpt %>
+              <p class='excerpt'>
+                <%= t.excerpt.html_safe %>
+              </p>
+            <% end %>
+          </td>
+          <td>
+            <span class='posts' title='<%= t 'posts' %>'><%= t.posts_count %></span>
+          </td>
+          <td>
+            <%= I18n.l(t.created_at, format: :date_only) %>
+          </td>
         </tr>
       <% end %>
     </tbody>