From 88ebeaac7fcf140cbab489e841bf9c197f84c4ea Mon Sep 17 00:00:00 2001
From: erdgeist <erdgeist@erdgeist.org>
Date: Tue, 14 Jul 2026 02:17:04 +0200
Subject: Make revisions locale aware

---
 app/views/page_translations/edit.html.erb  |  2 ++
 app/views/page_translations/index.html.erb | 19 -------------------
 app/views/page_translations/show.html.erb  |  2 ++
 app/views/revisions/index.html.erb         | 19 +++++++++++++++++--
 app/views/revisions/show.html.erb          | 11 ++++++-----
 5 files changed, 27 insertions(+), 26 deletions(-)
 delete mode 100644 app/views/page_translations/index.html.erb

(limited to 'app/views')

diff --git a/app/views/page_translations/edit.html.erb b/app/views/page_translations/edit.html.erb
index 89b594e..981fb74 100644
--- a/app/views/page_translations/edit.html.erb
+++ b/app/views/page_translations/edit.html.erb
@@ -2,6 +2,8 @@
   Editing the <strong><%= @locale.to_s.upcase %></strong> translation of <strong><%= title_for_node(@node) %></strong>.
 </div>
 
+<p class="node_action_bar standalone_action_bar"><%= link_to 'View revision history', node_revisions_path(@node, :locale => @locale) %></p>
+
 <p class="field_hint">
   Metadata such as images, tags, template, and author belong to the page
   as a whole, not to a single translation — change those from the
diff --git a/app/views/page_translations/index.html.erb b/app/views/page_translations/index.html.erb
deleted file mode 100644
index a8c4f5d..0000000
--- a/app/views/page_translations/index.html.erb
+++ /dev/null
@@ -1,19 +0,0 @@
-<h1>Translations — <%= title_for_node(@node) %></h1>
-
-<div class="node_content node_info_group">
-  <div class="node_info_group_items">
-    <% @translations.each do |t| %>
-      <div class="node_info_item">
-        <span class="node_info_label"><%= t[:locale].to_s.upcase %></span>
-        <% if t[:exists] %>
-          <%= t[:title] %> — updated <%= t[:updated_at] %>
-          <% if t[:outdated] %><span class="warning">may be outdated</span><% end %>
-          <%= link_to 'Edit', edit_node_translation_path(@node, t[:locale]) %>
-        <% else %>
-          <span class="field_hint">Not yet translated.</span>
-          <%= link_to 'Add', edit_node_translation_path(@node, t[:locale]) %>
-        <% end %>
-      </div>
-    <% end %>
-  </div>
-</div>
diff --git a/app/views/page_translations/show.html.erb b/app/views/page_translations/show.html.erb
index c5bd151..60eb7cf 100644
--- a/app/views/page_translations/show.html.erb
+++ b/app/views/page_translations/show.html.erb
@@ -1,5 +1,7 @@
 <h1>Compare — <%= title_for_node(@node) %></h1>
 
+<p class="node_action_bar standalone_action_bar"><%= link_to 'View revision history', node_revisions_path(@node, :locale => @locale) %></p>
+
 <div class="translation_compare">
   <div class="translation_compare_column">
     <h2><%= @locale.to_s.upcase %></h2>
diff --git a/app/views/revisions/index.html.erb b/app/views/revisions/index.html.erb
index 4680ae2..7ea74a6 100644
--- a/app/views/revisions/index.html.erb
+++ b/app/views/revisions/index.html.erb
@@ -4,6 +4,20 @@
   <%= link_to 'Back to node', node_path(@node) %>
 </p>
 
+<% if Page.non_default_locales.any? %>
+  <p class="diff_locale_toggle">
+    Locale:
+    <% ([I18n.default_locale] + Page.non_default_locales).each_with_index do |locale, i| %>
+      <%= " &middot; ".html_safe if i > 0 %>
+      <% if locale == @locale %>
+        <strong><%= locale.to_s.upcase %></strong>
+      <% else %>
+        <%= link_to locale.to_s.upcase, node_revisions_path(@node, :locale => locale) %>
+      <% end %>
+    <% end %>
+  </p>
+<% end %>
+
 <table id="revisions" class="revisions_table">
   <thead>
     <tr class="header">
@@ -20,6 +34,7 @@
       <td colspan="8">
         <%= button_to 'Diff revisions', diff_node_revisions_path(@node),
               method: :post,
+              params: { locale: @locale },
               form: { id: 'diff_form', class: 'button_to computation' },
               disabled: true %>
         <span id="diff_selection_label"></span>
@@ -35,10 +50,10 @@
       <td><%= radio_button_tag :start_revision, page.revision, index == 1 %></td>
       <td><%= radio_button_tag :end_revision, page.revision, index == 0 %></td>
       <td class="revision"><%= page.revision %></td>
-      <td class="title"><%= page.title %></td>
+      <td class="title"><%= page.translations.find_by(:locale => @locale)&.title || "—" %></td>
       <td class="user"><%= page.editor.try(:login) %></td>
       <td class="date"><%= page.updated_at %></td>
-      <td><%= link_to 'show', node_revision_path(@node, page) %></td>
+      <td><%= link_to 'show', node_revision_path(@node, page, :locale => @locale) %></td>
       <td>
         <%= button_to 'restore', restore_node_revision_path(@node, page),
               method: :put,
diff --git a/app/views/revisions/show.html.erb b/app/views/revisions/show.html.erb
index 92d959b..74fb51f 100644
--- a/app/views/revisions/show.html.erb
+++ b/app/views/revisions/show.html.erb
@@ -1,12 +1,13 @@
+<% translation = @page.translations.find_by(:locale => @locale) %>
 <div id="page_editor" class="show_node">
-  <h1>Revision <%= @page.revision %>: <%= @page.title %></h1>
+  <h1>Revision <%= @page.revision %>: <%= translation&.title %> <small>(<%= @locale.to_s.upcase %>)</small></h1>
 
   <div id="content">
     <div class="node_description">Actions</div>
     <div class="node_content node_info_group">
       <div class="node_info_group_items">
         <div class="node_info_item">
-          <%= link_to 'Show all revisions', node_revisions_path(@node) %>
+          <%= link_to 'Show all revisions', node_revisions_path(@node, :locale => @locale) %>
         </div>
       </div>
     </div>
@@ -27,12 +28,12 @@
     <div class="node_content"><%= @page.published_at %></div>
 
     <div class="node_description">Title</div>
-    <div class="node_content"><%= @page.title %></div>
+    <div class="node_content"><%= translation&.title %></div>
 
     <div class="node_description">Abstract</div>
-    <div class="node_content"><%= @page.abstract %></div>
+    <div class="node_content"><%= translation&.abstract %></div>
 
     <div class="node_description">Body</div>
-    <div class="node_content"><%= raw @page.body %></div>
+    <div class="node_content"><%= raw translation&.body %></div>
   </div>
 </div>
-- 
cgit v1.3