--- loncom/interface/lonhtmlcommon.pm	2009/04/25 16:53:13	1.208
+++ loncom/interface/lonhtmlcommon.pm	2009/05/16 18:06:41	1.214
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # a pile of common html routines
 #
-# $Id: lonhtmlcommon.pm,v 1.208 2009/04/25 16:53:13 www Exp $
+# $Id: lonhtmlcommon.pm,v 1.214 2009/05/16 18:06:41 tempelho Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -74,10 +74,18 @@ Successful completion of an operation me
 =cut
 
 sub confirm_success {
-   my ($message)=@_;
-   return '<span class="LC_success_confirm">'.
-          $message.
-          ' <img src="/adm/lonIcons/navmap.correct.gif" /></span>';
+   my ($message,$failure)=@_;
+   if ($failure) {
+      return '<span class="LC_error">'."\n"
+            .'<img src="/adm/lonIcons/navmap.wrong.gif" /> '."\n"
+            .$message."\n"
+            .'</span>'."\n";
+   } else {
+      return '<span class="LC_success">'."\n"
+            .'<img src="/adm/lonIcons/navmap.correct.gif" /> '."\n"
+            .$message."\n"
+            .'</span>'."\n";
+   }
 }
 
 ##############################################
@@ -972,7 +980,7 @@ sub Create_PrgWin {
          popwin=open(\'\',\'popwin\',\'width=400,height=100\');".
         "popwin.document.writeln(\'".$start_page.
               "<h4>".&mt("$heading")."<\/h4>".
-              "<form action= \"\" name=\"popremain\" method=\"post\">".
+              "<form action=\"\" name=\"popremain\" method=\"post\">".
               '<input type="text" size="'.$width.'" name="remaining" value="'.
 	      &mt('Starting').'" /><\\/form>'.$end_page.
               "\');".
@@ -1325,7 +1333,7 @@ returns: nothing
         my ($component,$component_help,$menulink,$helplink,$css_class,$no_mt, $no_realBreadcrumb) = @_;
         #
 	$css_class ||= 'LC_breadcrumbs';
-        my $Str1 = '<ol id="LC_MenuBreadcrumbs">';
+        my $Str1 = '<div id="LC_head_subbox"><ol id="LC_MenuBreadcrumbs">';
 	
 		if($no_realBreadcrumb){
 			$Str1 = '<ul class="LC_CourseBreadcrumbs">';
@@ -1417,7 +1425,7 @@ returns: nothing
 		if($no_realBreadcrumb){
         	$Str1 .= $links.'</ul>';
 		} else {
-        	$Str1 .= $links.'</ol>';		
+        	$Str1 .= $links.'</ol></div>';		
 		}
         #
         if (defined($component)) {
@@ -1540,9 +1548,9 @@ END
 }
 
 sub row_title {
+    my ($title,$css_title_class,$css_value_class) = @_;
     $row_count[0]++;
     my $css_class = ($row_count[0] % 2)?'LC_odd_row':'LC_even_row';
-    my ($title,$css_title_class,$css_value_class) = @_;
     $css_title_class ||= 'LC_pick_box_title';
     $css_title_class = 'class="'.$css_title_class.'"';
 
@@ -1801,7 +1809,7 @@ sub topic_bar {
     return '
 <div class="LC_topic_bar">
     <img alt="'.&mt('Step [_1]',$imgnum).
-              '"src="/res/adm/pages/bl_step'.$imgnum.'.gif" />&nbsp;
+              ' "src="/res/adm/pages/bl_step'.$imgnum.'.gif" />&nbsp;
     <span>'.$title.'</span>
 </div>
 ';