From aac501bfdac140de4116a8cfaebf757af8c00efb Mon Sep 17 00:00:00 2001
From: Alberto Garcia Cabeza <alberto@decabeza.es>
Date: Mon, 26 Dec 2016 17:23:17 +0100
Subject: [PATCH] improves styles for polls questions views

---
 app/views/polls/questions/_answers.html.erb  |  4 +-
 app/views/polls/questions/_question.html.erb |  2 +-
 app/views/polls/questions/show.html.erb      | 77 ++++++++++++--------
 3 files changed, 49 insertions(+), 34 deletions(-)

diff --git a/app/views/polls/questions/_answers.html.erb b/app/views/polls/questions/_answers.html.erb
index c7effd0c0..c88156b68 100644
--- a/app/views/polls/questions/_answers.html.erb
+++ b/app/views/polls/questions/_answers.html.erb
@@ -2,7 +2,7 @@
   <% if can? :answer, question %>
     <% question.valid_answers.each do |answer| %>
       <% if @answers_by_question_id[question.id] == answer %>
-        <span class="button" title="<%= t("poll_questions.show.voted", answer: answer)%>">
+        <span class="button answered" title="<%= t("poll_questions.show.voted", answer: answer)%>">
           <%= answer %>
         </span>
       <% else %>
@@ -16,7 +16,7 @@
     <% end %>
   <% else %>
     <% question.valid_answers.each do |answer| %>
-      <span class="button disabled"><%= answer %></span>
+      <span class="button secondary hollow disabled"><%= answer %></span>
     <% end %>
   <% end %>
 </div>
diff --git a/app/views/polls/questions/_question.html.erb b/app/views/polls/questions/_question.html.erb
index b8040c85b..f0958b6e4 100644
--- a/app/views/polls/questions/_question.html.erb
+++ b/app/views/polls/questions/_question.html.erb
@@ -3,7 +3,7 @@
     <%= link_to question.title, question_path(question) %>
   </h3>
 
-  <div id="<%= dom_id(question) %>_answers">
+  <div id="<%= dom_id(question) %>_answers" class="padding">
     <%= render 'polls/questions/answers', question: question %>
   </div>
 </div>
diff --git a/app/views/polls/questions/show.html.erb b/app/views/polls/questions/show.html.erb
index 76265797c..3f4cb0851 100644
--- a/app/views/polls/questions/show.html.erb
+++ b/app/views/polls/questions/show.html.erb
@@ -27,52 +27,67 @@
         <span class="title">
           <strong><%= t('poll_questions.show.author') %></strong>
         </span>
-      </p>
-      <p>
-        <%= link_to @question.author.name, @question.author, class: "margin-bottom" %>
+        <br>
+        <%= link_to @question.author.name, @question.author %>
       </p>
 
       <p>
-      <span class="title">
-        <strong><%= t('poll_questions.show.dates_title') %></strong>
+        <span class="title">
+          <strong><%= t('poll_questions.show.poll') %></strong>
+        </span>
+        <br>
+        <%= link_to @question.poll.name, @question.poll %>
+      </p>
+
+      <p>
+        <span class="title">
+          <strong><%= t('poll_questions.show.dates_title') %></strong>
+        </span>
+        <br>
+        <%= poll_dates(@question.poll) %>
       </p>
-      <p><%= poll_dates(@question.poll) %></p>
     </div>
   </div>
 </div>
 
-<% if @question.poll.current? %>
 <div class="row margin-top">
   <div class="small-12 column">
-    <% if current_user.nil? %>
+    <% if @question.poll.current? %>
+
+      <% if current_user.nil? %>
+        <div class="callout primary">
+          <%= t("poll_questions.show.not_logged_in",
+              signin: link_to(t("poll_questions.show.signin"), new_user_session_path, class: "probe-message"),
+              signup: link_to(t("poll_questions.show.signup"), new_user_registration_path, class: "probe-message")).html_safe %>
+        </div>
+      <% elsif current_user.unverified? %>
+        <div class="callout warning">
+          <%= t('poll_questions.show.cant_answer_verify_html',
+                 verify_link: link_to(t('poll_questions.show.verify_link'), verification_path)) %>
+        </div>
+      <% else %>
+        <div class="callout warning">
+          <%= t('poll_questions.show.cant_answer_wrong_geozone') %>
+        </div>
+      <% end %>
+
+      <div id="<%= dom_id(@question) %>_answers">
+        <div class="small-12 small-centered text-center column">
+          <%= render "answers", question: @question %>
+        </div>
+      </div>
+
+    <% elsif @question.poll.incoming? %>
+      <div class="callout alert">
+        <%= t('poll_questions.show.cant_answer_expired') %>
+      </div>
+    <% elsif @question.poll.expired? %>
       <div class="callout primary">
-        <%= t("poll_questions.show.not_logged_in",
-            signin: link_to(t("poll_questions.show.signin"), new_user_session_path, class: "probe-message"),
-            signup: link_to(t("poll_questions.show.signup"), new_user_registration_path, class: "probe-message")).html_safe %>
-      </div>
-    <% elsif current_user.unverified? %>
-      <div class="callout warning">
-        <%= t('poll_questions.show.cant_answer_verify_html',
-               verify_link: link_to(t('poll_questions.show.verify_link'), verification_path)) %>
-      </div>
-    <% else %>
-      <div class="callout warning">
-        <%= t('poll_questions.show.cant_answer_wrong_geozone') %>
+        <%= t('poll_questions.show.cant_answer_incoming') %>
       </div>
     <% end %>
   </div>
-
-  <div id="<%= dom_id(@question) %>_answers">
-    <div class="small-12 small-centered text-center column">
-      <%= render "answers", question: @question %>
-    </div>
-  </div>
 </div>
-<% elsif @question.poll.incoming? %>
-  <%= t('poll_questions.show.cant_answer_expired') %>
-<% elsif @question.poll.expired? %>
-  <%= t('poll_questions.show.cant_answer_incoming') %>
-<% end %>
 
 <div class="row margin-top">
   <div class="small-12 medium-9 column">