Extract admin strings to i18n: revisions, page_translations, and menu items,

3 files changed, 62 insertions, 62 deletions

diff --git a/app/views/revisions/diff.html.erb b/app/views/revisions/diff.html.erb
index 7e7b729e..1318bd37 100644
--- a/app/views/revisions/diff.html.erb
+++ b/app/views/revisions/diff.html.erb
@@ -1,18 +1,18 @@
-<h1>Revisions#diff</h1>
+<h1><%= t(".title") %></h1>
 
 <p class="node_action_bar standalone_action_bar">
-  <%= link_to 'Revisions', node_revisions_path(@node) %>
+  <%= link_to t(".revisions_link"), node_revisions_path(@node) %>
 </p>
 
 <p class="diff_comparison_label">
-  Comparing <strong><%= describe_page_reference(params[:start_revision]) %></strong>
-  against <strong><%= describe_page_reference(params[:end_revision]) %></strong>
-  — showing the <strong><%= @locale.to_s.upcase %></strong> translation
+  <%= t(".comparing") %> <strong><%= describe_page_reference(params[:start_revision]) %></strong>
+  <%= t(".against") %> <strong><%= describe_page_reference(params[:end_revision]) %></strong>
+  <%= t(".showing_prefix") %> <strong><%= @locale.to_s.upcase %></strong> <%= t(".translation_suffix") %>
 </p>
 
 <% if @locale_summary.size > 1 %>
   <p class="diff_locale_toggle">
-    Locale:
+    <%= t("revisions.locale_label") %>
     <% @locale_summary.each_with_index do |s, i| %>
       <%= " &middot; ".html_safe if i > 0 %>
       <% label = "#{s[:locale].to_s.upcase}#{' *' if s[:changed]}" %>
@@ -23,21 +23,21 @@
       <% end %>
     <% end %>
   </p>
-  <p class="field_hint">* has a difference between these two versions.</p>
+  <p class="field_hint"><%= t(".changed_hint") %></p>
 <% end %>
 
 <p class="diff_view_toggle">
-  View:
+  <%= t(".view_label") %>
   <% if @diff_view == :inline %>
-    <strong>Inline</strong>
+    <strong><%= t("revisions.inline") %></strong>
   <% else %>
-    <%= link_to 'Inline', diff_node_revisions_path(@node, start_revision: params[:start_revision], end_revision: params[:end_revision], view: 'inline', locale: @locale) %>
+    <%= link_to t("revisions.inline"), diff_node_revisions_path(@node, start_revision: params[:start_revision], end_revision: params[:end_revision], view: 'inline', locale: @locale) %>
   <% end %>
   &middot;
   <% if @diff_view == :side_by_side %>
-    <strong>Side by side</strong>
+    <strong><%= t("revisions.side_by_side") %></strong>
   <% else %>
-    <%= link_to 'Side by side', diff_node_revisions_path(@node, start_revision: params[:start_revision], end_revision: params[:end_revision], view: 'side_by_side', locale: @locale) %>
+    <%= link_to t("revisions.side_by_side"), diff_node_revisions_path(@node, start_revision: params[:start_revision], end_revision: params[:end_revision], view: 'side_by_side', locale: @locale) %>
   <% end %>
 </p>
 
@@ -49,17 +49,17 @@
     <%= select_tag :end_revision, options_for_select(@node.pages.map{|x| x.revision}, params[:end_revision].to_i) %>
     <%= hidden_field_tag :view, @diff_view %>
     <%= hidden_field_tag :locale, @locale %>
-    <%= submit_tag 'Diff' %>
+    <%= submit_tag t(".diff_button") %>
   <% end %>
 <% else %>
-  <p class="node_action_bar standalone_action_bar"><%= link_to 'Compare two numbered revisions instead', node_revisions_path(@node) %></p>
+  <p class="node_action_bar standalone_action_bar"><%= link_to t(".compare_numbered"), node_revisions_path(@node) %></p>
 <% end %>
 
 <% if @available_layer_pairs.present? %>
   <div class="node_action_bar standalone_action_bar">
     <% @available_layer_pairs.each do |pair| %>
       <% next if [params[:start_revision].to_s, params[:end_revision].to_s].sort == pair.map(&:to_s).sort %>
-      <%= button_to "Diff #{pair.first.to_s.capitalize} vs. #{pair.last.to_s.capitalize}",
+      <%= button_to t("nodes.show.diff_layers", :from => t("nodes.show.layer_#{pair.first}"), :to => t("nodes.show.layer_#{pair.last}")),
             diff_node_revisions_path(@node),
             method: :get,
             params: { start_revision: pair.first, end_revision: pair.last, view: @diff_view, locale: @locale },
@@ -68,9 +68,9 @@
     <% if !@locked_by_other && (@node.autosave || @node.draft) %>
       <%= button_to revert_node_path(@node), method: :put,
                   params: { return_to: request.fullpath },
-                  form: { data: { confirm: "This cannot be undone. Continue?" }, class: 'button_to destructive' } do %>
+                  form: { data: { confirm: t("admin.common.confirm_irreversible") }, class: 'button_to destructive' } do %>
         <%= icon("trash", library: "tabler", "aria-hidden": true) %>
