--- loncom/homework/edit.pm	2001/05/15 20:48:43	1.7
+++ loncom/homework/edit.pm	2001/06/16 18:35:27	1.11
@@ -6,22 +6,28 @@ package Apache::edit;
 use strict;
 use Apache::lonnet;
 
+# Global Vars
+# default list of colors to use in editing
+@Apache::edit::colorlist=('#ffffff','#ff0000','#00ff00','#0000ff','#0ff000','#000ff0','#f0000f');
+# depth of nesting of edit
+$Apache::edit::colordepth=0;
+
+sub initialize_edit {
+  $Apache::edit::colordepth=0;
+}
+
 sub tag_start {
-  my ($target,$token) = @_;
+  my ($target,$token,$description) = @_;
   my $result='';
   if ($target eq "edit") {
     my $tag=$token->[1];
-#    my $color = sprintf("#%06lx",(hex("ffffff")) >> scalar(split(/_/,$Apache::lonxml::curdepth)));
-    my $color = sprintf("#%02lxffff",33* scalar(split(/_/,$Apache::lonxml::curdepth)));
-    $result.="<table bgcolor=\"$color\" width=\"100%\" border=\"2\"><tr><td>&lt;$tag&gt;</td>
-<td>Delete:
-<select name=\"delete_$Apache::lonxml::curdepth\">
-<option>Nothing</option>
-<option>Tag</option>
-<option>Subtags</option>
-</select></td>
+    if (!$description) { $description="&lt;$tag&gt;"; }
+    $result.= &start_table($token)."<tr><td>$description</td>
+<td>Delete:".
+  &deletelist($target,$token)
+  ."</td>
 <td>".
-  &insertlist($token,$target).
+  &insertlist($target,$token).
     "</td>
 </tr><tr><td colspan=\"3\">\n";
   }
@@ -29,12 +35,52 @@ sub tag_start {
 }
 
 sub tag_end {
-  my ($target,$token) = @_;
+  my ($target,$token,$description) = @_;
   my $result='';
   if ($target eq 'edit') {
     my $tag=$token->[1];
-    $result.="</td></tr><tr><td>&lt;/$tag&gt;</td></tr></table>\n";
+    if (!defined($description)) {
+      $result.="</td></tr><tr><td>&lt;/$tag&gt;";
+    } else {
+      if ($description ne '') { $result.="</td></tr><tr><td>$description"; }
+    }
+    $result.="</td><td>&nbsp;</td><td>&nbsp;</td></tr>".&end_table()."\n";
+  }
+  return $result;
+}
+
+sub start_table {
+  my ($token)=@_;
+  my $tag = $token->[1];
+  my $tagnum;
+  foreach my $namespace (reverse @Apache::lonxml::namespace) {
+    my $testtag=$Apache::lonxml::namespace['-1'].'::'.$tag;
+    $tagnum=$Apache::lonxml::insertlist{"$testtag.num"};
+    if (defined($tagnum)) { last; }
+  }
+  if (!defined ($tagnum)) { $tagnum=$Apache::lonxml::insertlist{"$tag.num"}; }
+  my $color = $Apache::lonxml::insertlist{"$tagnum.color"};
+  if (!defined($color)) {
+    $color = $Apache::edit::colorlist[$Apache::edit::colordepth];
   }
+  $Apache::edit::colordepth++;
+  my $result="<table bgcolor=\"$color\" width=\"100%\" border=\"10\">";
+  return $result;
+}
+
+sub end_table {
+  $Apache::edit::colordepth--;
+  my $result="</table>";
+  return $result;
+}
+
+sub deletelist {
+  my ($target,$token) = @_;
+  my $result = "<select name=\"delete_$Apache::lonxml::curdepth\">
+<option>Nothing</option>
+<option>Tag</option>
+<option>Subtags</option>
+</select>";
   return $result;
 }
 
@@ -44,12 +90,10 @@ sub get_insert_list {
   my @tagnums= ();
   #&Apache::lonxml::debug("keys ".join("\n",sort(keys(%Apache::lonxml::insertlist))));
   if ($Apache::lonxml::insertlist{"$token->[1].which"}) {
-    &Apache::lonxml::debug("Adding1 $token->[1].which");
     push (@tagnums, @{ $Apache::lonxml::insertlist{"$token->[1].which"} });
   }
   foreach my $namespace (@Apache::lonxml::namespace) {
     if ($Apache::lonxml::insertlist{"$namespace".'::'."$token->[1].which"}) {
-      &Apache::lonxml::debug("Adding2 $namespace".'::'."$token->[1].which");
       push (@tagnums, @{ $Apache::lonxml::insertlist{"$namespace".'::'."$token->[1].which"} });
     }
   }
@@ -63,7 +107,7 @@ sub get_insert_list {
 }
 
 sub insertlist {
-  my ($token,$target) = @_;
+  my ($target,$token) = @_;
   my $result;
   if ($target eq 'edit') {
     my $optionlist= &get_insert_list($token);
@@ -72,6 +116,8 @@ sub insertlist {
 <select name=\"insert_$Apache::lonxml::curdepth\">
 $optionlist
 </select>"
+    } else {
+      $result="&nbsp;";
     }
   }
   return $result;
@@ -116,9 +162,9 @@ sub editfield {
   if ($maxlength < $minwidth) { $maxlength = $minwidth; }
   if ( $count < $minheight) { $count = $minheight; }
   if ($description) {
-    $description="<br />".$description;
+    $description="<br />".$description."<br />";
   }
-  return "$description<br />\n&nbsp;&nbsp;&nbsp;<textarea rows=\"$count\" cols=\"$maxlength\" name=homework_edit_".$Apache::lonxml::curdepth.">$data</textarea>\n";
+  return "$description\n&nbsp;&nbsp;&nbsp;<textarea rows=\"$count\" cols=\"$maxlength\" name=homework_edit_".$Apache::lonxml::curdepth.">$data</textarea>\n";
 #  return "<br />\n&lt;$tag&gt;<br />\n&nbsp;&nbsp;&nbsp;<textarea rows=\"$count\" cols=\"$maxlength\" name=homework_edit_".$Apache::lonxml::curdepth.">$data</textarea><br />\n&lt;/$tag&gt;<br />\n";
 }