--- loncom/xml/londefdef.pm 2010/11/28 02:46:23 1.427 +++ loncom/xml/londefdef.pm 2011/04/05 10:02:57 1.428 @@ -1,7 +1,7 @@ # The LearningOnline Network with CAPA # Tags Default Definition Module # -# $Id: londefdef.pm,v 1.427 2010/11/28 02:46:23 raeburn Exp $ +# $Id: londefdef.pm,v 1.428 2011/04/05 10:02:57 foxr Exp $ # # # Copyright Michigan State University Board of Trustees @@ -2063,15 +2063,22 @@ sub start_table { if ((defined $border) && ($border > 0)) { # &Apache::lonnet::logthis("Turning on table borders: $border"); $table->table_border(1); - if ($cell_border ne 'none') { - $table->cell_border(1); # html specs that border turns on both...unless rules='none'. + if (!defined $cell_border) { + $table->cell_border(1); # Default for rules is all if rules not defined. } } # Only all or nothing for cell borders for now: - if ((defined $cell_border) && ($cell_border ne 'none')) { - # &Apache::lonnet::logthis("Turning on cell borders: $cell_border"); - $table->cell_border(1); + if ((defined $cell_border)) { + if ($cell_border eq 'all') { + $table->cell_border(1); + } elsif ($cell_border eq 'rows') { + $table->cell_border(2); + } elsif ($cell_border eq 'cols') { + $table->cell_border(3); + } else { + $table->cell_border(0); + } } if (defined $theme) { $table->theme($theme);