From 7a888b71fd93564c4ce6799bc3356698fdd506c8 Mon Sep 17 00:00:00 2001
From: decabeza <alberto@decabeza.es>
Date: Wed, 28 Feb 2018 16:10:14 +0100
Subject: [PATCH] Removes duplicate heading info, fixes headings structure

---
 app/assets/stylesheets/participation.scss      |  4 ++--
 app/views/budgets/investments/_header.html.erb | 16 +++++++++++-----
 app/views/budgets/investments/index.html.erb   |  4 ----
 3 files changed, 13 insertions(+), 11 deletions(-)

diff --git a/app/assets/stylesheets/participation.scss b/app/assets/stylesheets/participation.scss
index aa1259bb3..404bea627 100644
--- a/app/assets/stylesheets/participation.scss
+++ b/app/assets/stylesheets/participation.scss
@@ -1152,12 +1152,12 @@
     min-height: $line-height * 10;
   }
 
-  h1 {
+  h2 {
     margin-bottom: 0;
   }
 
-  h1,
   h2,
+  h3,
   .back,
   .icon-angle-left,
   p,
diff --git a/app/views/budgets/investments/_header.html.erb b/app/views/budgets/investments/_header.html.erb
index d803ae9d6..c664abc01 100644
--- a/app/views/budgets/investments/_header.html.erb
+++ b/app/views/budgets/investments/_header.html.erb
@@ -23,13 +23,17 @@
             </div>
             <% if @ballot.valid_heading?(@heading) %>
               <div class="progress-bar-nav" data-fixed-bar>
-                <h1><%= @heading.name %></h1>
+                <h2>
+                  <%= t("budgets.investments.index.by_heading", heading: @heading.name) %>
+                </h2>
                 <div id="progress_bar" class="no-margin-top">
                   <%= render 'budgets/ballot/progress_bar' %>
                 </div>
               </div>
             <% else %>
-              <h1><%= @heading.name %></h1>
+              <h2>
+                <%= t("budgets.investments.index.by_heading", heading: @heading.name) %>
+              </h2>
               <div class="small-12 medium-9">
                 <div class="callout warning">
                   <%= t("budgets.investments.header.different_heading_assigned_html",
@@ -47,11 +51,13 @@
               </div>
             <% end %>
           <% else %>
-            <h1 class="margin-top"><%= @heading.name %></h1>
-            <h2>
+            <h2 class="margin-top">
+              <%= t("budgets.investments.index.by_heading", heading: @heading.name) %>
+            </h2>
+            <h3>
               <span class="tagline"><%= t("budgets.investments.header.price") %></span>
               <%= @budget.formatted_heading_price(@heading) %>
-            </h2>
+            </h3>
           <% end %>
         </div>
       </div>
diff --git a/app/views/budgets/investments/index.html.erb b/app/views/budgets/investments/index.html.erb
index 6ca99cf14..8505f9248 100644
--- a/app/views/budgets/investments/index.html.erb
+++ b/app/views/budgets/investments/index.html.erb
@@ -43,10 +43,6 @@
               definitions: link_to(t("budgets.investments.index.unfeasible_text_definitions"), "https://decide.madrid.es/participatory_budget_info#20")).html_safe %>
           </div>
         </div>
-      <% elsif @heading.present? %>
-        <div class="small-12 margin-bottom">
-          <h2><%= t("budgets.investments.index.by_heading", heading: @heading.name) %></h2>
-        </div>
       <% end %>
 
       <%= render("shared/advanced_search", search_path: budget_investments_url(@budget)) %>