From b11ef548b652db54c7b1e11f3d7a9511c9522c81 Mon Sep 17 00:00:00 2001
From: Pfingstfrosch <pfingstfrosch@gmail.com>
Date: Sat, 16 May 2020 15:06:42 +0200
Subject: [PATCH] style the table

---
 app/assets/stylesheets/favorites.scss | 21 +++++++
 app/views/favorites/people.html.erb   | 80 +++++++++++++++------------
 2 files changed, 67 insertions(+), 34 deletions(-)

diff --git a/app/assets/stylesheets/favorites.scss b/app/assets/stylesheets/favorites.scss
index c3f5840..f500a8d 100644
--- a/app/assets/stylesheets/favorites.scss
+++ b/app/assets/stylesheets/favorites.scss
@@ -26,4 +26,25 @@
 
 .thumb {
   width: 3em;
+}
+
+.table_wrapper {
+  table {
+    border-collapse: collapse;
+    width: 100%;
+  }
+
+  th, td {
+    padding: 8px;
+    text-align: left;
+    border-bottom: 1px solid #ddd;
+  }
+
+  th .asc {
+    background-image: url("app/assets/images/up.png");
+  }
+
+  th .desc {
+    background-image: url("app/assets/images/down.png");
+  }
 }
\ No newline at end of file
diff --git a/app/views/favorites/people.html.erb b/app/views/favorites/people.html.erb
index 5e5de03..c0db7db 100644
--- a/app/views/favorites/people.html.erb
+++ b/app/views/favorites/people.html.erb
@@ -24,37 +24,49 @@
   <%# end %>
 <!--</ul>-->
 
-<table>
-  <tr>
-    <th>Thumb</th>
-    <th><%= sortable "First Name", "first_name" %></th>
-    <th><%= sortable "Last Name", "last_name" %></th>
-    <th><%= sortable "Registration Date", "created_at" %></th>
-    <th>Del</th>
-  </tr>
-  <% for person in @favorite_people %>
-    <tr>
-      <td>
-        <span>
-          <% if person['picture_uri'] != nil %>
-            <%= image_tag @host + person['picture_uri'], class: 'thumb'%>
-          <% else %>
-            <%= show_svg('account_circle-black-48dp.svg')%>
-          <% end %>
-        </span>
-      </td>
-      <td>
-        <%= person['first_name'] %>
-      </td>
-      <td>
-        <%= person['last_name'] %>
-      </td>
-      <td>
-        <%= person['created_at'] %>
-      </td>
-      <td>
-        <%= link_to "X", {:action => "delete_person", :tiss_id => person['tiss_id']}, :method => 'delete' %>
-      </td>
-    </tr>
-  <% end %>
-</table>
+<% if !@favorite_people[0].blank? %>
+  <div class="table_wrapper">
+    <table>
+      <tr>
+        <th></th>
+        <th><%= sortable "First Name", "first_name" %></th>
+        <th><%= sortable "Last Name", "last_name" %></th>
+        <th><%= sortable "Registration Date", "created_at" %></th>
+        <th></th>
+        <th></th>
+      </tr>
+      <% for person in @favorite_people %>
+        <tr>
+          <td>
+            <%= link_to crawlers_people_crawler_show_detail_url(:tiss_id => person['tiss_id']) do %>
+            <span>
+              <% if person['picture_uri'] != nil %>
+                <%= image_tag @host + person['picture_uri'], class: 'thumb'%>
+              <% else %>
+                <%= show_svg('account_circle-black-48dp.svg')%>
+              <% end %>
+            </span>
+            <% end %>
+          </td>
+          <td>
+            <%= person['first_name'] %>
+          </td>
+          <td>
+            <%= person['last_name'] %>
+          </td>
+          <td>
+            <%= person['created_at'] %>
+          </td>
+          <td>
+            <%= button_to 'Details', crawlers_people_crawler_show_detail_url(:tiss_id => person['tiss_id']) %>
+          </td>
+          <td>
+            <%= button_to "Delete", {:action => "delete_person", :tiss_id => person['tiss_id']}, :method => 'delete' %>
+          </td>
+        </tr>
+      <% end %>
+    </table>
+  </div>
+<% else %>
+  No favorite people added yet!
+<% end %>
\ No newline at end of file