Fix for #812
[jquery.git] / src / jquery / jquery.js
index 83711de..0b0972e 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.
  *
@@ -38,14 +38,14 @@ var jQuery = function(a,c) {
        
        // Handle HTML strings
        if ( typeof a  == "string" ) {
-               // HANDLE: $(html) -> $(array)
                var m = /^[^<]*(<.+>)[^>]*$/.exec(a);
-               if ( m )
-                       a = jQuery.clean( [ m[1] ] );
+
+               a = m ?
+                       // HANDLE: $(html) -> $(array)
+                       jQuery.clean( [ m[1] ] ) :
                
-               // HANDLE: $(expr)
-               else
-                       return new jQuery( c ).find( a );
+                       // HANDLE: $(expr)
+                       jQuery.find( a, c );
        }
        
        return this.setArray(
@@ -122,11 +122,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.
  *
@@ -320,17 +315,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 +411,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 ) {
@@ -435,12 +436,12 @@ jQuery.fn = jQuery.prototype = {
                        }
                
                // 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 +545,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;
        },
 
        /**
@@ -1094,8 +1101,9 @@ jQuery.fn = jQuery.prototype = {
                        if ( table && this.nodeName.toUpperCase() == "TABLE" && a[0].nodeName.toUpperCase() == "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 ] );
+                       });
 
                });
        }
@@ -1254,13 +1262,16 @@ jQuery.extend({
                return obj;
        },
        
-       prop: function(elem, value, type){
+       prop: function(elem, value, type, index, prop){
                        // Handle executable functions
                        if ( jQuery.isFunction( value ) )
-                               return value.call( elem );
+                               return value.call( elem, [index] );
+                               
+                       // exclude the following css properties to add px
+                       var exclude = /z-?index|font-?weight|opacity/i;
 
                        // Handle passing in a number to a CSS property
-                       if ( value.constructor == Number && type == "curCSS" )
+                       if ( value.constructor == Number && type == "curCSS" && !exclude.test(prop) )
                                return value + "px";
 
                        return value;
@@ -1308,10 +1319,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") {
@@ -1387,10 +1398,8 @@ jQuery.extend({
        clean: function(a) {
                var r = [];
 
-               for ( var i = 0, al = a.length; i < al; i++ ) {
-                       var arg = a[i];
-
-                       if ( !arg ) continue;
+               jQuery.each( a, function(i,arg){
+                       if ( !arg ) return;
 
                        if ( arg.constructor == Number )
                                arg = arg.toString();
@@ -1445,14 +1454,14 @@ jQuery.extend({
                        }
 
                        if ( arg.length === 0 )
-                               continue;
+                               return;
                        
                        if ( arg[0] == undefined )
                                r.push( arg );
                        else
                                r = jQuery.merge( r, arg );
 
-               }
+               });
 
                return r;
        },