--- loncom/interface/loncommon.pm	2009/10/24 03:24:13	1.902
+++ loncom/interface/loncommon.pm	2009/10/28 08:47:56	1.904
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # a pile of common routines
 #
-# $Id: loncommon.pm,v 1.902 2009/10/24 03:24:13 raeburn Exp $
+# $Id: loncommon.pm,v 1.904 2009/10/28 08:47:56 droeschl Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -4444,48 +4444,44 @@ sub bodytag {
 
     if ($env{'environment.remote'} eq 'off') {
         # No Remote
-        if ($env{'request.state'} eq 'construct') {
-            $forcereg=1;
-        }
+        if ($no_nav_bar) { return $bodytag; } 
 
-    #    if ($env{'request.state'} eq 'construct') {
-    #        $titleinfo = &CSTR_pageheader(); #FIXME: Will be removed once all scripts have their own calls
-    #    }
+        if ($env{'request.state'} eq 'construct') { $forcereg=1; }
 
-        my $titletable = '<table id="LC_title_bar">'
-                            ."<tr><td> $titleinfo $dc_info</td>"
-                            .'</tr></table>';
+        #    if ($env{'request.state'} eq 'construct') {
+        #        $titleinfo = &CSTR_pageheader(); #FIXME: Will be removed once all scripts have their own calls
+        #    }
 
-        if ($no_nav_bar) {
-            $bodytag .= $titletable;
-        } else {
-            $bodytag .= qq|<div id="LC_nav_bar">$name $role<br />
-                <em>$realm</em> $dc_info</div>| unless $env{'form.inhibitmenu'};
+        $bodytag .= qq|<div id="LC_nav_bar">$name $role<br />
+            <em>$realm</em> $dc_info</div>| unless $env{'form.inhibitmenu'};
+
+        if (   $env{'form.inhibitmenu'} eq 'yes' 
+            || $ENV{'REQUEST_URI'} eq '/adm/logout'
+            || $env{'request.noversionuri'} =~ m{^/res/adm/pages/}) {
+
+            return $bodytag;
+        }
+
+        $bodytag .= Apache::lonhtmlcommon::scripttag(
+            Apache::lonmenu::utilityfunctions(), 'start');
 
-#SD $titletable is obsolete
-#SD            if ($env{'request.state'} eq 'construct') {
-#SD                $bodytag .= &Apache::lonmenu::menubuttons($forcereg,$titletable);
-#SD            } else {
-#SD                $bodytag .= &Apache::lonmenu::menubuttons($forcereg).$titletable;
-#SD            }
-               if (   $env{'form.inhibitmenu'} eq 'yes' 
-                   || $ENV{'REQUEST_URI'} eq '/adm/logout'
-                   || $env{'request.noversionuri'} =~ m{^/res/adm/pages/}) {
-                   
-                   return $bodytag;
-               }
-
-               $bodytag .= Apache::lonhtmlcommon::scripttag(
-                                Apache::lonmenu::utilityfunctions(),
-                                'start');
-               $bodytag .= Apache::lonmenu::primary_menu();
-               $bodytag .= Apache::lonmenu::secondary_menu();
-               #SD remove next line
-               #$bodytag .= Apache::lonmenu::menubuttons($forcereg);
-               $bodytag .= Apache::lonmenu::serverform();
-               $bodytag .= Apache::lonhtmlcommon::scripttag('', 'end'); 
-               $bodytag .= Apache::lonmenu::innerregister($forcereg) if $forcereg;
+        $bodytag .= Apache::lonmenu::primary_menu();
+
+        #don't show menus for public users
+        if($env{'user.name'} ne 'public' && $env{'user.domain'} ne 'public'){
+            $bodytag .= Apache::lonmenu::secondary_menu();
+            $bodytag .= Apache::lonmenu::serverform();
+            $bodytag .= Apache::lonhtmlcommon::scripttag('', 'end'); 
+            $bodytag .= Apache::lonmenu::innerregister($forcereg) if $forcereg;
+        }else{
+            # this is to seperate menu from content when there's no secondary
+            # menu. Especially needed for public accessible ressources.
+            $bodytag .= '<hr style="clear:both" />';
+            $bodytag .= Apache::lonhtmlcommon::scripttag('', 'end'); 
         }
+
+        #SD testing
+        #$bodytag .= Apache::lonmenu::menubuttons($forcereg);
         return $bodytag;
     }
 
@@ -4884,7 +4880,10 @@ table#LC_mainmenu td.LC_mainmenu_column
  clear:both;
  background: $sidebg;
  border-bottom: 1px solid $lg_border_color;
- line-height: 32px; 
+ line-height: 2.5em; 
+ /* SD working here
+ height: 2.5em;
+ overflow: hidden; */
  margin: 0;
  padding: 0;
 }
@@ -5808,7 +5807,7 @@ ul#LC_secondary_menu a:hover,
 .LC_FormSectionClearButton input:hover
 ul.LC_TabContent   li:hover a {
 	color:#BF2317;
-        text-decoration:none;
+    text-decoration:none;
 }
 
 h1 {
@@ -6038,6 +6037,8 @@ ol#LC_PathBreadcrumbs {
 	padding-left: 10px;
 	margin: 0;
 	list-style-position: inside;
+    /* SD working here
+    white-space: nowrap; */
 }
 
 ol#LC_MenuBreadcrumbs li, 
@@ -6045,6 +6046,8 @@ ol#LC_PathBreadcrumbs li,
 ul.LC_CourseBreadcrumbs li {
     display: inline;
     white-space: nowrap;
+    /* SD working here
+    white-space: normal; */
 }
 
 ol#LC_MenuBreadcrumbs li a,