beautified the revision and diff interface. so pretty and so functional. i'm happy

4 files changed, 39 insertions, 19 deletions

diff --git a/app/controllers/revisions_controller.rb b/app/controllers/revisions_controller.rb
index 6914509..7ae22f8 100644
--- a/app/controllers/revisions_controller.rb
+++ b/app/controllers/revisions_controller.rb
@@ -10,20 +10,20 @@ class RevisionsController < ApplicationController
     
     puts @node.pages.length
     if @node.pages.length > 1
-      params[:start]  ||= @node.pages.all[-2].revision
-      params[:end]    ||= @node.pages.all[-1].revision
+      params[:start_revision]  ||= @node.pages.all[-2].revision
+      params[:end_revision]    ||= @node.pages.all[-1].revision
     else
       params[:start], params[:end] = 1, 1
     end
     
     @start = Page.find( :first, :conditions => {
       :node_id => params[:id],
-      :revision => params[:start]
+      :revision => params[:start_revision]
     })
     
     @end = Page.find( :first, :conditions => {
       :node_id => params[:id],
-      :revision => params[:end]
+      :revision => params[:end_revision]
     })
     
   end
diff --git a/app/views/nodes/edit.html.erb b/app/views/nodes/edit.html.erb
index 6a243a7..d8fb8b4 100644
--- a/app/views/nodes/edit.html.erb
+++ b/app/views/nodes/edit.html.erb
@@ -2,7 +2,7 @@
   <%= link_to 'metadata', '#', :id => 'button', :class => "unselected" %>
   <%= link_to 'Preview', @node %>
   <%= link_to 'Publish', publish_node_path, :method => :put, :confirm => "Publish this draft?" %>
-  <%= link_to 'Revisions', :controller => :revisions, :action => :diff, :id => params[:id] %>  
+  <%= link_to 'Revisions', revision_path(params[:id]) %>  
 <% end %>
 
 <div id="page_editor">
@@ -30,7 +30,7 @@
     </tr>
   </table>
   
-  <table>
+  <table id="content">
     <tr>
       <td class="description">Title</td>
       <td><%= d.text_field :title %></td>
diff --git a/app/views/revisions/diff.html.erb b/app/views/revisions/diff.html.erb
index a8a0276..d8ba6d2 100644
--- a/app/views/revisions/diff.html.erb
+++ b/app/views/revisions/diff.html.erb
@@ -1,8 +1,14 @@
+<% content_for :subnavigation do %>
+  <%= link_to 'Diff', :action => :diff, :id => params[:id] %>
+  <%= link_to 'Edit',     edit_node_path(@node) %>
+  <%= link_to 'Revisions', revision_path(params[:id]) %>
+<% end %>
+
 <h1>Revisions#diff</h1>
 
-<% form_tag url_for(:action => :diff), :method => :get do %>
-  <%= select_tag :start, options_for_select(@node.pages.map{|x| x.revision}, params[:start].to_i) %>
-  <%= select_tag :end, options_for_select(@node.pages.map{|x| x.revision}, params[:end].to_i) %>
+<% form_tag diff_revision_path do %>
+  <%= select_tag :start_revision, options_for_select(@node.pages.map{|x| x.revision}, params[:start_revision].to_i) %>
+  <%= select_tag :end_revision, options_for_select(@node.pages.map{|x| x.revision}, params[:end_revision].to_i) %>
   <%= submit_tag 'Diff' %>
 <% end %>
 
diff --git a/app/views/revisions/show.html.erb b/app/views/revisions/show.html.erb
index 64979fc..3e7ce09 100644
--- a/app/views/revisions/show.html.erb
+++ b/app/views/revisions/show.html.erb
@@ -1,18 +1,32 @@
-<div id="subnavigation">
-  <%= link_to 'Diff revisions', :action => :diff, :id => params[:id] %>
+<% content_for :subnavigation do %>
   <%= link_to 'Edit',     edit_node_path(@node) %>
-</div>
+<% end %>
 
 <h2>Revisions for Node: <%= @node.unique_name %></h2>
 
-<h3>Current title: <%= @node.head.title %></h3>
-<table>
+<% form_tag diff_revision_path do %>
+<table id="revisions">
+  <tr class="header">
+    <th>First</th>
+    <th>Last</th>
+    <th>Rev.</th>
+    <th>Title</th>
+    <th>User</th>
+    <th>Date</th>
+  </tr>
 <% @node.pages.reverse.each do |page| %>
   <tr>
-    <td><%= page.revision %></td>
-    <td><%= page.title %></td>
-    <td><%= page.user.try(:login)  %></td>
-    <td><%= page.updated_at %></td>
+    <td><%= radio_button_tag :start_revision, page.revision %></td>
+    <td><%= radio_button_tag :end_revision, page.revision %></td>
+    <td class="revision"><%= page.revision %></td>
+    <td class="title"><%= page.title %></td>
+    <td class="user"><%= page.user.try(:login)  %></td>
+    <td class="date"><%= page.updated_at %></td>
   </tr>
 <% end %>
-</table>
\ No newline at end of file
+  <tr>
+    <td colspan="6" class="right"><%= submit_tag 'Diff revisions' %></td>
+  </tr>
+</table>
+
+<% end %>
\ No newline at end of file