-        <%= @node.draft && !@node.autosave ? "Destroy Draft" : "Discard Autosave" %>
+        <%= @node.draft && !@node.autosave ? t("nodes.show.destroy_draft") : t("nodes.show.discard_autosave") %>
       <% end %>
     <% end %>
   </div>
@@ -80,34 +80,34 @@
   <% if @diff_view == :side_by_side %>
     <div class="diff_side_by_side">
       <div class="diff_column">
-        <h3>Title</h3>
+        <h3><%= Page.human_attribute_name(:title) %></h3>
         <p><%= raw @diff[:title][0] %></p>
-        <h3>Abstract</h3>
+        <h3><%= Page.human_attribute_name(:abstract) %></h3>
         <p><%= raw @diff[:abstract][0] %></p>
-        <h3>Body</h3>
+        <h3><%= Page.human_attribute_name(:body) %></h3>
         <%= raw @diff[:body][0] %>
       </div>
       <div class="diff_column">
-        <h3>Title</h3>
+        <h3><%= Page.human_attribute_name(:title) %></h3>
         <p><%= raw @diff[:title][1] %></p>
-        <h3>Abstract</h3>
+        <h3><%= Page.human_attribute_name(:abstract) %></h3>
         <p><%= raw @diff[:abstract][1] %></p>
-        <h3>Body</h3>
+        <h3><%= Page.human_attribute_name(:body) %></h3>
         <%= raw @diff[:body][1] %>
       </div>
     </div>
   <% else %>
-    <h3>Title</h3>
+    <h3><%= Page.human_attribute_name(:title) %></h3>
     <p><%= raw @diff[:title] %></p>
-    <h3>Abstract</h3>
+    <h3><%= Page.human_attribute_name(:abstract) %></h3>
     <p><%= raw @diff[:abstract] %></p>
-    <h3>Body</h3>
+    <h3><%= Page.human_attribute_name(:body) %></h3>
     <%= raw @diff[:body] %>
   <% end %>
 
-  <h3>Tags</h3>
+  <h3><%= Page.human_attribute_name(:tag_list) %></h3>
   <% if @diff[:tags][:added].empty? && @diff[:tags][:removed].empty? %>
-    <p class="diff_unchanged">No change.</p>
+    <p class="diff_unchanged"><%= t(".no_change") %></p>
   <% else %>
     <ul class="diff_set_list">
       <% @diff[:tags][:added].each do |tag| %><li><ins><%= tag %></ins></li><% end %>
@@ -115,16 +115,16 @@
     </ul>
   <% end %>
 
-  <h3>Template</h3>
+  <h3><%= Page.human_attribute_name(:template_name) %></h3>
   <% if @diff[:template_name][:changed] %>
-    <p><del><%= @diff[:template_name][:from] || '(none)' %></del> <ins><%= @diff[:template_name][:to] || '(none)' %></ins></p>
+    <p><del><%= @diff[:template_name][:from] || t(".none_marker") %></del> <ins><%= @diff[:template_name][:to] || t(".none_marker") %></ins></p>
   <% else %>
-    <p class="diff_unchanged">No change.</p>
+    <p class="diff_unchanged"><%= t(".no_change") %></p>
   <% end %>
 
-  <h3>Assets</h3>
+  <h3><%= Page.human_attribute_name(:assets) %></h3>
   <% if @diff[:assets][:added].empty? && @diff[:assets][:removed].empty? %>
-    <p class="diff_unchanged">No change.</p>
+    <p class="diff_unchanged"><%= t(".no_change") %></p>
   <% else %>
     <ul class="diff_set_list">
       <% @diff[:assets][:added].each do |asset| %><li><ins><%= asset.upload_file_name %></ins></li><% end %>
diff --git a/app/views/revisions/index.html.erb b/app/views/revisions/index.html.erb
index 7ea74a6c..6bcf1820 100644
--- a/app/views/revisions/index.html.erb
+++ b/app/views/revisions/index.html.erb
@@ -1,12 +1,12 @@
-<h2>Revisions for Node: <%= @node.unique_name %></h2>
+<h2><%= t(".title", :path => @node.unique_name) %></h2>
 
 <p class="node_action_bar standalone_action_bar">
-  <%= link_to 'Back to node', node_path(@node) %>
+  <%= link_to t(".back_to_node"), node_path(@node) %>
 </p>
 
 <% if Page.non_default_locales.any? %>
   <p class="diff_locale_toggle">
-    Locale:
+    <%= t("revisions.locale_label") %>
     <% ([I18n.default_locale] + Page.non_default_locales).each_with_index do |locale, i| %>
       <%= " &middot; ".html_safe if i > 0 %>
       <% if locale == @locale %>
@@ -21,25 +21,25 @@
 <table id="revisions" class="revisions_table">
   <thead>
     <tr class="header">
