Apply Jacqui's patch, so that "The "Toggle Notes" link in the navigation at the top should toggle the display of notes on items in a page, not the entire page.". Fixes #563.

Thanks, Jacqui!


git-svn-id: http://www.rousette.org.uk/svn/tracks-repos/trunk@596 a4c988fc-2ded-0310-b66e-134b36920a42

tracks/app/views/layouts/standard.rhtml

@@ -34,8 +34,8 @@ window.onload=function(){
   </h1>
 </div>
 <div id="minilinks">
-	<%= link_to_function("Toggle notes", nil, {:accesskey => "S", :title => "Toggle all notes"}) do |page|
-		  page.select('.notes').each { |e| e.toggle }
+	<%= link_to_function("Toggle notes", nil, {:accesskey => "S", :title => "Toggle all notes", :id => "toggle-notes-nav"}) do |page|
+		  page.select('body .notes').each { |e| e.toggle }
 		end
 	-%>&nbsp;|&nbsp;
     <%= link_to "Logout (#{current_user.display_name}) »", logout_path %>

tracks/test/selenium/notes/toggle_notes.rsel

@@ -0,0 +1,5 @@
+setup :fixtures => :all
+login :as => 'admin'
+open "/notes"
+click_and_wait "css=#toggle-notes-nav"
+assert_element_present "body#notes"

tracks/test/selenium/project_detail/toggle_notes.rsel

@@ -0,0 +1,5 @@
+setup :fixtures => :all
+login :as => 'admin'
+open "/projects/1"
+click_and_wait "css=#toggle-notes-nav"
+assert_element_present "notes_todo_6"