From 603ccfbd5fe96f0b83aaf3d5118aded2a39992fb Mon Sep 17 00:00:00 2001
From: erdgeist <erdgeist@erdgeist.org>
Date: Thu, 16 Jul 2026 13:50:15 +0200
Subject: Condense the action log into a scannable table

Several minor improvements to the action log presentation:
* Now a table with date and human readable presentation as
  rows is displayed
* If no changes in a title were detected, the old version
  is omitted
* The "inferred" flag is demoted to the end of the line
* You can zoom in on the node's history directly from a
  log line
* byline for the first publish action is preserved
* Revisions are directly linked to when a new one i
  published
---
 app/views/node_actions/_change_details.html.erb | 30 ++++++++++++-------------
 app/views/node_actions/index.html.erb           | 25 ++++++++++++---------
 2 files changed, 30 insertions(+), 25 deletions(-)

(limited to 'app/views/node_actions')

diff --git a/app/views/node_actions/_change_details.html.erb b/app/views/node_actions/_change_details.html.erb
index 383f53a..2583e8b 100644
--- a/app/views/node_actions/_change_details.html.erb
+++ b/app/views/node_actions/_change_details.html.erb
@@ -1,27 +1,27 @@
 <details class="node_action_details">
   <summary><%= t("node_actions.show_changes") %></summary>
   <table>
-    <% action_entry.metadata["translation_diff"].each do |locale, diff| %>
+    <% if (default_items = default_locale_changes(action_entry)).any? %>
+      <tr>
+        <th><%= I18n.default_locale.to_s.upcase %></th>
+        <td>
+          <%= safe_join(default_items, ", ") %>
+          <% if action_entry.page && action_entry.node %>
+            <%= link_to t("node_actions.view_revision"), node_revision_path(action_entry.node, action_entry.page) %>
+          <% end %>
+        </td>
+      </tr>
+    <% end %>
+    <% (action_entry.metadata["translation_diff"] || {}).each do |locale, diff| %>
       <tr>
         <th><%= locale.upcase %></th>
         <td>
-          <% case diff["status"] %>
-          <% when "added" %>
-            <%= t("node_actions.locale_added", :title => diff.dig("title", "to")) %>
-          <% when "removed" %>
-            <%= t("node_actions.locale_removed", :title => diff.dig("title", "from")) %>
-          <% else %>
-            <% if diff["title"] %>
-              <%= diff.dig("title", "from") %> → <%= diff.dig("title", "to") %>
-            <% end %>
-            <%= t("node_actions.abstract_changed") if diff["abstract_changed"] %>
-            <%= t("node_actions.body_changed") if diff["body_changed"] %>
+          <%= safe_join(translation_changes(diff), ", ") %>
+          <% if action_entry.page && action_entry.node %>
+            <%= link_to t("node_actions.view_revision"), node_revision_path(action_entry.node, action_entry.page, :locale => locale) %>
           <% end %>
         </td>
       </tr>
     <% end %>
   </table>
-  <% if action_entry.page && action_entry.node %>
-    <%= link_to t("node_actions.view_revision"), node_revision_path(action_entry.node, action_entry.page) %>
-  <% end %>
 </details>
diff --git a/app/views/node_actions/index.html.erb b/app/views/node_actions/index.html.erb
index 92740cd..ee06213 100644
--- a/app/views/node_actions/index.html.erb
+++ b/app/views/node_actions/index.html.erb
@@ -6,17 +6,22 @@
 
 <%= will_paginate @actions %>
 
-<ul id="node_action_list">
+<table id="node_action_list">
   <% @actions.each do |action| %>
-    <li class="node_action node_action--<%= action.action %>">
-      <span class="node_action_time"><%= l(action.occurred_at, :format => :ccc) %></span>
-      <% if action.inferred_from %>
-        <span class="node_action_inferred" title="<%= action.inferred_from %>"><%= t("node_actions.backfilled") %></span>
-      <% end %>
-      <span class="node_action_summary"><%= action_summary(action) %></span>
-      <%= render "change_details", :action_entry => action if action.metadata["translation_diff"].present? %>
-    </li>
+    <tr class="node_action node_action--<%= action.action %>">
+      <td class="node_action_time"><%= action.occurred_at.strftime("%Y-%m-%d %H:%M") %></td>
+      <td class="node_action_body">
+        <%= action_summary(action) %>
+        <% if action.node_id && params[:node_id].blank? %>
+          <%= link_to t("node_actions.node_history"), admin_log_path(:node_id => action.node_id), :class => "node_action_zoom" %>
+        <% end %>
+        <% if action.inferred_from %>
+          <span class="node_action_inferred" title="<%= action.inferred_from %>"><%= t("node_actions.backfilled") %></span>
+        <% end %>
+        <%= render "change_details", :action_entry => action if action_details?(action) %>
+      </td>
+    </tr>
   <% end %>
-</ul>
+</table>
 
 <%= will_paginate @actions %>
-- 
cgit v1.3