Changeset 25784 for lang/javascript/userchrome

Timestamp:

12/03/08 19:59:53 (5 weeks ago)

Author:

whym

Message:

fix

Files:

• lang/javascript/userchrome/ToggleAllInSelection.uc.js (modified) (2 diffs)

lang/javascript/userchrome/ToggleAllInSelection.uc.js

@@ -6 +6 @@
 // @author       Whym
 // @version      1.0.20081203
-// @homepage     http://www.xuldev.org/misc/ucjs.php
+// @homepage     http://coderepos.org/share/browser/lang/javascript/userchrome
+// @compatibility  Firefox 3.0
+// @include        chrome://browser/content/browser.xul
+// @note           
 // ==/UserScript==
 
@@ -14 +17 @@
         var menuitem = document.getElementById(eltID);
 
-
         if (!menuitem) {
             var _toggleall = function(event) {
-                var sel = content.getSelection();
+                var sel = window.content.getSelection();
                 if (!sel) { return; }
                 for ( var i=0; i < sel.rangeCount; i++ ) {
                     var range = sel.getRangeAt(i);
-                    var iter = document.createTreeWalker(range.commonAncestorContainer, NodeFilter.SHOW_ELEMENT, null, false);
-                    while(iter.nextNode() && iter.currentNode != range.startContainer){ // skip until startnode of selection
-                    }
-                    iter.previousNode();
-                    while(iter.nextNode()){
-                        var n = iter.currentNode;
-                        if(n.tagName == "INPUT" && n.type.toLowerCase() =="checkbox") {
-                            n.checked = !n.checked; // toggle
+                    var set = range.commonAncestorContainer.getElementsByTagName("INPUT");
+                    for ( var j=0; j < set.length; j++ ) {
+                        var n = set[j];
+                        if ( sel.containsNode(n,true) ) {
+                            if(n.tagName == "INPUT" && n.type.toLowerCase() == "checkbox") {
+                                n.checked = !n.checked;
+                            }
                         }
-                        if( n == range.endContainer ){break;}
+                        if ( n == range.endContainer ) {
+                            break;
+                        }
                     }
                 }