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

schualex schualex@source.lon-capa.org
Tue, 09 Jun 2009 23:27:04 -0000


schualex		Tue Jun  9 23:27:04 2009 EDT

  Modified files:              
    /loncom/interface	lonparmset.pm 
  Log:
  Bug: The javascript method for the overview-list works wrong with the shortcut-links and the checkboxes. When using the shortcuts mixed with the checkboxes the overview-lables are dispersing with the checkbox-selections.
  
  Bugfix: The checkbox javascript-function has called the checkCheckbox-function, that was written for the overview-list, to hide and show the list-items but it works only well with the checkboxes. When using the shortcuts the overview and the checkboxes for the parm-list will disperse.
  I modified the checkthis()-function so that it only enables the overview-item without minding the current status of the checkboxes.
  
  
  
Index: loncom/interface/lonparmset.pm
diff -u loncom/interface/lonparmset.pm:1.457 loncom/interface/lonparmset.pm:1.458
--- loncom/interface/lonparmset.pm:1.457	Tue Jun  9 22:50:28 2009
+++ loncom/interface/lonparmset.pm	Tue Jun  9 23:27:04 2009
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # Handler to set parameters for assessments
 #
-# $Id: lonparmset.pm,v 1.457 2009/06/09 22:50:28 schualex Exp $
+# $Id: lonparmset.pm,v 1.458 2009/06/09 23:27:04 schualex Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -1269,7 +1269,9 @@
     }
 
     function checkthis(thisvalue, checkName) {
-        checkboxChecked(thisvalue);
+
+        document.getElementById(thisvalue.concat("_li")).style.display = "";        
+
 	for (i=0; i<document.forms.parmform.elements.length; i++) {
             ele = document.forms.parmform.elements[i];
             if (ele.name == checkName) {