Update of /var/cvs/html/mmbase/validation
In directory james.mmbase.org:/tmp/cvs-serv32445

Modified Files:
        Widgets.js 
Log Message:
bugfix


See also: http://cvs.mmbase.org/viewcvs/html/mmbase/validation


Index: Widgets.js
===================================================================
RCS file: /var/cvs/html/mmbase/validation/Widgets.js,v
retrieving revision 1.7
retrieving revision 1.8
diff -u -b -r1.7 -r1.8
--- Widgets.js  10 Dec 2008 11:53:43 -0000      1.7
+++ Widgets.js  30 Dec 2008 14:14:00 -0000      1.8
@@ -9,7 +9,7 @@
  *  -  Widgets.instance.boxes(selector):  Makes select into a list of 
checkboxes (multiple) or radioboxes (single)
  *  -  Widgets.instance.twoMultiples(selector):  Splits up multiple selection 
into 2 boxes, the left one containing the selected values, the right one the 
optiosn which are not selected.
  *
- * @version $Id: Widgets.js,v 1.7 2008/12/10 11:53:43 michiel Exp $   BETA
+ * @version $Id: Widgets.js,v 1.8 2008/12/30 14:14:00 michiel Exp $   BETA
  * @author Michiel Meeuwissen
 
  */
@@ -139,6 +139,8 @@
 Widgets.prototype.multipleBoxes = function(select) {
     var t = $(select);
     var text = $("<div class='mm_boxes' />");
+    text.attr("id", t.attr("id"));
+    text.addClass(t.attr("class"));
     var hidden = $("<input type='hidden' />");
     text.append(hidden);
     hidden.attr("name", t.attr("name"));
@@ -234,24 +236,26 @@
             var left  = $("<select multiple='multiple' />");
             left.attr("name", t.attr("name"));
             left.attr("id", t.attr("id"));
+            var right = $("<select multiple='multiple' />");
+
             t.parents("form").submit(function() {
                 for (var i = 0; i < left[0].options.length; i++) {
                     left[0].options[i].selected = true;
                 }
             });
-            var right = $("<select multiple='multiple' />");
+            var opts = [];
             for (var i = 0; i < select.options.length; i++) {
                 var option = select.options[i];
+                opts[i] = option;
                 option.originalPosition = option.index;
             }
-            for (var i = 0; i < select.options.length; i++) {
-                var option = select.options[i];
-
+            for (var i = 0; i < opts.length; i++) {
+                var option = opts[i];
                 if (option.value == null || option.value == '') {
                 } else if (option.selected) {
                     left.append(option);
                 } else {
-                    right.append(option)[0];
+                    right.append(option);
                 }
             }
             var nobr = $("<nobr />");
_______________________________________________
Cvs mailing list
[email protected]
http://lists.mmbase.org/mailman/listinfo/cvs

Reply via email to