--- loncom/interface/lonmenu.pm	2003/02/14 19:35:54	1.38
+++ loncom/interface/lonmenu.pm	2003/02/17 19:28:54	1.41
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # Routines to control the menu
 #
-# $Id: lonmenu.pm,v 1.38 2003/02/14 19:35:54 www Exp $
+# $Id: lonmenu.pm,v 1.41 2003/02/17 19:28:54 www Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -41,17 +41,20 @@ use Apache::loncommon;
 use Apache::File;
 use vars qw(@desklines $readdesk);
 
-# ======================================================= Make the menu buttons
+# ============================= This gets called at the top of the body section
 
 sub menubuttons {
     my $forcereg=shift;
     my $target  =shift;
+    my $registration=shift;
     unless ($ENV{'browser.interface'} eq 'textual') { return ''; }
-    return "Menu here<hr />";
+# Textual display only
+    my $output='<a href="/adm/menu">Main Menu</a><br />';
+    if ($registration) { $output.=&innerregister($forcereg,$target); }
+    return $output."<hr />";
 }
 
-# ============================================== Register a URL with the remote
-
+# ====================================== This gets called in the header section
 
 sub registerurl {
     my $forcereg=shift;
@@ -71,36 +74,56 @@ sub registerurl {
 	return $result.
          '<script type="text/javascript">function LONCAPAreg(){;} function LONCAPAstale(){}</script>';
     }
+# Graphical display after login only
     if ($Apache::lonxml::registered && !$forcereg) { return ''; }
+    $result.=&innerregister($forcereg,$target);
+    return $result;
+}
+
+# =========== This gets called in order to register a URL, both with the Remote
+# =========== and in the body of the document
+
+sub innerregister {
+    my $forcereg=shift;
+    my $target = shift;
+    my $result = '';
+
     $Apache::lonxml::registered=1;
+
+    my $textual=($ENV{'browser.interface'} eq 'textual');
     my $reopen=&Apache::lonmenu::reopenmenu();
+
     my $newmail='';
     if (&Apache::lonmsg::newmail()) { 
-       $newmail='swmenu.setstatus("you have","messages");';
+       $newmail=($textual?
+ '<b><a href="/adm/communicate">You have new messages</a></b><br />':
+                          'swmenu.setstatus("you have","messages");');
     }
-    my $timesync='swmenu.syncclock(1000*'.time.');';
+    my $timesync=($textual?'':'swmenu.syncclock(1000*'.time.');');
+# =============================================================================
+# ============================ This is for URLs that actually can be registered
     if (($ENV{'REQUEST_URI'}!~/^\/(res\/)*adm\//) || ($forcereg)) {
+# -- This applies to homework problems for users with grading privileges
         my $hwkadd='';
-        if ($ENV{'request.filename'}=~/\.(problem|exam|quiz|assess|survey|form)$/) {
+        if 
+      ($ENV{'request.filename'}=~/\.(problem|exam|quiz|assess|survey|form)$/) {
 	    if (&Apache::lonnet::allowed('vgr',$ENV{'request.course.id'})) {
-		$hwkadd.=(<<ENDSUBM);
-                     swmenu.switchbutton(7,1,'subm.gif','view sub','missions','gocmd("/adm/grades","submission")',
-                     'View user submissions for this assessment resource');
-ENDSUBM
+		$hwkadd.=&switch('','',7,1,'subm.gif','view sub','missions',
+                       "gocmd('/adm/grades','submission')",
+		       'View user submissions for this assessment resource');
             }
 	    if (&Apache::lonnet::allowed('mgr',$ENV{'request.course.id'})) {
-		$hwkadd.=(<<ENDGRDS);
-                     swmenu.switchbutton(7,2,'pgrd.gif','problem','grades','gocmd("/adm/grades","gradingmenu")',
-                     'Modify user grades for this assessment resource');
-ENDGRDS
+		$hwkadd.=&switch('','',7,2,'pgrd.gif','problem','grades',
+                       "gocmd('/adm/grades','gradingmenu')",
+                       'Modify user grades for this assessment resource');
             }
 	    if (&Apache::lonnet::allowed('opa',$ENV{'request.course.id'})) {
-		$hwkadd.=(<<ENDPARM);
-                     swmenu.switchbutton(7,3,'pparm.gif','problem','parms','gocmd("/adm/parmset","set")',
-                     'Modify deadlines, etc, for this assessment resource');
-ENDPARM
+		$hwkadd.=&switch('','',7,3,'pparm.gif','problem','parms',
+                       "gocmd('/adm/parmset','set')",
+                       'Modify deadlines, etc, for this assessment resource');
             }
 	}
+# -- End Homework
         ###
         ### Determine whether or not to display the 'cstr' button for this
         ### resource
@@ -124,7 +147,7 @@ ENDPARM
                 # Check that we are on the correct machine
                 my $home = &Apache::lonnet::homeserver($caname,$cadom);
                 if ($home eq $Apache::lonnet::perlvar{'lonHostID'}) {
-                    $editbutton=&Apache::lonmenu::switch
+                    $editbutton=&switch
                         ('','',6,1,$top,,$bottom,$action,$desc);
                 }
             }
@@ -148,16 +171,49 @@ ENDPARM
             }        
             # Finally, turn the button on or off
             if ($cfile) {
-                $editbutton=&Apache::lonmenu::switch
+                $editbutton=&switch
                     ('','',6,1,'cstr.gif','edit','resource',
                      "go('".$cfile."');","Edit this resource");
             } elsif ($editbutton eq '') {
-                $editbutton = '    swmenu.clearbut(6,1);';
+                $editbutton=&clear(6,1);
             }
         }
         ###
         ###
-	$result = (<<ENDREGTHIS);
+# Prepare the rest of the buttons
+	my $menuitems=(<<ENDMENUITEMS);
+c&3&1
+s&2&1&back.gif&backward&&gopost('/adm/flip','back:'+currentURL)&Go to the previous resource in the course sequence&1
+s&2&3&forw.gif&forward&&gopost('/adm/flip','forward:'+currentURL)&Go to the next resource in the course sequence&1
+s&6&3&catalog.gif&catalog&info&catalog_info()&Show catalog information
+s&8&1&eval.gif&evaluate&this&gopost('/adm/evaluate',currentURL)&Provide my evaluation of this resource
+s&8&2&fdbk.gif&feedback&discuss&gopost('/adm/feedback',currentURL)&Provide feedback messages or contribute to the course discussion about this resource
+s&8&3&prt.gif&prepare&printout&gopost('/adm/printout',currentURL)&Prepare a printable document
+s&9&1&sbkm.gif&set&bookmark&set_bookmark()&Set a bookmark for this resource&2
+s&9&2&vbkm.gif&view&bookmark&edit_bookmarks()&Use or edit my bookmark collection&2
+s&9&3&anot.gif&anno-&tations&annotate()&Make notes and annotations about this resource&2
+ENDMENUITEMS
+        my $buttons='';
+        foreach (split(/\n/,$menuitems)) {
+	    my ($command,@rest)=split(/\&/,$_);
+            if ($command eq 's') {
+		$buttons.=&switch('','',@rest);
+            } else {
+                $buttons.=&clear(@rest);
+            }
+        }
+        if ($textual) {
+# Registered, textual output
+	    $result =(<<ENDREGTEXT);
+$timesync
+$newmail
+$buttons
+$hwkadd
+$editbutton
+ENDREGTEXT
+# Registered, graphical output
+        } else {
+	    $result = (<<ENDREGTHIS);
      
 <script language="JavaScript">
 // BEGIN LON-CAPA Internal
@@ -167,30 +223,12 @@ ENDPARM
           swmenu.clearTimeout(swmenu.menucltim);
           $timesync
           $newmail
+          $buttons
 	  swmenu.currentURL=window.location.pathname;
           swmenu.reloadURL=window.location.pathname;
           swmenu.currentSymb="$ENV{'request.symb'}";
           swmenu.reloadSymb="$ENV{'request.symb'}";
           swmenu.currentStale=0;
-          swmenu.clearbut(3,1);
-          swmenu.switchbutton
-       (6,3,'catalog.gif','catalog','info','catalog_info()','Show catalog information');
-          swmenu.switchbutton
-       (8,1,'eval.gif','evaluate','this','gopost("/adm/evaluate",currentURL)','Provide my evaluation of this resource');
-          swmenu.switchbutton
-    (8,2,'fdbk.gif','feedback','discuss','gopost("/adm/feedback",currentURL)','Provide feedback messages or contribute to the course discussion about this resource');
-          swmenu.switchbutton
-     (8,3,'prt.gif','prepare','printout','gopost("/adm/printout",currentURL)','Prepare a printable document');
-          swmenu.switchbutton
-       (2,1,'back.gif','backward','','gopost("/adm/flip","back:"+currentURL)','Go to the previous resource in the course sequence');
-          swmenu.switchbutton
-     (2,3,'forw.gif','forward','','gopost("/adm/flip","forward:"+currentURL)','Go to the next resource in the course sequence');
-          swmenu.switchbutton
-                            (9,1,'sbkm.gif','set','bookmark','set_bookmark()','Set a bookmark for this resource');
-          swmenu.switchbutton
-                         (9,2,'vbkm.gif','view','bookmark','edit_bookmarks()','Use or edit my bookmark collection');
-          swmenu.switchbutton
-                               (9,3,'anot.gif','anno-','tations','annotate()','Make notes and annotations about this resource');
           $hwkadd
           $editbutton
     }
@@ -215,9 +253,17 @@ ENDPARM
 // END LON-CAPA Internal
 </script>
 ENDREGTHIS
-
+        }
+# =============================================================================
     } else {
-        $result = (<<ENDDONOTREGTHIS);
+# ========================================== This can or will not be registered
+        if ($textual) {
+# Not registered, textual
+	    $result= (<<ENDDONOTREGTEXT);
+ENDDONOTREGTEXT
+        } else {
+# Not registered, graphical
+           $result = (<<ENDDONOTREGTHIS);
 
 <script language="JavaScript">
 // BEGIN LON-CAPA Internal
@@ -245,6 +291,8 @@ ENDREGTHIS
 // END LON-CAPA Internal
 </script>
 ENDDONOTREGTHIS
+       }
+# =============================================================================
     }
     return $result;
 }
@@ -372,19 +420,23 @@ sub clear {
    } else { return ''; }
 }
 
+# ============================================ Switch a button or create a link
 # Switch acts on the javascript that is executed when a button is clicked.  
 # The javascript is usually similar to "go('/adm/roles')" or "cstrgo(..)".
+
 sub switch {
-    my ($uname,$udom,$row,$col,$img,$top,$bot,$act,$desc)=@_;
+    my ($uname,$udom,$row,$col,$img,$top,$bot,$act,$desc,$nobreak)=@_;
     $act=~s/\$uname/$uname/g;
     $act=~s/\$udom/$udom/g;
     unless ($ENV{'browser.interface'} eq 'textual') {
        return "\n".
  qq(window.status+='.';swmenu.switchbutton($row,$col,"$img","$top","$bot","$act","$desc"););
    } else {
+       if ($nobreak==2) { return ''; }
        my $text=$top.' '.$bot;
        $text=~s/\- //;
-       return '<br /><a href="javascript:'.$act.';">'.$text.'</a> '.$desc;
+       return ($nobreak?'':'<br />').
+        '<a href="javascript:'.$act.';">'.$text.'</a> '.($nobreak?'':$desc);
    }
 }