diff options
| author | erdgeist <erdgeist@erdgeist.org> | 2026-07-16 13:50:15 +0200 |
|---|---|---|
| committer | erdgeist <erdgeist@erdgeist.org> | 2026-07-16 13:50:15 +0200 |
| commit | 603ccfbd5fe96f0b83aaf3d5118aded2a39992fb (patch) | |
| tree | 0e88ec41af6ce1a03e65bedb3f68a5a9881d96bb /app/views/node_actions/index.html.erb | |
| parent | bb7ef80d084f474bb4c3be0ae0d033aff3de0272 (diff) | |
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
Diffstat (limited to 'app/views/node_actions/index.html.erb')
| -rw-r--r-- | app/views/node_actions/index.html.erb | 25 |
1 files changed, 15 insertions, 10 deletions
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 @@ | |||
| 6 | 6 | ||
| 7 | <%= will_paginate @actions %> | 7 | <%= will_paginate @actions %> |
| 8 | 8 | ||
| 9 | <ul id="node_action_list"> | 9 | <table id="node_action_list"> |
| 10 | <% @actions.each do |action| %> | 10 | <% @actions.each do |action| %> |
| 11 | <li class="node_action node_action--<%= action.action %>"> | 11 | <tr class="node_action node_action--<%= action.action %>"> |
| 12 | <span class="node_action_time"><%= l(action.occurred_at, :format => :ccc) %></span> | 12 | <td class="node_action_time"><%= action.occurred_at.strftime("%Y-%m-%d %H:%M") %></td> |
| 13 | <% if action.inferred_from %> | 13 | <td class="node_action_body"> |
| 14 | <span class="node_action_inferred" title="<%= action.inferred_from %>"><%= t("node_actions.backfilled") %></span> | 14 | <%= action_summary(action) %> |
| 15 | <% end %> | 15 | <% if action.node_id && params[:node_id].blank? %> |
| 16 | <span class="node_action_summary"><%= action_summary(action) %></span> | 16 | <%= link_to t("node_actions.node_history"), admin_log_path(:node_id => action.node_id), :class => "node_action_zoom" %> |
| 17 | <%= render "change_details", :action_entry => action if action.metadata["translation_diff"].present? %> | 17 | <% end %> |
| 18 | </li> | 18 | <% if action.inferred_from %> |
| 19 | <span class="node_action_inferred" title="<%= action.inferred_from %>"><%= t("node_actions.backfilled") %></span> | ||
| 20 | <% end %> | ||
| 21 | <%= render "change_details", :action_entry => action if action_details?(action) %> | ||
| 22 | </td> | ||
| 23 | </tr> | ||
| 19 | <% end %> | 24 | <% end %> |
| 20 | </ul> | 25 | </table> |
| 21 | 26 | ||
| 22 | <%= will_paginate @actions %> | 27 | <%= will_paginate @actions %> |