--- loncom/interface/lonnavmaps.pm	2012/05/28 10:31:17	1.485
+++ loncom/interface/lonnavmaps.pm	2012/06/08 21:24:16	1.486
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # Navigate Maps Handler
 #
-# $Id: lonnavmaps.pm,v 1.485 2012/05/28 10:31:17 foxr Exp $
+# $Id: lonnavmaps.pm,v 1.486 2012/06/08 21:24:16 raeburn Exp $
 
 #
 # Copyright Michigan State University Board of Trustees
@@ -484,6 +484,7 @@ use POSIX qw (floor strftime);
 use Time::HiRes qw( gettimeofday tv_interval );
 use LONCAPA;
 use DateTime();
+use HTML::Entities;
 
 # For debugging
 
@@ -1481,9 +1482,10 @@ sub render {
 		      'document.clearbubbles.submit()',
 		      'Mark all posts read');
 	my $time=time;
+        my $querystr = &HTML::Entities::encode($ENV{'QUERY_STRING'},'<>&"');
 	$result .= (<<END);
     <form name="clearbubbles" method="post" action="/adm/feedback">
-	<input type="hidden" name="navurl" value="$ENV{'QUERY_STRING'}" />
+	<input type="hidden" name="navurl" value="$querystr" />
 	<input type="hidden" name="navtime" value="$time" />
 END
         if ($env{'form.register'}) {
@@ -1825,6 +1827,8 @@ END
 	    }
 	}
     }
+
+    $result.=&Apache::loncommon::end_data_table();
     
     # Print out the part that jumps to #curloc if it exists
     # delay needed because the browser is processing the jump before
@@ -1841,8 +1845,6 @@ if (location.href.indexOf('#curloc')==-1
 ");
     }
 
-    $result.=&Apache::loncommon::end_data_table();
-
     if ($r) {
         $r->print($result);
         $result = "";
@@ -1873,7 +1875,6 @@ sub show_linkitems_toolbar {
                           '<td class="LC_middle">'.&mt('Tools:').'</td>';
             }
             $result .= '<td align="left">'."\n".
-                       '<span class="LC_nobreak">'."\n".
                        '<ul id="LC_toolbar">';
             my @linkorder = ('firsthomework','everything','uncompleted',
                              'changefolder','clearbubbles','edittoplevel');
@@ -1901,7 +1902,7 @@ sub show_linkitems_toolbar {
                 }
             }
             $result .= '</ul>'.
-                       '</span></td>';
+                       '</td>';
             if (($numlinks==1) && (exists($args->{'linkitems'}{'edittoplevel'}))) {
                 $result .= '<td><a href="'.$args->{'linkitems'}{'edittoplevel'}{'cmd'}.'">'.
                            &mt('Content Editor').'</a></td>';