From fcb6e67cedb2d32d04f6608f263ae9f52f1e98a0 Mon Sep 17 00:00:00 2001
From: decabeza <alberto@decabeza.es>
Date: Fri, 5 Oct 2018 18:10:10 +0200
Subject: [PATCH] Adds styles to proposals views

---
 app/views/proposals/created.html.erb | 26 ++++++++++++++++++--------
 app/views/proposals/share.html.erb   | 16 +++++++++-------
 app/views/proposals/show.html.erb    |  7 ++++---
 app/views/users/_proposal.html.erb   | 10 ++++++----
 4 files changed, 37 insertions(+), 22 deletions(-)

diff --git a/app/views/proposals/created.html.erb b/app/views/proposals/created.html.erb
index d973ceba0..215a5e151 100644
--- a/app/views/proposals/created.html.erb
+++ b/app/views/proposals/created.html.erb
@@ -1,13 +1,23 @@
-<div class=proposal-created>
+<div class="jumbo light">
   <div id="<%= dom_id(@proposal) %>" class="row">
-      <div class="small-12 medium-12 column">
-        <h1><%= t("proposals.created.title") %></h1>
+    <div class="small-12 column">
+      <h2><%= t("proposals.created.title") %></h2>
 
-        <%= raw t("proposals.created.motivation_html") %>
-        
-        <%= link_to t("proposals.created.dashboard"), proposal_dashboard_path(@proposal), class: "button" if can?(:dashboard, @proposal) %>
-        <%= link_to t("proposals.created.publish"), publish_proposal_path(@proposal), method: :patch, class: "button" if can?(:publish, @proposal) %>
-      </div>
+      <p><%= t("proposals.created.motivation") %></p>
+      <p><%= t("proposals.created.motivation_2_html") %></p>
+
+      <% if can?(:dashboard, @proposal) %>
+        <%= link_to t("proposals.created.dashboard"),
+                    proposal_dashboard_path(@proposal),
+                    class: "button" %>
+      <% end %>
+
+      <% if can?(:publish, @proposal) %>
+        <%= link_to t("proposals.created.publish"),
+                    publish_proposal_path(@proposal),
+                    method: :patch, class: "button" %>
+      <% end %>
+    </div>
   </div>
 </div>
 
diff --git a/app/views/proposals/share.html.erb b/app/views/proposals/share.html.erb
index 110f89822..f1a6e289b 100644
--- a/app/views/proposals/share.html.erb
+++ b/app/views/proposals/share.html.erb
@@ -27,13 +27,14 @@
           description: @proposal.summary
         } %>
 
-        <% if can?(:dashboard, @proposal) %> 
-          <div class="callout light">
-            <p class="centered">
-              <strong><%= t('.improve_it') %></strong>
+        <% if can?(:dashboard, @proposal) %>
+          <div class="callout light margin">
+            <p class="text-center">
+              <strong><%= t("proposals.share.improve_it") %></strong>
             </p>
-            <div class="centered">
-              <%= link_to t('.dashboard'), proposal_dashboard_path(@proposal), class: 'button' %>
+            <div class="small-12 medium-6 large-4 column small-centered">
+              <%= link_to t("proposals.share.dashboard"),
+                          proposal_dashboard_path(@proposal), class: "button expanded" %>
             </div>
           </div>
         <% end %>
@@ -41,7 +42,8 @@
         <% if @proposal_improvement_path.present? %>
           <div class="callout highlight margin-top text-center">
             <p class="lead"><strong><%= t("proposals.proposal.improve_info") %></strong></p>
-            <%= link_to t("proposals.proposal.improve_info_link"), @proposal_improvement_path, class: "button" %>
+            <%= link_to t("proposals.proposal.improve_info_link"),
+                        @proposal_improvement_path, class: "button" %>
           </div>
         <% end %>
 
diff --git a/app/views/proposals/show.html.erb b/app/views/proposals/show.html.erb
index 5759487ad..1c7ac61d5 100644
--- a/app/views/proposals/show.html.erb
+++ b/app/views/proposals/show.html.erb
@@ -130,9 +130,10 @@
             <div class="sidebar-divider"></div>
             <h2><%= t("proposals.show.author") %></h2>
             <div class="show-actions-menu">
-              <%= link_to proposal_dashboard_path(@proposal), class: 'button hollow expanded', id: "proposal-dashboard-#{@proposal.id}" do %>
-                          <span class="icon-edit"></span>
-                          <%= t("proposals.show.dashboard_proposal_link") %>
+              <%= link_to proposal_dashboard_path(@proposal), class: "button hollow expanded",
+                          id: "proposal_dashboard_#{@proposal.id}" do %>
+                <span class="icon-edit"></span>
+                <%= t("proposals.show.dashboard_proposal_link") %>
               <% end %>
             </div>
           <% end %>
diff --git a/app/views/users/_proposal.html.erb b/app/views/users/_proposal.html.erb
index 4a350b606..da9dbee9a 100644
--- a/app/views/users/_proposal.html.erb
+++ b/app/views/users/_proposal.html.erb
@@ -1,6 +1,8 @@
 <tr id="proposal_<%= proposal.id %>">
   <td class="small-7">
-    <strong><%= link_to proposal.title, proposal, proposal.retired? ? { class: 'retired' } : {} %></strong>
+    <strong>
+      <%= link_to proposal.title, proposal, proposal.retired? ? { class: 'retired' } : {} %>
+    </strong>
     <br>
     <%= proposal.summary %>
   </td>
@@ -18,12 +20,12 @@
     <td class="help-text"><%= t("users.proposals.retired_help_text") %></td>
   <% elsif can?(:dashboard, proposal) %>
     <td>
-        <%= link_to t("proposals.show.dashboard_proposal_link"), 
-                    proposal_dashboard_path(proposal), class: 'button hollow expanded' %>
+        <%= link_to t("proposals.show.dashboard_proposal_link"),
+                    proposal_dashboard_path(proposal), class: "button hollow expanded" %>
     </td>
   <% else %>
     <td class="text-center">
-      <%= link_to t("users.proposals.see"), proposal, class: 'button hollow' %>
+      <%= link_to t("users.proposals.see"), proposal, class: "button hollow" %>
     </td>
   <% end %>
 </tr>