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

[www]

www		Sat Sep  7 14:48:27 2002 EDT

  Modified files:              
    /loncom/interface	lonparmset.pm 
  Log:
  User remarked on "goofy" behavior of "set" checkbox. Turns out, he was
  filling in a value, and then clicked on the "set" checkbox. As a result,
  the checkbox was "off" ... why? onchange is triggered only after the field
  lost focus - and it did so, because the user clicked on the "set" box.
  
  OnFocus might be a little wasteful, but less goofy.
  
  
Index: loncom/interface/lonparmset.pm
diff -u loncom/interface/lonparmset.pm:1.68 loncom/interface/lonparmset.pm:1.69
--- loncom/interface/lonparmset.pm:1.68	Sat Sep  7 14:25:42 2002
+++ loncom/interface/lonparmset.pm	Sat Sep  7 14:48:26 2002
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # Handler to set parameters for assessments
 #
-# $Id: lonparmset.pm,v 1.68 2002/09/07 18:25:42 www Exp $
+# $Id: lonparmset.pm,v 1.69 2002/09/07 18:48:26 www Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -1429,7 +1429,7 @@
 	}
 	foreach (sort keys %descriptions) {
             # onchange is javascript to automatically check the 'Set' button.
-            my $onchange = 'onchange="javascript:window.document.forms'.
+            my $onchange = 'onFocus="javascript:window.document.forms'.
                 '[\'envform\'].elements[\''.$_.'_setparmval\']'.
                 '.checked=true;"';
 	    $output.='<tr><td>'.$descriptions{$_}.'</td>'.
@@ -1438,7 +1438,7 @@
                 '<td><input type=checkbox name="'.$_.'_setparmval"></td>'.
                 '</tr>'."\n";
 	}
-        my $onchange = 'onchange="javascript:window.document.forms'.
+        my $onchange = 'onFocus="javascript:window.document.forms'.
             '[\'envform\'].elements[\'newp_setparmval\']'.
             '.checked=true;"';
 	$output.='<tr><td><i>Create New Environment Variable</i><br />'.