Fix for #907
[jquery.git] / src / jquery / jquery.js
index d8fa3a6..e58f895 100644 (file)
@@ -1,7 +1,7 @@
 /*
  * jQuery @VERSION - New Wave Javascript
  *
- * Copyright (c) 2006 John Resig (jquery.com)
+ * Copyright (c) 2007 John Resig (jquery.com)
  * Dual licensed under the MIT (MIT-LICENSE.txt)
  * and GPL (GPL-LICENSE.txt) licenses.
  *
@@ -32,14 +32,13 @@ var jQuery = function(a,c) {
        
        // HANDLE: $(function)
        // Shortcut for document ready
-       // Safari reports typeof on DOM NodeLists as a function
-       if ( typeof a == "function" && !a.nodeType && a[0] == undefined )
+       if ( jQuery.isFunction(a) )
                return new jQuery(document)[ jQuery.fn.ready ? "ready" : "load" ]( a );
        
        // Handle HTML strings
        if ( typeof a  == "string" ) {
                // HANDLE: $(html) -> $(array)
-               var m = /^[^<]*(<.+>)[^>]*$/.exec(a);
+               var m = /^[^<]*(<(.|\s)+>)[^>]*$/.exec(a);
                if ( m )
                        a = jQuery.clean( [ m[1] ] );
                
@@ -122,11 +121,6 @@ var $ = jQuery;
  * This function also accepts XML Documents and Window objects
  * as valid arguments (even though they are not DOM Elements).
  *
- * @example $(document).find("div > p")
- * @before <p>one</p> <div><p>two</p></div> <p>three</p>
- * @result [ <p>two</p> ]
- * @desc Same as $("div > p") because the document
- *
  * @example $(document.body).background( "black" );
  * @desc Sets the background color of the page to black.
  *
@@ -263,9 +257,9 @@ jQuery.fn = jQuery.prototype = {
         * @cat Core
         */
        pushStack: function( a ) {
-               var ret = jQuery(this);
+               var ret = jQuery(a);
                ret.prevObject = this;
-               return ret.setArray( a );
+               return ret;
        },
        
        /**
@@ -320,17 +314,17 @@ jQuery.fn = jQuery.prototype = {
         * Returns -1 if the object wasn't found.
         *
         * @example $("*").index( $('#foobar')[0] ) 
-        * @before <div id="foobar"></div><b></b><span id="foo"></span>
+        * @before <div id="foobar"><b></b><span id="foo"></span></div>
         * @result 0
         * @desc Returns the index for the element with ID foobar
         *
-        * @example $("*").index( $('#foo')) 
-        * @before <div id="foobar"></div><b></b><span id="foo"></span>
+        * @example $("*").index( $('#foo')[0] ) 
+        * @before <div id="foobar"><b></b><span id="foo"></span></div>
         * @result 2
-        * @desc Returns the index for the element with ID foo
+        * @desc Returns the index for the element with ID foo within another element
         *
-        * @example $("*").index( $('#bar')) 
-        * @before <div id="foobar"></div><b></b><span id="foo"></span>
+        * @example $("*").index( $('#bar')[0] ) 
+        * @before <div id="foobar"><b></b><span id="foo"></span></div>
         * @result -1
         * @desc Returns -1, as there is no element with ID bar
         *
@@ -416,10 +410,16 @@ jQuery.fn = jQuery.prototype = {
         * @result <img src="test.jpg" title="test.jpg" />
         * @desc Sets title attribute from src attribute.
         *
+        * @example $("img").attr("title", function(index) { return this.title + (i + 1); });
+        * @before <img title="pic" /><img title="pic" /><img title="pic" />
+        * @result <img title="pic1" /><img title="pic2" /><img title="pic3" />
+        * @desc Enumerate title attribute.
+        *
         * @name attr
         * @type jQuery
         * @param String key The name of the property to set.
         * @param Function value A function returning the value to set.
+        *                Scope: Current element, argument: Index of current element
         * @cat DOM/Attributes
         */
        attr: function( key, value, type ) {
@@ -428,19 +428,19 @@ jQuery.fn = jQuery.prototype = {
                // Look for the case where we're accessing a style value
                if ( key.constructor == String )
                        if ( value == undefined )
-                               return jQuery[ type || "attr" ]( this[0], key );
+                               return this.length && jQuery[ type || "attr" ]( this[0], key ) || undefined;
                        else {
                                obj = {};
                                obj[ key ] = value;
                        }
                
                // Check to see if we're setting style values
-               return this.each(function(){
+               return this.each(function(index){
                        // Set all the styles
                        for ( var prop in obj )
                                jQuery.attr(
                                        type ? this.style : this,
-                                       prop, jQuery.prop(this, obj[prop], type)
+                                       prop, jQuery.prop(this, obj[prop], type, index, prop)
                                );
                });
        },
@@ -544,12 +544,18 @@ jQuery.fn = jQuery.prototype = {
         * @cat DOM/Attributes
         */
        text: function(e) {
-               var type = this.length && this[0].innerText == undefined ?
-                       "textContent" : "innerText";
-                       
-               return e == undefined ?
-                       jQuery.map(this, function(a){ return a[ type ]; }).join('') :
-                       this.each(function(){ this[ type ] = e; });
+               if ( typeof e == "string" )
+                       return this.empty().append( document.createTextNode( e ) );
+
+               var t = "";
+               jQuery.each( e || this, function(){
+                       jQuery.each( this.childNodes, function(){
+                               if ( this.nodeType != 8 )
+                                       t += this.nodeType != 1 ?
+                                               this.nodeValue : jQuery.fn.text([ this ]);
+                       });
+               });
+               return t;
        },
 
        /**
@@ -797,7 +803,7 @@ jQuery.fn = jQuery.prototype = {
        find: function(t) {
                return this.pushStack( jQuery.map( this, function(a){
                        return jQuery.find(t,a);
-               }) );
+               }), t );
        },
 
        /**
@@ -864,7 +870,7 @@ jQuery.fn = jQuery.prototype = {
         */
        filter: function(t) {
                return this.pushStack(
-                       t.constructor == Function &&
+                       jQuery.isFunction( t ) &&
                        jQuery.grep(this, function(el, index){
                                return t.apply(el, [index])
                        }) ||
@@ -915,28 +921,28 @@ jQuery.fn = jQuery.prototype = {
         *
         * @name not
         * @type jQuery
-        * @param Array|jQuery elems A set of elements to remove from the jQuery set of matched elements.
+        * @param jQuery elems A set of elements to remove from the jQuery set of matched elements.
         * @cat DOM/Traversing
         */
        not: function(t) {
                return this.pushStack(
                        t.constructor == String &&
-                       jQuery.multiFilter(t,this,true) ||
-
-                       jQuery.grep(this,function(a){
-                                       if ( t.constructor == Array || t.jquery )
-                                               return jQuery.inArray( t, a ) < 0;
-                                       else
-                                               return a != t;
-                       }) );
+                       jQuery.multiFilter(t, this, true) ||
+
+                       jQuery.grep(this, function(a) {
+                               return ( t.constructor == Array || t.jquery )
+                                       ? jQuery.inArray( a, t ) < 0
+                                       : a != t;
+                       })
+               );
        },
 
        /**
-        * Adds the elements matched by the expression to the jQuery object. This
-        * can be used to concatenate the result sets of two expressions.
+        * Adds more elements, matched by the given expression,
+        * to the set of matched elements.
         *
         * @example $("p").add("span")
-        * @before <p>Hello</p><p><span>Hello Again</span></p>
+        * @before <p>Hello</p><span>Hello Again</span>
         * @result [ <p>Hello</p>, <span>Hello Again</span> ]
         *
         * @name add
@@ -946,7 +952,8 @@ jQuery.fn = jQuery.prototype = {
         */
         
        /**
-        * Adds the on the fly created elements to the jQuery object.
+        * Adds more elements, created on the fly, to the set of
+        * matched elements.
         *
         * @example $("p").add("<span>Again</span>")
         * @before <p>Hello</p>
@@ -961,15 +968,13 @@ jQuery.fn = jQuery.prototype = {
        /**
         * Adds one or more Elements to the set of matched elements.
         *
-        * This is used to add a set of Elements to a jQuery object.
-        *
         * @example $("p").add( document.getElementById("a") )
         * @before <p>Hello</p><p><span id="a">Hello Again</span></p>
         * @result [ <p>Hello</p>, <span id="a">Hello Again</span> ]
         *
-        * @example $("p").add([document.getElementById("a"), document.getElementById("b")])
-        * @before <p>Hello</p><p><span id="a">Hello Again</span><span id="b">And Again</span></p>
-        * @result [ <p>Hello</p>, <span id="a">Hello Again</span>, <span id="b">And Again</span> ]
+        * @example $("p").add( document.forms[0].elements )
+        * @before <p>Hello</p><p><form><input/><button/></form>
+        * @result [ <p>Hello</p>, <input/>, <button/> ]
         *
         * @name add
         * @type jQuery
@@ -979,7 +984,10 @@ jQuery.fn = jQuery.prototype = {
        add: function(t) {
                return this.pushStack( jQuery.merge(
                        this.get(),
-                       typeof t == "string" ? jQuery(t).get() : t )
+                       t.constructor == String ?
+                               jQuery(t).get() :
+                               t.length != undefined && (!t.nodeName || t.nodeName == "FORM") ?
+                                       t : [t] )
                );
        },
 
@@ -1092,11 +1100,12 @@ jQuery.fn = jQuery.prototype = {
                return this.each(function(){
                        var obj = this;
 
-                       if ( table && this.nodeName.toUpperCase() == "TABLE" && a[0].nodeName.toUpperCase() == "TR" )
+                       if ( table && jQuery.nodeName(this, "table") && jQuery.nodeName(a[0], "tr") )
                                obj = this.getElementsByTagName("tbody")[0] || this.appendChild(document.createElement("tbody"));
 
-                       for ( var i = 0, al = a.length; i < al; i++ )
-                               fn.apply( obj, [ clone ? a[i].cloneNode(true) : a[i] ] );
+                       jQuery.each( a, function(){
+                               fn.apply( obj, [ clone ? this.cloneNode(true) : this ] );
+                       });
 
                });
        }
@@ -1210,6 +1219,23 @@ jQuery.extend({
        noConflict: function() {
                if ( jQuery._$ )
                        $ = jQuery._$;
+               return jQuery;
+       },
+
+       // This may seem like some crazy code, but trust me when I say that this
+       // is the only cross-browser way to do this. --John
+       isFunction: function( fn ) {
+               return !!fn && typeof fn != "string" &&
+                       typeof fn[0] == "undefined" && /function/i.test( fn + "" );
+       },
+       
+       // check if an element is in a XML document
+       isXMLDoc: function(elem) {
+               return elem.tagName && elem.ownerDocument && !elem.ownerDocument.body;
+       },
+
+       nodeName: function( elem, name ) {
+               return elem.nodeName && elem.nodeName.toUpperCase() == name.toUpperCase();
        },
 
        /**
@@ -1251,16 +1277,18 @@ jQuery.extend({
                return obj;
        },
        
-       prop: function(elem, value, type){
+       prop: function(elem, value, type, index, prop){
                        // Handle executable functions
-                       if ( value.constructor == Function )
-                               return value.call( elem );
+                       if ( jQuery.isFunction( value ) )
+                               return value.call( elem, [index] );
+                               
+                       // exclude the following css properties to add px
+                       var exclude = /z-?index|font-?weight|opacity|zoom|line-?height/i;
 
                        // Handle passing in a number to a CSS property
-                       if ( value.constructor == Number && type == "css" )
-                               return value + "px";
-
-                       return value;
+                       return value && value.constructor == Number && type == "curCSS" && !exclude.test(prop) ?
+                               value + "px" :
+                               value;
        },
 
        className: {
@@ -1277,7 +1305,7 @@ jQuery.extend({
                        elem.className = c ?
                                jQuery.grep( elem.className.split(/\s+/), function(cur){
                                        return !jQuery.className.has( c, cur ); 
-                               }).join(' ') : "";
+                               }).join(" ") : "";
                },
 
                // internal only, use is(".class")
@@ -1305,10 +1333,10 @@ jQuery.extend({
                if ( p == "height" || p == "width" ) {
                        var old = {}, oHeight, oWidth, d = ["Top","Bottom","Right","Left"];
 
-                       for ( var i = 0, dl = d.length; i < dl; i++ ) {
-                               old["padding" + d[i]] = 0;
-                               old["border" + d[i] + "Width"] = 0;
-                       }
+                       jQuery.each( d, function(){
+                               old["padding" + this] = 0;
+                               old["border" + this + "Width"] = 0;
+                       });
 
                        jQuery.swap( e, old, function() {
                                if (jQuery.css(e,"display") != "none") {
@@ -1344,8 +1372,8 @@ jQuery.extend({
        curCSS: function(elem, prop, force) {
                var ret;
                
-               if (prop == 'opacity' && jQuery.browser.msie)
-                       return jQuery.attr(elem.style, 'opacity');
+               if (prop == "opacity" && jQuery.browser.msie)
+                       return jQuery.attr(elem.style, "opacity");
                        
                if (prop == "float" || prop == "cssFloat")
                    prop = jQuery.browser.msie ? "styleFloat" : "cssFloat";
@@ -1363,12 +1391,12 @@ jQuery.extend({
 
                        if ( cur )
                                ret = cur.getPropertyValue(prop);
-                       else if ( prop == 'display' )
-                               ret = 'none';
+                       else if ( prop == "display" )
+                               ret = "none";
                        else
-                               jQuery.swap(elem, { display: 'block' }, function() {
-                                   var c = document.defaultView.getComputedStyle(this, '');
-                                   ret = c && c.getPropertyValue(prop) || '';
+                               jQuery.swap(elem, { display: "block" }, function() {
+                                   var c = document.defaultView.getComputedStyle(this, "");
+                                   ret = c && c.getPropertyValue(prop) || "";
                                });
 
                } else if (elem.currentStyle) {
@@ -1384,10 +1412,11 @@ jQuery.extend({
        clean: function(a) {
                var r = [];
 
-               for ( var i = 0, al = a.length; i < al; i++ ) {
-                       var arg = a[i];
+               jQuery.each( a, function(i,arg){
+                       if ( !arg ) return;
 
-                       if ( !arg ) continue;
+                       if ( arg.constructor == Number )
+                               arg = arg.toString();
                        
                         // Convert html string into DOM nodes
                        if ( typeof arg == "string" ) {
@@ -1430,26 +1459,29 @@ jQuery.extend({
                                                tb = div.childNodes;
 
                                        for ( var n = tb.length-1; n >= 0 ; --n )
-                                               if ( tb[n].nodeName.toUpperCase() == "TBODY" && !tb[n].childNodes.length )
+                                               if ( jQuery.nodeName(tb[n], "tbody") && !tb[n].childNodes.length )
                                                        tb[n].parentNode.removeChild(tb[n]);
                                        
                                }
                                
                                arg = div.childNodes;
                        }
+
+                       if ( arg.length === 0 )
+                               return;
                        
                        if ( arg[0] == undefined )
                                r.push( arg );
                        else
                                r = jQuery.merge( r, arg );
 
-               }
+               });
 
                return r;
        },
        
        attr: function(elem, name, value){
-               var fix = {
+               var fix = jQuery.isXMLDoc(elem) ? {} : {
                        "for": "htmlFor",
                        "class": "className",
                        "float": jQuery.browser.msie ? "styleFloat" : "cssFloat",
@@ -1480,18 +1512,21 @@ jQuery.extend({
                // Mozilla doesn't play well with opacity 1
                if ( name == "opacity" && jQuery.browser.mozilla && value == 1 )
                        value = 0.9999;
+                       
+               // 
 
                // Certain attributes only work when accessed via the old DOM 0 way
                if ( fix[name] ) {
                        if ( value != undefined ) elem[fix[name]] = value;
                        return elem[fix[name]];
 
-               } else if ( value == undefined && jQuery.browser.msie && elem.nodeName && elem.nodeName.toUpperCase() == 'FORM' && (name == 'action' || name == 'method') )
+               } else if ( value == undefined && jQuery.browser.msie && jQuery.nodeName(elem, "form") && (name == "action" || name == "method") )
                        return elem.getAttributeNode(name).nodeValue;
 
                // IE elem.getAttribute passes even for style
                else if ( elem.tagName ) {
-                       if ( value != undefined ) elem.setAttribute( name, value );
+                       if ( value != undefined ) 
+                               elem.setAttribute( name, value );
                        return elem.getAttribute( name );
 
                } else {
@@ -1538,21 +1573,22 @@ jQuery.extend({
        /**
         * Merge two arrays together, removing all duplicates.
         *
-        * The new array is: All the results from the first array, followed
-        * by the unique results from the second array.
+        * The result is the altered first argument with
+        * the unique elements from the second array added.
         *
         * @example $.merge( [0,1,2], [2,3,4] )
         * @result [0,1,2,3,4]
         * @desc Merges two arrays, removing the duplicate 2
         *
-        * @example $.merge( [3,2,1], [4,3,2] )
-        * @result [3,2,1,4]
+        * @example var array = [3,2,1];
+        * $.merge( array, [4,3,2] )
+        * @result array == [3,2,1,4]
         * @desc Merges two arrays, removing the duplicates 3 and 2
         *
         * @name $.merge
         * @type Array
-        * @param Array first The first array to merge.
-        * @param Array second The second array to merge.
+        * @param Array first The first array to merge, the unique elements of second added.
+        * @param Array second The second array to merge into the first, unaltered.
         * @cat JavaScript
         */
        merge: function(first, second) {
@@ -1701,7 +1737,7 @@ jQuery.extend({
  */
  
 /*
- * Wheather the W3C compliant box model is being used.
+ * Whether the W3C compliant box model is being used.
  *
  * @property
  * @name $.boxModel
@@ -2177,7 +2213,7 @@ jQuery.each( [ "eq", "lt", "gt", "contains" ], function(i,n){
  *
  * @name width
  * @type jQuery
- * @param Number|String val Set the CSS property to the specified value.
+ * @param String|Number val Set the CSS property to the specified value.
  * @cat CSS
  */
  
@@ -2207,7 +2243,7 @@ jQuery.each( [ "eq", "lt", "gt", "contains" ], function(i,n){
  *
  * @name height
  * @type jQuery
- * @param Number|String val Set the CSS property to the specified value.
+ * @param String|Number val Set the CSS property to the specified value.
  * @cat CSS
  */