Merge pull request #3042 from consul/pages-texts

Pages texts

app/views/pages/accessibility.html.erb

@@ -1,35 +1,41 @@
-<% provide :title do %><%= t('pages.titles.accessibility') %><% end %>
+<% provide :title do %><%= t("pages.titles.accessibility") %><% end %>
 <% content_for :canonical do %>
   <%= render "shared/canonical", href: page_url("accessibility") %>
 <% end %>
 
 <div class="row">
-<div class="small-12 column">
+  <div class="small-12 column">
 
-  <h1><%= t('.title') %></h1>
+    <h1><%= t("pages.accessibility.title") %></h1>
-  <%= simple_format(t('.description')) %>
+    <%= simple_format(t("pages.accessibility.description")) %>
 
     <ul>
-    <% t('.examples').each do |example| %>
+      <% t("pages.accessibility.examples").each do |example| %>
         <li><%= example %></li>
       <% end %>
     </ul>
 
-  <h2><%= t('.keyboard_shortcuts.title') %></h2>
+    <h2><%= t("pages.accessibility.keyboard_shortcuts.title") %></h2>
 
-  <p><%= t('.keyboard_shortcuts.navigation_table.description') %></p>
+    <p><%= t("pages.accessibility.keyboard_shortcuts.navigation_table.description") %></p>
 
     <div class="small-12 medium-6">
       <table>
-      <caption class="show-for-sr"><%= t('.keyboard_shortcuts.navigation_table.caption') %></caption>
+        <caption class="show-for-sr">
+          <%= t("pages.accessibility.keyboard_shortcuts.navigation_table.caption") %>
+        </caption>
         <thead>
           <tr>
-          <th scope="col" class="text-center"><%= t('.keyboard_shortcuts.navigation_table.key_header') %></th>
+            <th scope="col" class="text-center">
-          <th scope="col"><%= t('.keyboard_shortcuts.navigation_table.page_header') %></th>
+              <%= t("pages.accessibility.keyboard_shortcuts.navigation_table.key_header") %>
+            </th>
+            <th scope="col">
+              <%= t("pages.accessibility.keyboard_shortcuts.navigation_table.page_header") %>
+            </th>
           </tr>
         </thead>
         <tbody>
-        <% t('.keyboard_shortcuts.navigation_table.rows').each do |row| %>
+          <% t("pages.accessibility.keyboard_shortcuts.navigation_table.rows").each do |row| %>
             <tr>
               <td class="text-center"><%= row[:key_column] %></td>
               <td><%= row[:page_column] %></td>
@@ -39,19 +45,25 @@
       </table>
     </div>
 
-  <p><%= t('.keyboard_shortcuts.browser_table.description') %></p>
+    <p><%= t("pages.accessibility.keyboard_shortcuts.browser_table.description") %></p>
 
     <div class="small-12 medium-6">
       <table>
-      <caption class="show-for-sr"><%= t('.keyboard_shortcuts.browser_table.caption') %></caption>
+        <caption class="show-for-sr">
+          <%= t("pages.accessibility.keyboard_shortcuts.browser_table.caption") %>
+        </caption>
         <thead>
           <tr>
-          <th scope="col"><%= t('.keyboard_shortcuts.browser_table.browser_header') %></th>
+            <th scope="col">
-          <th scope="col"><%= t('.keyboard_shortcuts.browser_table.key_header') %></th>
+              <%= t("pages.accessibility.keyboard_shortcuts.browser_table.browser_header") %>
+            </th>
+            <th scope="col">
+              <%= t("pages.accessibility.keyboard_shortcuts.browser_table.key_header") %>
+            </th>
           </tr>
         </thead>
         <tbody>
-        <% t('.keyboard_shortcuts.browser_table.rows').each do |row| %>
+          <% t("pages.accessibility.keyboard_shortcuts.browser_table.rows").each do |row| %>
             <tr>
               <td><%= row[:browser_column] %></td>
               <td><%= row[:key_column] %></td>
@@ -61,19 +73,23 @@
       </table>
     </div>
 
-  <h2><%= t('.textsize.title') %></h2>
+    <h2><%= t("pages.accessibility.textsize.title") %></h2>
-  <p><%= t('.textsize.browser_settings_table.description') %></p>
+    <p><%= t("pages.accessibility.textsize.browser_settings_table.description") %></p>
 
     <div class="small-12 medium-6">
       <table>
         <thead>
           <tr>
-          <th scope="col"><%= t('.textsize.browser_settings_table.browser_header') %></th>
+            <th scope="col">
-          <th scope="col"><%= t('.textsize.browser_settings_table.action_header') %></th>
+              <%= t("pages.accessibility.textsize.browser_settings_table.browser_header") %>
+            </th>
+            <th scope="col">
+              <%= t("pages.accessibility.textsize.browser_settings_table.action_header") %>
+            </th>
           </tr>
         </thead>
         <tbody>
-        <% t('.textsize.browser_settings_table.rows').each do |row| %>
+          <% t("pages.accessibility.textsize.browser_settings_table.rows").each do |row| %>
             <tr>
               <td><%= row[:browser_column] %></td>
               <td><%= row[:action_column] %></td>
@@ -83,17 +99,16 @@
       </table>
     </div>
 
-  <p><%= t('.textsize.browser_shortcuts_table.description') %></p>
+    <p><%= t("pages.accessibility.textsize.browser_shortcuts_table.description") %></p>
 
     <ul>
-    <% t('.textsize.browser_shortcuts_table.rows').each do |row| %>
+      <% t("pages.accessibility.textsize.browser_shortcuts_table.rows").each do |row| %>
         <li><code><%= row[:shortcut_column] %></code> <%= row[:description_column] %></li>
       <% end %>
     </ul>
 
