--- loncom/interface/lonmenu.pm 2013/08/13 11:26:38 1.419 +++ loncom/interface/lonmenu.pm 2014/12/20 15:35:40 1.430 @@ -1,7 +1,7 @@ # The LearningOnline Network with CAPA # Routines to control the menu # -# $Id: lonmenu.pm,v 1.419 2013/08/13 11:26:38 bisitz Exp $ +# $Id: lonmenu.pm,v 1.430 2014/12/20 15:35:40 raeburn Exp $ # # Copyright Michigan State University Board of Trustees # @@ -283,8 +283,14 @@ sub primary_menu { $menu{$position} .= prep_menuitem($menuitem); } } - return ("
", - " "); + my @output = ('',''); + if ($menu{'left'} ne '') { + $output[0] = " "; + } + if ($menu{'right'} ne '') { + $output[1] = " "; + } + return @output; } #returns hashref {user=>'',dom=>''} containing: @@ -313,6 +319,7 @@ sub getauthor{ } sub secondary_menu { + my ($httphost) = @_; my $menu; my $crstype = &Apache::loncommon::course_type(); @@ -417,7 +424,8 @@ sub secondary_menu { ($roleswitcher_js,$roleswitcher_form,my $switcher) = &roles_selector( $env{'course.' . $env{'request.course.id'} . '.domain'}, - $env{'course.' . $env{'request.course.id'} . '.num'} + $env{'course.' . $env{'request.course.id'} . '.num'}, + $httphost ); $menu .= $switcher; } else { @@ -506,14 +514,17 @@ sub create_submenu { if ($count == $numsub) { $borderbot = 'border-bottom:1px solid black;'; } + unless (($href eq '') || ($href =~ /^\#/)) { + $target = ' target="_top"'; + } $menu .= '