-      <th>First</th>
-      <th>Last</th>
-      <th>Rev.</th>
-      <th>Title</th>
-      <th>Editor</th>
-      <th>Date</th>
+      <th><%= t(".first") %></th>
+      <th><%= t(".last") %></th>
+      <th><%= t("admin.columns.rev") %></th>
+      <th><%= t("admin.columns.title") %></th>
+      <th><%= t("admin.columns.editor") %></th>
+      <th><%= t("admin.columns.date") %></th>
       <th></th>
       <th></th>
     </tr>
     <tr class="no_hover diff_sticky_bar">
       <td colspan="8">
-        <%= button_to 'Diff revisions', diff_node_revisions_path(@node),
+        <%= button_to t(".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>
-        <label><%= radio_button_tag :view, 'inline', true %> Inline</label>
-        <label><%= radio_button_tag :view, 'side_by_side', false %> Side by side</label>
+        <span id="diff_selection_label" data-against="<%= t(".against_word") %>" data-selected="<%= t(".selected_word") %>"></span>
+        <label><%= radio_button_tag :view, 'inline', true %> <%= t("revisions.inline") %></label>
+        <label><%= radio_button_tag :view, 'side_by_side', false %> <%= t("revisions.side_by_side") %></label>
       </td>
     </tr>
   </thead>
@@ -53,11 +53,11 @@
       <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, :locale => @locale) %></td>
+      <td><%= link_to t(".show_link"), node_revision_path(@node, page, :locale => @locale) %></td>
       <td>
-        <%= button_to 'restore', restore_node_revision_path(@node, page),
+        <%= button_to t(".restore_link"), restore_node_revision_path(@node, page),
               method: :put,
-              form: { data: { confirm: "Restore this revision?" }, class: 'button_to state_changing' } %>
+              form: { data: { confirm: t(".confirm_restore") }, class: 'button_to state_changing' } %>
       </td>
     </tr>
   <% end %>
@@ -73,9 +73,9 @@
 
     var label = document.getElementById('diff_selection_label');
     if (start && end) {
-      label.textContent = start.value + ' against ' + end.value;
+      label.textContent = start.value + ' ' + label.dataset.against + ' ' + end.value;
     } else if (start || end) {
-      label.textContent = (start || end).value + ' selected';
+      label.textContent = (start || end).value + ' ' + label.dataset.selected;
     } else {
       label.textContent = '';
     }
diff --git a/app/views/revisions/show.html.erb b/app/views/revisions/show.html.erb
index 64bc0eea..a39071c6 100644
--- a/app/views/revisions/show.html.erb
+++ b/app/views/revisions/show.html.erb
@@ -1,39 +1,39 @@
 <% translation = @page.translations.find_by(:locale => @locale) %>
 <div id="admin_layout" class="show_node">
-  <h1>Revision <%= @page.revision %>: <%= translation&.title %> <small>(<%= @locale.to_s.upcase %>)</small></h1>
+  <h1><%= t(".title", :rev => @page.revision) %>: <%= translation&.title %> <small>(<%= @locale.to_s.upcase %>)</small></h1>
 
   <div id="content">
-    <div class="layout_row_label">Actions</div>
+    <div class="layout_row_label"><%= t("admin.columns.actions") %></div>
     <div class="layout_row_content info_group">
       <div class="info_group_items">
         <div class="info_item">
-          <%= link_to 'Show all revisions', node_revisions_path(@node, :locale => @locale) %>
+          <%= link_to t(".show_all"), node_revisions_path(@node, :locale => @locale) %>
         </div>
       </div>
     </div>
 
-    <div class="layout_row_label">Path</div>
+    <div class="layout_row_label"><%= t("admin.columns.path") %></div>
     <div class="layout_row_content"><%= @page.public_link %></div>
 
-    <div class="layout_row_label">Author</div>
+    <div class="layout_row_label"><%= t("admin.columns.author") %></div>
     <div class="layout_row_content"><%= @page.user.try(:login) %></div>
 
-    <div class="layout_row_label">Editor</div>
+    <div class="layout_row_label"><%= t("admin.columns.editor") %></div>
     <div class="layout_row_content"><%= @page.editor.try(:login) %></div>
 
-    <div class="layout_row_label">Last updated</div>
+    <div class="layout_row_label"><%= t("nodes.show.last_updated") %></div>
     <div class="layout_row_content"><%= @page.updated_at %></div>
 
-    <div class="layout_row_label">Published at</div>
+    <div class="layout_row_label"><%= t("nodes.show.published_at") %></div>
     <div class="layout_row_content"><%= @page.published_at %></div>
 
-    <div class="layout_row_label">Title</div>
+    <div class="layout_row_label"><%= Page.human_attribute_name(:title) %></div>
     <div class="layout_row_content"><%= translation&.title %></div>
 
-    <div class="layout_row_label">Abstract</div>
+    <div class="layout_row_label"><%= Page.human_attribute_name(:abstract) %></div>
     <div class="layout_row_content"><%= translation&.abstract %></div>
 
-    <div class="layout_row_label">Body</div>
+    <div class="layout_row_label"><%= Page.human_attribute_name(:body) %></div>
     <div class="layout_row_content"><%= raw translation&.body %></div>
   </div>
 </div>