From 3de185e5a55e6dc3d9072b683f2d4072213c0b5d Mon Sep 17 00:00:00 2001
From: Luke Barratt <luke@barratt.me>
Date: Sun, 27 Apr 2014 02:46:11 +0100
Subject: [PATCH] Improve admin challenge listing

---
 app/assets/stylesheets/components/_tables.scss |  4 ++++
 app/views/challenges/index.html.erb            | 12 ++++--------
 2 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/app/assets/stylesheets/components/_tables.scss b/app/assets/stylesheets/components/_tables.scss
index 3d2e0f0..2a2330a 100644
--- a/app/assets/stylesheets/components/_tables.scss
+++ b/app/assets/stylesheets/components/_tables.scss
@@ -41,4 +41,8 @@ table.striped > tbody {
   > tr:nth-child(2n) td {
     background-color: transparentize($light-blue, 0.9);
   }
+
+  .actions {
+    text-align: right;
+  }
 }
diff --git a/app/views/challenges/index.html.erb b/app/views/challenges/index.html.erb
index 5888c65..c69a461 100644
--- a/app/views/challenges/index.html.erb
+++ b/app/views/challenges/index.html.erb
@@ -1,25 +1,21 @@
-<h1>Listing challenges</h1>
+<h1 class="title">Listing Challenges</h1>
 
-<table class="data">
+<table class="striped">
   <tr>
-    <th></th>
     <th>Challenger</th>
     <th>Recipient</th>
     <th>Match time</th>
-    <th>Default time</th>
-    <th>Server</th>
     <th>Status</th>
+    <th class="actions"></th>
   </tr>
 
   <% @challenges.each do |challenge| %>
     <tr class="<%= cycle('even', 'odd') %>">
-      <td><%= link_to "[X]", challenge, :confirm => 'Are you sure?', :method => :delete %>
       <td><%= namelink challenge.contester1.team %></td>
       <td><%= namelink challenge.contester2.team %></td>
       <td><%= link_to (shorttime challenge.match_time), challenge %></td>
-      <td><%= challenge.mandatory ? shorttime(challenge.default_time) : "--" %></td>
-      <td><%= h challenge.server %></td>
       <td><%= challenge.statuses[challenge.status] %></td>
+      <td class="actions"><%= link_to icon('times'), challenge, confirm: 'Are you sure?', method: :delete %></td>
     </tr>
   <% end %>
 </table>