Give node lists a flag column and a shared action bar

1 files changed, 30 insertions, 10 deletions

diff --git a/app/views/nodes/_node_list.html.erb b/app/views/nodes/_node_list.html.erb
index 0dc3c5a0..458ab1ae 100644
--- a/app/views/nodes/_node_list.html.erb
+++ b/app/views/nodes/_node_list.html.erb
@@ -15,26 +15,46 @@
 <%= will_paginate @nodes %>
 <table class="node_table">
   <tr class="header">
-    <th class="node_id"><%= t("admin.columns.id") %></th>
     <th class="title"><%= t("admin.columns.title") %></th>
     <th class="actions"><%= t("admin.columns.actions") %></th>
-    <th class="editor"><%= t("admin.columns.locked_by") %></th>
     <th class="revision"><%= t("admin.columns.rev") %></th>
   </tr>
   <% @nodes.each do |node| %>
   <tr class="<%= cycle("even", "odd") %>">
-    <td class="node_id"><%= node.id %></td>
     <td class="title">
-      <h4><%= link_to title_for_node(node), node_path(node) %></h4>
-      <p><%= link_to_path(node.unique_name, node.unique_name) %></p>
+      <div class="title_with_flags">
+        <span class="flag_stack">
+          <%= flag("lock", t(".flag_locked", :login => node.lock_owner.login),
+                   :tier => :attention) if node.lock_owner %>
+          <%= flag("calendar-clock",
+                   t(".flag_embargo", :date => admin_datetime(node.head.published_at)),
+                   :tier => :attention) if node.embargoed? %>
+          <%= flag("file-pencil", t(".flag_draft")) if node.draft %>
+          <%= flag("file-off", t(".flag_no_head")) unless node.head %>
+        </span>
+        <div class="title_body">
+          <div class="node_title"><%= link_to title_for_node(node), node_path(node) %></div>
+          <div class="node_path"><%= link_to_path(node.unique_name, node.unique_name) %></div>
+        </div>
+      </div>
     </td>
     <td class="actions">
-      <%= link_to 'show',      node_path(node) %>
-      <%= link_to 'edit',      edit_node_path(node) %>
-      <%= link_to 'revisions', node_revisions_path(node) %>
+      <div class="action_grid">
+        <span class="action_item">
+          <%= link_to node_path(node), "aria-label" => t("admin.common.show"),
+                title: t("admin.common.show") do %>
+            <%= icon("eye", library: "tabler", "aria-hidden": true) %>
+          <% end %>
+        </span>
+        <span class="action_item">
+          <%= link_to node_revisions_path(node), "aria-label" => t("admin.common.revisions"),
+                title: t("admin.common.revisions") do %>
+            <%= icon("history", library: "tabler", "aria-hidden": true) %>
+          <% end %>
+        </span>
+      </div>
     </td>
-    <td><%= node.lock_owner.login if node.lock_owner %></td>
-    <td><%= node.draft ? node.draft.revision : (node.head ? node.head.revision : t(".no_revision")) %></td>
+    <td><%= node.head ? node.head.revision : t(".no_revision") %></td>
   </tr>
   <% end %>
 </table>