From 995449d73308bfcc52c29e57cf3c321bdca0beb0 Mon Sep 17 00:00:00 2001
From: decabeza <alberto@decabeza.es>
Date: Fri, 16 Feb 2018 13:28:30 +0100
Subject: [PATCH] Adds share buttons and refactors processes header

---
 .../legislation/processes/_header.html.erb    | 59 ++++++++++---------
 .../processes/_header_full.html.erb           | 44 +++++---------
 config/locales/en/legislation.yml             |  7 +--
 config/locales/es/legislation.yml             |  7 +--
 4 files changed, 51 insertions(+), 66 deletions(-)

diff --git a/app/views/legislation/processes/_header.html.erb b/app/views/legislation/processes/_header.html.erb
index d228673de..3c0d358f9 100644
--- a/app/views/legislation/processes/_header.html.erb
+++ b/app/views/legislation/processes/_header.html.erb
@@ -1,34 +1,37 @@
-<% if header == :small %>
-  <div class="legislation-hero jumbo grey-heading">
-    <div class="row headline">
-      <div class="small-12 medium-8 column">
-        <h3 class="headline-small"><%= process.title %></h3>
-      </div>
-    </div>
+<div class="legislation-hero jumbo">
+  <div class="row">
+    <div class="small-12 medium-9 column">
+      <%= back_link_to legislation_processes_path %>
+      <h2><%= @process.title %></h2>
 
-    <div id="debate-show" class="row description is-hidden" data-toggler=".is-hidden">
-      <div class="small-12 column">
-        <% if process.description.present? %>
-          <p class="title"><%= t('legislation.processes.header_full.description') %></p>
-          <%= markdown process.description %>
-        <% end %>
-      </div>
-      <% if process.additional_info.present? %>
-        <div class="small-12 column debate-add-info">
-          <%= markdown process.additional_info if process.additional_info %>
+      <% if header == :small %>
+        <div id="legislation_info" class="is-hidden" data-toggler=".is-hidden">
+
+          <% if @process.description.present? %>
+            <p class="title"><%= t("legislation.processes.header.description") %></p>
+            <%= markdown @process.description %>
+          <% end %>
+
+          <% if @process.additional_info.present? %>
+            <hr>
+            <p class="title"><%= t("legislation.processes.header.additional_info") %></p>
+            <%= markdown @process.additional_info if @process.additional_info %>
+          <% end %>
         </div>
+
+        <a class="button" data-toggle="legislation_info">
+          <%= t("legislation.processes.header.more_info") %>
+        </a>
+      <% else %>
+        <%= render 'legislation/processes/header_full', process: @process %>
       <% end %>
     </div>
 
-    <% if process.description.present? || process.additional_info.present? %>
-      <div class="text-center">
-        <a class="button big text-center button-circle" title="<%= t('.view_process_information') %>" id="js-toggle-small-debate" data-toggle="debate-show">
-          <span class="icon-angle-down" aria-hidden="true"></span>
-        </a>
-      </div>
-    <% end %>
+    <aside class="small-12 medium-3 column">
+      <%= render partial: 'shared/social_share', locals: {
+        share_title: t("proposals.show.share"),
+        title: @process.title,
+        url: legislation_process_url(@process) } %>
+    </aside>
   </div>
-
-<% else %>
-  <%= render 'legislation/processes/header_full', process: @process %>
-<% end %>
+</div>
diff --git a/app/views/legislation/processes/_header_full.html.erb b/app/views/legislation/processes/_header_full.html.erb
index 4505a9ba5..c1d490331 100644
--- a/app/views/legislation/processes/_header_full.html.erb
+++ b/app/views/legislation/processes/_header_full.html.erb
@@ -1,32 +1,16 @@
-<div class="legislation-hero jumbo grey-heading">
-  <div class="row headline">
-    <div class="small-12 medium-8 column">
-      <%= back_link_to legislation_processes_path %>
-      <h2>
-        <%= process.title %>
-      </h2>
-    </div>
+<% if @process.description.present? %>
+  <p class="title"><%= t("legislation.processes.header.description") %></p>
+  <%= markdown @process.description %>
+<% end %>
+
+<% if @process.additional_info.present? %>
+  <div id="additional_info" class="is-hidden" data-toggler=".is-hidden">
+    <hr>
+    <p class="title"><%= t("legislation.processes.header.additional_info") %></p>
+    <%= markdown @process.additional_info if @process.additional_info %>
   </div>
 
-  <div class="row description">
-    <div class="small-12 column">
-      <% if process.description.present? %>
-        <p class="title"><%= t('.description') %></p>
-        <%= markdown process.description %>
-      <% end %>
-    </div>
-    <% if process.additional_info.present? %>
-      <div id="additional_info" class="small-12 column debate-add-info is-hidden" data-toggler=".is-hidden">
-        <%= markdown process.additional_info if process.additional_info %>
-      </div>
-    <% end %>
-  </div>
-
-  <% if process.additional_info.present? %>
-    <div class="text-center">
-      <a id="js-toggle-debate" class="button big text-center" title="<%= t('.more_info') %>" data-toggle="additional_info">
-        <strong><%= t('.more_info') %></strong>
-      </a>
-    </div>
-  <% end %>
-</div>
+  <a class="button" data-toggle="additional_info">
+    <%= t("legislation.processes.header.additional_info") %>
+  </a>
+<% end %>
diff --git a/config/locales/en/legislation.yml b/config/locales/en/legislation.yml
index 00f2263f4..b0f387997 100644
--- a/config/locales/en/legislation.yml
+++ b/config/locales/en/legislation.yml
@@ -47,15 +47,14 @@ en:
         text_comments: Comments
     processes:
       header:
-        view_process_information: View process information
+        additional_info: Additional information
+        description: Description
+        more_info: More information and context
       proposals:
         empty_proposals: There are no proposals
       debate:
         empty_questions: There aren't any questions
         participate: Participate in the debate
-      header_full:
-        description: Description
-        more_info: More information and context
       index:
         filter: Filter
         filters:
diff --git a/config/locales/es/legislation.yml b/config/locales/es/legislation.yml
index 9249abe01..e4654f6a6 100644
--- a/config/locales/es/legislation.yml
+++ b/config/locales/es/legislation.yml
@@ -47,15 +47,14 @@ es:
         text_comments: Comentarios
     processes:
       header:
-        view_process_information: Ver información del proceso
+        additional_info: Información adicional
+        description: En qué consiste
+        more_info: Más información y contexto
       proposals:
         empty_proposals: No hay propuestas
       debate:
         empty_questions: No hay preguntas
         participate: Realiza tus aportaciones al debate previo participando en los siguientes temas.
-      header_full:
-        description: En qué consiste
-        more_info: Más información y contexto
       index:
         filter: Filtro
         filters: