[LON-CAPA-cvs] cvs: loncom /interface lonprintout.pm

[raeburn]

raeburn		Sun Feb 17 03:42:24 2019 EDT

  Modified files:              
    /loncom/interface	lonprintout.pm 
  Log:
  - Unwanted comma in <dropdown> tag causes invalid select element in SLES15.
  - Balance braces when font size is empty string.
  
  
Index: loncom/interface/lonprintout.pm
diff -u loncom/interface/lonprintout.pm:1.668 loncom/interface/lonprintout.pm:1.669
--- loncom/interface/lonprintout.pm:1.668	Thu Jan 31 16:08:07 2019
+++ loncom/interface/lonprintout.pm	Sun Feb 17 03:42:24 2019
@@ -1,7 +1,7 @@
 # The LearningOnline Network
 # Printout
 #
-# $Id: lonprintout.pm,v 1.668 2019/01/31 16:08:07 raeburn Exp $
+# $Id: lonprintout.pm,v 1.669 2019/02/17 03:42:24 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -847,8 +847,9 @@
     if ($font_size ne '') {
 
 	$text =~ s/\\begin\{document}/\\begin{document}{\\$font_size/;
+        $text =~ s/\\end\{document}/}\\end{document}/;
+
     }
-    $text =~ s/\\end\{document}/}\\end{document}/;
     return $text;
 
 
@@ -4826,7 +4827,7 @@
             my $xmlfrag = << "FONT_SELECTION";
 
           
-            <dropdown variable='fontsize' multichoice='0', allowempty='0'>
+            <dropdown variable='fontsize' multichoice='0' allowempty='0'>
             <defaultvalue>
 		  return 'normalsize';
             </defaultvalue>