From 3dc6b02c17bdf531c5ef72fe170f1e6f30de2b54 Mon Sep 17 00:00:00 2001
From: Reinier Balt <lrbalt@gmail.com>
Date: Wed, 23 Jul 2008 23:49:10 +0200
Subject: [PATCH] fixed error where completed container could not be selected
 because of naming inconsistency

---
 .../recurring_todos/_recurring_todo_form.erb   | 18 +++++++++---------
 app/views/todos/_completed.rhtml               |  2 +-
 2 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/app/views/recurring_todos/_recurring_todo_form.erb b/app/views/recurring_todos/_recurring_todo_form.erb
index f6c1fc4e..5fb99272 100644
--- a/app/views/recurring_todos/_recurring_todo_form.erb
+++ b/app/views/recurring_todos/_recurring_todo_form.erb
@@ -12,10 +12,8 @@
       <div id="recurring_todo">
         <label for="recurring_todo_description">Description</label><%= 
         text_field_tag( "recurring_todo[description]", "", "size" => 30, "tabindex" => 1) -%>
-                  
         <label for="recurring_todo_notes">Notes</label><%= 
         text_area_tag( "recurring_todo[notes]", nil, {:cols => 29, :rows => 6, :tabindex => 2}) -%>
-                
         <label for="recurring_todo_project_name">Project</label>
         <input id="recurring_todo_project_name" name="project_name" autocomplete="off" tabindex="3" size="30" type="text" value="" />
         <div class="page_name_auto_complete" id="project_list" style="display:none"></div>
@@ -63,13 +61,15 @@
       </div>        
     </div>
     <div id="recurring_period_id">
-      <label>Recurrence period</label><br/>
-      <%= radio_button_tag('recurring_todo[recurring_period]', 'daily', true)%> Daily<br/>
-      <%= radio_button_tag('recurring_todo[recurring_period]', 'weekly')%> Weekly<br/>
-      <%= radio_button_tag('recurring_todo[recurring_period]', 'monthly')%> Monthly<br/>
-      <%= radio_button_tag('recurring_todo[recurring_period]', 'yearly')%> Yearly<br/>
-      <% apply_behaviour "#recurring_period_id:click", 
-        "TracksForm.hide_all_recurring(); $('recurring_'+TracksForm.get_period()).show();" %>
+      <div id="recurring_period">
+        <label>Recurrence period</label><br/>
+        <%= radio_button_tag('recurring_todo[recurring_period]', 'daily', true)%> Daily<br/>
+        <%= radio_button_tag('recurring_todo[recurring_period]', 'weekly')%> Weekly<br/>
+        <%= radio_button_tag('recurring_todo[recurring_period]', 'monthly')%> Monthly<br/>
+        <%= radio_button_tag('recurring_todo[recurring_period]', 'yearly')%> Yearly<br/>
+        <% apply_behaviour "#recurring_period:click", 
+          "TracksForm.hide_all_recurring(); $('recurring_'+TracksForm.get_period()).show();" %>
+      </div>
       <div id="recurring_timespan">
         <br/>
         <label for="recurring_todo[start_from]">Starts on </label><%= 
diff --git a/app/views/todos/_completed.rhtml b/app/views/todos/_completed.rhtml
index e2a1f962..60b864e0 100644
--- a/app/views/todos/_completed.rhtml
+++ b/app/views/todos/_completed.rhtml
@@ -9,7 +9,7 @@
       <% end %>
       Completed actions <%= suffix %>
   </h2>
-  <div id="completed" class="items toggle_target">
+  <div id="completed_containeritems" class="items toggle_target">
 
     <div id="empty-d" style="display:<%= @done.empty? ? 'block' : 'none' %>">
       <div class="message"><p>Currently there are no completed actions.</p></div>