From 0c798ff9ed5ee9b66ea912b993e625588eb87506 Mon Sep 17 00:00:00 2001
From: Alberto Garcia Cabeza <alberto@decabeza.es>
Date: Wed, 2 Mar 2016 12:41:27 +0100
Subject: [PATCH] Improves styles for admin section

---
 app/assets/stylesheets/admin.scss             | 19 +++++++++++++++++++
 .../admin/spending_proposals/index.html.erb   | 10 ++++++----
 .../admin/spending_proposals/show.html.erb    |  4 ++--
 3 files changed, 27 insertions(+), 6 deletions(-)

diff --git a/app/assets/stylesheets/admin.scss b/app/assets/stylesheets/admin.scss
index ceae3b0ee..bc9d05e48 100644
--- a/app/assets/stylesheets/admin.scss
+++ b/app/assets/stylesheets/admin.scss
@@ -25,6 +25,25 @@ body.admin {
     }
   }
 
+  table {
+
+    tr {
+      background: #f4f4f4;
+
+      &:nth-child(odd) {
+        background: white;
+      }
+
+      &:nth-child(even) {
+        background: #f0f0f0;
+      }
+    }
+
+    td.small {
+      font-size: $small-font-size;
+    }
+  }
+
   #proposals {
     width: 100% !important;
   }
diff --git a/app/views/admin/spending_proposals/index.html.erb b/app/views/admin/spending_proposals/index.html.erb
index 18be63a1b..7c8bca37f 100644
--- a/app/views/admin/spending_proposals/index.html.erb
+++ b/app/views/admin/spending_proposals/index.html.erb
@@ -27,17 +27,19 @@
   <% @spending_proposals.each do |spending_proposal| %>
     <tr id="<%= dom_id(spending_proposal) %>">
       <td>
-        <strong><%= spending_proposal.id %> -
-                <%= link_to spending_proposal.title, admin_spending_proposal_path(spending_proposal) %></strong>
+        <strong><%= spending_proposal.id %></strong>
       </td>
       <td>
+        <%= link_to spending_proposal.title, admin_spending_proposal_path(spending_proposal) %>
+      </td>
+      <td class="small">
         <% if spending_proposal.administrator.present? %>
           <span title="<%= t('admin.spending_proposals.index.admin_assigned') %>"><%= spending_proposal.administrator.name %></span>
         <% else %>
           <%= t("admin.spending_proposals.index.no_admin_assigned") %>
         <% end %>
       </td>
-      <td>
+      <td class="small">
         <% case spending_proposal.valuators.size %>
         <% when 0 %>
           <%= t("admin.spending_proposals.index.no_valuators_assigned") %>
@@ -51,7 +53,7 @@
           </span>
         <% end %>
       </td>
-      <td>
+      <td class="small">
         <%= geozone_name(spending_proposal) %>
       </td>
     </tr>
diff --git a/app/views/admin/spending_proposals/show.html.erb b/app/views/admin/spending_proposals/show.html.erb
index 0396b5454..5a768782a 100644
--- a/app/views/admin/spending_proposals/show.html.erb
+++ b/app/views/admin/spending_proposals/show.html.erb
@@ -40,12 +40,12 @@
   <%= render "assigned_valuators" %>
 </div>
 
-<h3><%= link_to t("admin.spending_proposals.show.assign_valuators"), "", class: "js-toggle-link", data: {"toggle-selector" => "#valuators-assign-list"} %></h3>
+<%= link_to t("admin.spending_proposals.show.assign_valuators"), "", class: "js-toggle-link", data: {"toggle-selector" => "#valuators-assign-list"} %>
 
 <div style="display:none" id="valuators-assign-list" class="row">
   <%= form_for(@spending_proposal, url: assign_valuators_admin_spending_proposal_path(@spending_proposal), remote: true, html: {id: 'valuators_assignment_form'}) do |f| %>
     <% @valuators.each do |valuator| %>
-      <div class="small-12 medium-4 column">
+      <div class="small-12 medium-4 column end">
         <%= check_box_tag "valuator_ids_#{valuator.id}", valuator.id, @spending_proposal.valuators.include?(valuator),
                           name: 'spending_proposal[valuator_ids][]',
                           class: "js-submit-on-change" %>