Add missing thead & tbody tags on moderation index views

app/views/moderation/comments/index.html.erb

@@ -9,28 +9,27 @@
 </div>
 
 <%= form_tag moderate_moderation_comments_path(request.query_parameters), method: :put do %>
-  <p class="js-check">
+  <p class="float-left js-check">
     <%= t('shared.check') %>:
-    <%= link_to t('shared.check_all'), '#', data: {check_all: "comment_ids[]"} %>
+    <%= link_to t('shared.check_all'), '#', data: { check_all: "comment_ids[]" } %>
     |
-    <%= link_to t('shared.check_none'), '#', data: {check_none: "comment_ids[]"} %>
+    <%= link_to t('shared.check_none'), '#', data: { check_none: "comment_ids[]" } %>
   </p>
 
-  <table>
+  <table class="clear">
+    <thead>
       <tr>
-      <th>
-        <%= t("moderation.comments.index.headers.comment") %>
-      </th>
-      <th>
-        <%= t("moderation.comments.index.headers.moderate") %>
-      </th>
+        <th><%= t("moderation.comments.index.headers.comment") %></th>
+        <th><%= t("moderation.comments.index.headers.moderate") %></th>
       </tr>
+    </thead>
 
+    <tbody>
       <% @comments.each do |comment| %>
         <tr id="comment_<%= comment.id %>">
           <td>
             <%= comment.commentable_type.constantize.model_name.human %> -
-          <%= link_to comment.commentable.title, commentable_path(comment) %>
+            <%= link_to comment.commentable.title, commentable_path(comment), target: "_blank" %>
             <br>
             <span class="date"><%= l comment.updated_at.to_date %></span>
             <span class="bullet">&nbsp;&bull;&nbsp;</span>
@@ -38,34 +37,33 @@
             <span class="bullet">&nbsp;&bull;&nbsp;</span>
             <%= comment.author.username %>
             <br>
-          <p>
+            <div class="moderation-description">
               <%= comment.body %>
-          </p>
+            </div>
           </td>
           <td class="text-center">
             <%= check_box_tag "comment_ids[]", comment.id, nil, id: "#{dom_id(comment)}_check" %>
           </td>
         </tr>
       <% end %>
+    </tbody>
   </table>
 
   <%= submit_tag t('moderation.comments.index.block_authors'),
                  name: "block_authors",
                  class: "button hollow alert",
-    data: {confirm: t('moderation.comments.index.confirm')}
-  %>
+                 data: { confirm: t('moderation.comments.index.confirm') } %>
 
   <div class="float-right">
     <%= submit_tag t('moderation.comments.index.hide_comments'),
                    name: "hide_comments",
                    class: "button hollow alert",
-      data: {confirm: t('moderation.comments.index.confirm')}
-    %>
+                   data: { confirm: t('moderation.comments.index.confirm') } %>
+
     <%= submit_tag t('moderation.comments.index.ignore_flags'),
                    name: "ignore_flags",
                    class: "button hollow",
-      data: {confirm: t('moderation.comments.index.confirm')}
-    %>
+                   data: { confirm: t('moderation.comments.index.confirm') } %>
   </div>
 
   <%= paginate @comments %>

app/views/moderation/debates/index.html.erb

@@ -11,20 +11,20 @@
 <%= form_tag moderate_moderation_debates_path(request.query_parameters), method: :put do %>
   <p class="float-left js-check">
     <%= t('shared.check') %>:
-    <%= link_to t('shared.check_all'), '#', data: {check_all: "debate_ids[]"} %>
+    <%= link_to t('shared.check_all'), '#', data: { check_all: "debate_ids[]" } %>
     |
-    <%= link_to t('shared.check_none'), '#', data: {check_none: "debate_ids[]"} %>
+    <%= link_to t('shared.check_none'), '#', data: { check_none: "debate_ids[]" } %>
   </p>
 
   <table class="clear">
+    <thead>
       <tr>
-      <th>
-        <%= t("moderation.debates.index.headers.debate") %>
-      </th>
-      <th>
-       <%= t("moderation.debates.index.headers.moderate") %>
-      </th>
+        <th><%= t("moderation.debates.index.headers.debate") %></th>
+        <th><%= t("moderation.debates.index.headers.moderate") %></th>
       </tr>
+    </thead>
+
+    <tbody>
       <% @debates.each do |debate| %>
         <tr id="debate_<%= debate.id %>">
           <td>
@@ -45,25 +45,24 @@
           </td>
         </tr>
       <% end %>
+    </tbody>
   </table>
 
   <%= submit_tag t('moderation.debates.index.block_authors'),
                  name: "block_authors",
-    class: "button alert",
-    data: {confirm: t('moderation.debates.index.confirm')}
-  %>
+                 class: "button hollow alert",
+                 data: { confirm: t('moderation.debates.index.confirm') } %>
 
   <div class="float-right">
     <%= submit_tag t('moderation.debates.index.hide_debates'),
                    name: "hide_debates",
-      class: "button alert",
-      data: {confirm: t('moderation.debates.index.confirm')}
-    %>
+                   class: "button hollow alert",
+                   data: { confirm: t('moderation.debates.index.confirm') } %>
+
     <%= submit_tag t('moderation.debates.index.ignore_flags'),
                    name: "ignore_flags",
-      class: "button success",
-      data: {confirm: t('moderation.debates.index.confirm')}
-    %>
+                   class: "button hollow",
+                   data: { confirm: t('moderation.debates.index.confirm') } %>
   </div>
 
   <%= paginate @debates %>

