--- loncom/xml/londefdef.pm	2004/08/25 19:08:32	1.237
+++ loncom/xml/londefdef.pm	2004/08/25 19:57:27	1.238
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # Tags Default Definition Module 
 #
-# $Id: londefdef.pm,v 1.237 2004/08/25 19:08:32 sakharuk Exp $
+# $Id: londefdef.pm,v 1.238 2004/08/25 19:57:27 albertel Exp $
 # 
 #
 # Copyright Michigan State University Board of Trustees
@@ -66,8 +66,7 @@ sub initialize_londefdef {
     @Apache::londefdef::description=();
     $Apache::londefdef::DD_redirection=0;
     $Apache::londefdef::DT_redirection=0;
-    @Apache::londefdef::alphabit=('A'..'Z');
-    $Apache::londefdef::alphabit_index=0;
+    $Apache::londefdef::list_index=0;
 }
 
 #======================= TAG SUBROUTINES =====================
@@ -1489,23 +1488,26 @@ sub start_li {
     } elsif ($target eq 'tex') {
 	my $type=&Apache::lonxml::get_param('type',$parstack,$safeeval,undef,0);
 	my $value=&Apache::lonxml::get_param('value',$parstack,$safeeval,undef,0);
-	if ($type=~/circle/) {
-	    $currentstring .= ' \item[o] ';
+	#FIXME need to support types i and I 
+	if ($type=~/disc/) {
+	    $currentstring .= ' \item[$\bullet$] ';
+	} elsif ($type=~/circle/) {
+	    $currentstring .= ' \item[$\circ$] ';
 	} elsif ($type=~/square/) {
-	    $currentstring .= ' \item[$\Box$] ';
+	    $currentstring .= ' \item[$\diamond$] ';
+	} elsif ($type eq '1') {
+	    $currentstring .= ' \item['.($Apache::londefdef::list_index+1).'.]';
 	} elsif ($type eq 'A') {
-	    $currentstring .= ' \item['.$Apache::londefdef::alphabit[$Apache::londefdef::alphabit_index].']';
-	    $Apache::londefdef::alphabit_index++;
+	    $currentstring .= ' \item['.('A'..'Z')[$Apache::londefdef::list_index].'.]';
 	} elsif ($type eq 'a') {
-	    my $lower_case=lc $Apache::londefdef::alphabit[$Apache::londefdef::alphabit_index];
-	    $currentstring .= ' \item['.$lower_case.']';
-	    $Apache::londefdef::alphabit_index++;
+	    $currentstring .= ' \item['.('a'..'z')[$Apache::londefdef::list_index].'.]';
 	} elsif ($value ne '') {
 	    $currentstring .= ' \item['.$value.'] ';
 	} else {
 	    $currentstring .= ' \item ';
 	}  
-    } 
+	$Apache::londefdef::list_index++;
+    }
     return $currentstring;
 }
 
@@ -1552,6 +1554,7 @@ sub start_ul {
 	$currentstring = $token->[4];     
     } elsif ($target eq 'tex') {
 	my $TeXtype=&Apache::lonxml::get_param('type',$parstack,$safeeval,undef,0);
+	$Apache::londefdef::list_index=0;
 	if ($TeXtype eq 'disc') {
 	    $currentstring .= '\renewcommand{\labelitemi}{$\bullet$}'.
                               '\renewcommand{\labelitemii}{$\bullet$}'. 
@@ -1640,7 +1643,7 @@ sub start_ol {
     if ($target eq 'web') {
 	$currentstring = $token->[4];     
     } elsif ($target eq 'tex') {
-	$Apache::londefdef::alpabit_index=0;
+	$Apache::londefdef::list_index=0;
 	my $type=&Apache::lonxml::get_param('type',$parstack,$safeeval,undef,0);
 	if ($type eq '1') {
 	    $currentstring .= '\renewcommand{\labelenumi}{\arabic{enumi}.}'.