From 80279ca638b9edf7ba23d19ee9385806b7f379c7 Mon Sep 17 00:00:00 2001
From: Alberto Garcia Cabeza <alberto@decabeza.es>
Date: Fri, 28 Apr 2017 12:44:38 +0200
Subject: [PATCH] improves admin banners index page

---
 app/views/admin/banners/index.html.erb | 72 +++++++++++++++-----------
 config/locales/admin.en.yml            |  1 +
 config/locales/admin.es.yml            |  1 +
 3 files changed, 44 insertions(+), 30 deletions(-)

diff --git a/app/views/admin/banners/index.html.erb b/app/views/admin/banners/index.html.erb
index db58c1303..11f2ef6ab 100644
--- a/app/views/admin/banners/index.html.erb
+++ b/app/views/admin/banners/index.html.erb
@@ -7,36 +7,48 @@
 
 <h3><%= page_entries_info @banners %></h3>
 
-<table>
-  <% @banners.each do |banner| %>
-    <tr id="<%= dom_id(banner) %>">
-      <td class="small-12 medium-9">
-        <small>
-        <%= t("admin.banners.banner.post_started_at")%> <strong><%= banner.post_started_at %></strong>
-        &nbsp;|&nbsp;
-        <%= t("admin.banners.banner.post_ended_at")%> <strong><%= banner.post_ended_at%></strong>
-        </small>
-
-        <div class="<%= banner.style %>">
-          <div class="<%= banner.image %>">
-            <%= link_to banner.target_url do %>
-              <h2><%= banner.title %></h2>
-              <h3><%= banner.description %></h3>
-           <% end %>
+<% @banners.each do |banner| %>
+  <table>
+    <thead>
+      <tr id="<%= dom_id(banner) %>">
+        <th scope="col"><%= t("admin.banners.banner.post_started_at")%></th>
+        <th scope="col"><%= t("admin.banners.banner.post_ended_at")%></th>
+        <th scope="col" class="text-center"><%= t("admin.actions.actions")%></th>
+      </tr>
+    </thead>
+    <tbody>
+      <tr>
+        <td><%= banner.post_started_at %></td>
+        <td><%= banner.post_ended_at %></td>
+        <td class="text-right">
+          <div class="small-12 medium-6 column">
+            <%= link_to t("admin.banners.index.edit"), edit_admin_banner_path(banner),
+                        class: 'button hollow expanded' %>
           </div>
-        </div>
-      </td>
-
-      <td class="text-right">
-        <%= link_to t("admin.banners.index.edit"), edit_admin_banner_path(banner),
-                    class: 'edit-banner button hollow' %>
-
-        <%= link_to t("admin.banners.index.delete"), admin_banner_path(banner),
-                    method: :delete,
-                    class: 'button hollow alert' %>
-      </td>
-    </tr>
-  <% end %>
-</table>
+          <div class="small-12 medium-6 column">
+            <%= link_to t("admin.banners.index.delete"), admin_banner_path(banner),
+                        method: :delete,
+                        class: 'button hollow alert expanded' %>
+          </div>
+        </td>
+      </tr>
+      <tr>
+        <th colspan="3" scope="col" class="text-center"><%= t("admin.banners.index.preview") %></th>
+      </tr>
+      <tr>
+        <td colspan="3">
+          <div class="<%= banner.style %>">
+            <div class="<%= banner.image %>">
+              <%= link_to banner.target_url do %>
+                <h2><%= banner.title %></h2>
+                <h3><%= banner.description %></h3>
+              <% end %>
+            </div>
+          </div>
+        </td>
+      </tr>
+    </tbody>
+  </table>
+<% end %>
 
 <%= paginate @banners %>
\ No newline at end of file
diff --git a/config/locales/admin.en.yml b/config/locales/admin.en.yml
index f99ef1780..f6b796b1c 100755
--- a/config/locales/admin.en.yml
+++ b/config/locales/admin.en.yml
@@ -24,6 +24,7 @@ en:
           all: All
           with_active: Active
           with_inactive: Inactive
+        preview: Preview
       banner:
         title: Title
         description:  Description
diff --git a/config/locales/admin.es.yml b/config/locales/admin.es.yml
index 817576494..951b34a2b 100644
--- a/config/locales/admin.es.yml
+++ b/config/locales/admin.es.yml
@@ -24,6 +24,7 @@ es:
           all: Todos
           with_active: Activos
           with_inactive: Inactivos
+        preview: Vista previa
       banner:
         title: Título
         description:  Descripción