app/views/moderation/proposal_notifications/index.html.erb

@@ -9,22 +9,22 @@
 </div>
 
 <%= form_tag moderate_moderation_proposal_notifications_path(request.query_parameters), method: :put do %>
-  <p class="js-check">
+  <p class="float-left js-check">
     <%= t('shared.check') %>:
-    <%= link_to t('shared.check_all'), '#', data: {check_all: "proposal_notification_ids[]"} %>
+    <%= link_to t('shared.check_all'), '#', data: { check_all: "proposal_notification_ids[]" } %>
     |
-    <%= link_to t('shared.check_none'), '#', data: {check_none: "proposal_notification_ids[]"} %>
+    <%= link_to t('shared.check_none'), '#', data: { check_none: "proposal_notification_ids[]" } %>
   </p>
 
   <table class="clear">
+    <thead>
       <tr>
-      <th>
-        <%= t("moderation.proposal_notifications.index.headers.proposal_notification") %>
-      </th>
-      <th>
-       <%= t("moderation.proposal_notifications.index.headers.moderate") %>
-      </th>
+        <th><%= t("moderation.proposal_notifications.index.headers.proposal_notification") %></th>
+        <th><%= t("moderation.proposal_notifications.index.headers.moderate") %></th>
       </tr>
+    </thead>
+
+    <tbody>
       <% @proposal_notifications.each do |proposal_notification| %>
         <tr id="proposal_notification_<%= proposal_notification.id %>">
           <td>
@@ -37,29 +37,31 @@
             </div>
           </td>
           <td class="text-center">
-          <%= check_box_tag "proposal_notification_ids[]", proposal_notification.id, nil, id: "#{dom_id(proposal_notification)}_check" %>
+            <%= check_box_tag "proposal_notification_ids[]",
+                              proposal_notification.id,
+                              nil,
+                              id: "#{dom_id(proposal_notification)}_check" %>
           </td>
         </tr>
       <% end %>
+    </tbody>
   </table>
 
   <%= submit_tag t('moderation.proposal_notifications.index.block_authors'),
                  name: "block_authors",
                  class: "button hollow alert",
-    data: {confirm: t('moderation.proposal_notifications.index.confirm')}
-  %>
+                 data: { confirm: t('moderation.proposal_notifications.index.confirm') } %>
 
   <div class="float-right">
     <%= submit_tag t('moderation.proposal_notifications.index.hide_proposal_notifications'),
                    name: "hide_proposal_notifications",
                    class: "button hollow alert",
-      data: {confirm: t('moderation.proposal_notifications.index.confirm')}
-    %>
+                   data: { confirm: t('moderation.proposal_notifications.index.confirm') } %>
+
     <%= submit_tag t('moderation.proposal_notifications.index.ignore_flags'),
                    name: "ignore_flags",
                    class: "button hollow",
-      data: {confirm: t('moderation.proposal_notifications.index.confirm')}
-    %>
+                   data: { confirm: t('moderation.proposal_notifications.index.confirm') } %>
   </div>
 
   <%= paginate @proposal_notifications %>

app/views/moderation/proposals/index.html.erb

@@ -9,22 +9,22 @@
 </div>
 
 <%= form_tag moderate_moderation_proposals_path(request.query_parameters), method: :put do %>
-  <p class="js-check">
+  <p class="float-left js-check">
     <%= t('shared.check') %>:
-    <%= link_to t('shared.check_all'), '#', data: {check_all: "proposal_ids[]"} %>
+    <%= link_to t('shared.check_all'), '#', data: { check_all: "proposal_ids[]" } %>
     |
-    <%= link_to t('shared.check_none'), '#', data: {check_none: "proposal_ids[]"} %>
+    <%= link_to t('shared.check_none'), '#', data: { check_none: "proposal_ids[]" } %>
   </p>
 
   <table class="clear">
+    <thead>
       <tr>
-      <th>
-        <%= t("moderation.proposals.index.headers.proposal") %>
-      </th>
-      <th>
-       <%= t("moderation.proposals.index.headers.moderate") %>
-      </th>
+        <th><%= t("moderation.proposals.index.headers.proposal") %></th>
+        <th><%= t("moderation.proposals.index.headers.moderate") %></th>
       </tr>
+    </thead>
+
+    <tbody>
       <% @proposals.each do |proposal| %>
         <tr id="proposal_<%= proposal.id %>">
           <td>
@@ -45,28 +45,26 @@
           </td>
         </tr>
       <% end %>
+    </tbody>
   </table>
 
   <%= submit_tag t('moderation.proposals.index.block_authors'),
                  name: "block_authors",
                  class: "button hollow alert",
-    data: {confirm: t('moderation.proposals.index.confirm')}
-  %>
+                 data: { confirm: t('moderation.proposals.index.confirm') } %>
 
   <div class="float-right">
     <%= submit_tag t('moderation.proposals.index.hide_proposals'),
                    name: "hide_proposals",
                    class: "button hollow alert",
-      data: {confirm: t('moderation.proposals.index.confirm')}
-    %>
+                   data: { confirm: t('moderation.proposals.index.confirm') } %>
+
     <%= submit_tag t('moderation.proposals.index.ignore_flags'),
                    name: "ignore_flags",
                    class: "button hollow",
-      data: {confirm: t('moderation.proposals.index.confirm')}
-    %>
+                   data: { confirm: t('moderation.proposals.index.confirm') } %>
   </div>
 
   <%= paginate @proposals %>
 
 <% end %>
-