From e58b4e6943dffd431669b43c311d4e03e535b335 Mon Sep 17 00:00:00 2001
From: Dobrica Pavlinusic <dpavlin@rot13.org>
Date: Thu, 3 Jun 2010 22:34:40 +0200
Subject: [PATCH] reformat edits into table

---
 public/mojo_facets.css          | 13 ++++---------
 templates/changes/edits.html.ep | 26 ++++++++++++++++++--------
 2 files changed, 22 insertions(+), 17 deletions(-)

diff --git a/public/mojo_facets.css b/public/mojo_facets.css
index 93d83b6..09a8fb6 100644
--- a/public/mojo_facets.css
+++ b/public/mojo_facets.css
@@ -174,26 +174,21 @@ a.facet {
 	text-decoration: line-through;
 }
 
-.edit-options {
-	margin-top: 3em;
-	float: right;
-	text-align: center;
-	
-}
+/* /changes/edits */
 
 div.edit pre {
 	color: #000;
 }
 
 .edit-unknown {
-	border-bottom: 1px dotted #ccc;
+	background: #eee;
 }
 
 .edit-found {
-	border: 3px dotted #cfc;
+	background: #cfc;
 }
 
 .edit-missing {
-	border: 3px dotted #fcc;
+	background: #fcc;
 }
 
diff --git a/templates/changes/edits.html.ep b/templates/changes/edits.html.ep
index 0687c8f..7192346 100644
--- a/templates/changes/edits.html.ep
+++ b/templates/changes/edits.html.ep
@@ -20,20 +20,30 @@
 </ul>
 % }
 
+<table>
+<tr><th></th><th>old</th><th>new</th></tr>
+
 % foreach my $e ( @$edits ) {
-<div class="edit edit-<%= $e->{_status} || 'unknown' %>">
-<span class="edit-options">
-<a href="<%= url_for( action => 'edit' )->query( remove => $e->{time} ) %>">remove</a>
+<tr class="edit edit-<%= $e->{_status} || 'unknown' %>">
+<td>
+<%= $e->{time} %>
+
 %  if ( my $status = $e->{_status} ) {
-<div title="status">
+%    my ( $pk, $id ) = %{ $e->{unique} };
+<a href="<%= url_for( controller => 'data', action => 'filter' )->query( filter_name => $pk, filter_vals => $id ) %>"><%= $id %></a>
 <%= $status %>
-</div>
+%  } else {
+<a href="<%= url_for( action => 'edit' )->query( remove => $e->{time} ) %>">remove</a>
 %  }
-</span>
-<pre class=debug><%= dumper $e %></pre>
-</div>
+
+</td><td><%== defined $e->{old} && join('<span class=d>&para;</span>', @{$e->{old}}) %>
+</td><td><%== defined $e->{new} && join('<span class=d>&para;</span>', @{$e->{new}}) %>
+
+</tr>
 % }
 
+</table>
+
 <pre class=debug>
 <%= dumper( $stats ) %>
 </pre>
-- 
2.20.1