-  <h2><%= t('.compatibility.title') %></h2>
+    <h2><%= t("pages.accessibility.compatibility.title") %></h2>
-
-  <p><%= t('.compatibility.description_html') %></p>
 
+    <p><%= t("pages.accessibility.compatibility.description_html") %></p>
   </div>
 </div>

app/views/pages/census_terms.html.erb

@@ -1,11 +0,0 @@
-<% content_for :canonical do %>
-  <%= render "shared/canonical", href: page_url("census_terms") %>
-<% end %>
-
-<div class="page row-full">
-  <div class="row">
-    <div class="text small-12 column">
-      <h2>Terminos de acceso al Padrón</h2>
-    </div>
-  </div>
-</div>

app/views/pages/conditions.html.erb

@@ -1,4 +1,4 @@
-<% provide :title do %><%= t('pages.titles.conditions') %><% end %>
+<% provide :title do %><%= t("pages.titles.conditions") %><% end %>
 <% content_for :canonical do %>
   <%= render "shared/canonical", href: page_url("conditions") %>
 <% end %>
@@ -6,9 +6,9 @@
 <div class="row margin-top">
 
   <div class="small-12 medium-9 column">
-    <h1><%= t('.title') %></h1>
+    <h1><%= t("pages.conditions.title") %></h1>
-    <h2><%= t('.subtitle') %></h2>
+    <h2><%= t("pages.conditions.subtitle") %></h2>
-    <p><%= t('.description') %></p>
+    <p><%= t("pages.conditions.description") %></p>
   </div>
 
   <div class="small-12 medium-3 column">

app/views/pages/general_terms.html.erb

@@ -1,11 +0,0 @@
-<% content_for :canonical do %>
-  <%= render "shared/canonical", href: page_url("general_terms") %>
-<% end %>
-
-<div class="row-full page">
-  <div class="row">
-    <div class="small-12 column">
-      <h1><%= t('pages.general_terms') %></h1>
-    </div>
-  </div>
-</div>

app/views/pages/privacy.html.erb

@@ -1,4 +1,4 @@
-<% provide :title do %><%= t('pages.titles.privacy') %><% end %>
+<% provide :title do %><%= t("pages.titles.privacy") %><% end %>
 <% content_for :canonical do %>
   <%= render "shared/canonical", href: page_url("privacy") %>
 <% end %>
@@ -6,12 +6,11 @@
 <div class="row margin-top">
 
   <div class="small-12 medium-9 column">
-
+    <h1><%= t("pages.privacy.title") %></h1>
-    <h1><%= t(".title") %></h1>
+    <h2><%= t("pages.privacy.subtitle") %></h2>
-    <h2><%= t('.subtitle') %></h2>
 
     <ol>
-      <% t('.info_items').each do |item| %>
+      <% t("pages.privacy.info_items").each do |item| %>
         <% if item.key? :text %>
           <li><%= item[:text] %></li>
         <% else %>
@@ -26,12 +25,9 @@
         <% end %>
       <% end %>
     </ol>
-
   </div>
 
   <div class="small-12 medium-3 column">
     <%= render '/shared/print' %>
   </div>
-
 </div>
-

config/locales/en/pages.yml

@@ -4,7 +4,6 @@ en:
       title: Terms and conditions of use
       subtitle: LEGAL NOTICE ON THE CONDITIONS OF USE, PRIVACY AND PROTECTION OF PERSONAL DATA OF THE OPEN GOVERNMENT PORTAL
       description: Information page on the conditions of use, privacy and protection of personal data.
-    general_terms: Terms and Conditions
     help:
       title: "%{org} is a platform for citizen participation"
       guide: "This guide explains what each of the %{org} sections are for and how they work."
@@ -80,8 +79,6 @@ en:
             description: Managing participatory processes to control the qualification of the people participating in them and merely numerical and statistical recount of the results derived from citizen participation processes.
           - field: 'Institution in charge of the file:'
             description: INSTITUTION IN CHARGE OF THE FILE
-        - text: The interested party may exercise the rights of access, rectification, cancellation and opposition, before the responsible body indicated, all of which is reported in compliance with article 5 of the Organic Law 15/1999, of December 13, on the Protection of Data of Character Personal.
-        - text: As a general principle, this website does not share or disclose information obtained, except when it has been authorized by the user, or the information is required by the judicial authority, prosecutor's office or the judicial police, or any of the cases regulated in the Article 11 of the Organic Law 15/1999, of December 13, on the Protection of Personal Data.
     accessibility:
       title: Accessibility
       description: |-
@@ -155,8 +152,7 @@ en:
               description_column: Decreases text size
       compatibility:
         title: Compatibility with standards and visual design
-        description_html: 'All pages of this website comply with the <strong> Accessibility Guidelines </strong> or General Principles of Accessible Design established by the Working Group <abbr title = "Web Accessibility Initiative" lang = "en"> WAI </ abbr> belonging to W3C.'
+        description_html: 'All pages of this website comply with the <strong>Accessibility Guidelines</strong> or General Principles of Accessible Design established by the Working Group <abbr title="Web Accessibility Initiative" lang="en">WAI</abbr> belonging to W3C.'
-
     titles:
       accessibility: Accessibility
       conditions: Terms of use

spec/controllers/pages_controller_spec.rb

@@ -13,16 +13,6 @@ describe PagesController do
       expect(response).to be_ok
     end
 
-    it 'includes a general terms page' do
-      get :show, id: :general_terms
-      expect(response).to be_ok
-    end
-
-    it 'includes a terms page' do
-      get :show, id: :census_terms
-      expect(response).to be_ok
-    end
-
     it 'includes a accessibility page' do
       get :show, id: :accessibility
       expect(response).to be_ok