Fixed a bug with .append() and <td>/<tr>.
[jquery.git] / src / jquery / jquery.js
index 50549bb..f927a35 100644 (file)
@@ -430,28 +430,26 @@ jQuery.fn = jQuery.prototype = {
         * @cat DOM/Attributes
         */
        attr: function( key, value, type ) {
+               var obj = key;
+               
+               // 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 );
+                       else {
+                               obj = {};
+                               obj[ key ] = value;
+                       }
+               
                // Check to see if we're setting style values
-               return typeof key != "string" || value != undefined ?
-                       this.each(function(){
-                               // See if we're setting a hash of styles
-                               if ( value == undefined )
-                                       // Set all the styles
-                                       for ( var prop in key )
-                                               jQuery.attr(
-                                                       type ? this.style : this,
-                                                       prop, jQuery.prop(this, prop, key[prop], type)
-                                               );
-
-                               // See if we're setting a single key/value style
-                               else
-                                       jQuery.attr(
-                                               type ? this.style : this,
-                                               key, jQuery.prop(this, key, value, type)
-                                       );
-                       }) :
-
-                       // Look for the case where we're accessing a style value
-                       jQuery[ type || "attr" ]( this[0], key );
+               return this.each(function(){
+                       // Set all the styles
+                       for ( var prop in obj )
+                               jQuery.attr(
+                                       type ? this.style : this,
+                                       prop, jQuery.prop(this, prop, obj[prop], type)
+                               );
+               });
        },
 
        /**
@@ -528,10 +526,8 @@ jQuery.fn = jQuery.prototype = {
        /**
         * Set the text contents of all matched elements.
         *
-        * Similar to html(), but  escapes HTML (replace "<" and ">" with their
-        * HTML entities.
-        *
-        * If stripTags argument is set to true, HTML is stripped.
+        * Similar to html(), but escapes HTML (replace "<" and ">" with their
+        * HTML entities).
         *
         * @example $("p").text("<b>Some</b> new text.");
         * @before <p>Test Paragraph.</p>
@@ -546,23 +542,15 @@ jQuery.fn = jQuery.prototype = {
         * @name text
         * @type String
         * @param String val The text value to set the contents of the element to.
-        * @param Boolean stripTags (optional) Wheather to strip or only escape tags
         * @cat DOM/Attributes
         */
-       text: function(e, stripTags) {
-               if ( typeof e == "string" )
-                       return this.html( stripTags ? e.replace(/<\/?[^>]+>/gi, '') : e.replace(/</g, "&lt;").replace(/>/g, "&gt;") );
-
-               e = e || this;
-               var t = "";
-               for ( var j = 0, el = e.length; j < el; j++ ) {
-                       var r = e[j].childNodes;
-                       for ( var i = 0, rl = r.length; i < rl; i++ )
-                               if ( r[i].nodeType != 8 )
-                                       t += r[i].nodeType != 1 ?
-                                               r[i].nodeValue : jQuery.fn.text([ r[i] ]);
-               }
-               return t;
+       text: function(e) {
+               var type = this.length && this[0].innerText == undefined ?
+                       "textContent" : "innerText";
+                       
+               return e == undefined ?
+                       this.length && this[0][ type ] :
+                       this.each(function(){ this[ type ] = e; });
        },
 
        /**
@@ -1051,7 +1039,7 @@ jQuery.fn = jQuery.prototype = {
         * @cat DOM/Attributes
         */
        html: function( val ) {
-               return val == undefined ?\r                      ( this.length ? this[0].innerHTML : null ) :\r                   this.attr( "innerHTML", val );
+               return val == undefined ?\r                      ( this.length ? this[0].innerHTML : null ) :\r                   this.empty().append( val );
        },
        
        /**
@@ -1235,24 +1223,28 @@ jQuery.extend({
        prop: function(elem, key, value){
                // Handle executable functions
                return value.constructor == Function &&
-                       value.call( elem, val ) || value;
+                       value.call( elem ) || value;
        },
 
        className: {
+               // internal only, use addClass("class")
                add: function( elem, c ){
                        jQuery.each( c.split(/\s+/), function(i, cur){
                                if ( !jQuery.className.has( elem.className, cur ) )
                                        elem.className += ( elem.className ? " " : "" ) + cur;
                        });
                },
+               // internal only, use removeClass("class")
                remove: function( elem, c ){
             elem.className = c ?
                 jQuery.grep( elem.className.split(/\s+/), function(cur){
                                    return !jQuery.className.has( c, cur );     
                 }).join(' ') : "";
                },
-               has: function( classes, c ){
-                       return classes && new RegExp("(^|\\s)" + c + "(\\s|$)").test( classes );
+               // internal only, use is(".class")
+               has: function( t, c ) {
+                       t = t.className || t;
+                       return t && new RegExp("(^|\\s)" + c + "(\\s|$)").test( t );
                }
        },
 
@@ -1319,11 +1311,10 @@ jQuery.extend({
                if (prop == "float" || prop == "cssFloat")
                    prop = jQuery.browser.msie ? "styleFloat" : "cssFloat";
 
-               if (!force && elem.style[prop]) {
-
+               if (!force && elem.style[prop])
                        ret = elem.style[prop];
 
-               } else if (document.defaultView && document.defaultView.getComputedStyle) {
+               else if (document.defaultView && document.defaultView.getComputedStyle) {
 
                        if (prop == "cssFloat" || prop == "styleFloat")
                                prop = "float";
@@ -1353,51 +1344,64 @@ jQuery.extend({
        
        clean: function(a) {
                var r = [];
+               
                for ( var i = 0, al = a.length; i < al; i++ ) {
                        var arg = a[i];
-                       if ( typeof arg == "string" ) { // Convert html string into DOM nodes
+                       
+                        // Convert html string into DOM nodes
+                       if ( typeof arg == "string" ) {
                                // Trim whitespace, otherwise indexOf won't work as expected
-                               var s = jQuery.trim(arg), s3 = s.substring(0,3), s6 = s.substring(0,6),
-                                       div = document.createElement("div"), wrap = [0,"",""];
-
-                               if ( s.substring(0,4) == "<opt" ) // option or optgroup
-                                       wrap = [1, "<select>", "</select>"];
-                               else if ( s6 == "<thead" || s6 == "<tbody" || s6 == "<tfoot" )
-                                       wrap = [1, "<table>", "</table>"];
-                               else if ( s3 == "<tr" )
-                                       wrap = [2, "<table><tbody>", "</tbody></table>"];
-                               else if ( s3 == "<td" || s3 == "<th" ) // <thead> matched above
-                                       wrap = [3, "<table><tbody><tr>", "</tr></tbody></table>"];
+                               var s = jQuery.trim(arg), div = document.createElement("div"), tb = [];
+
+                               var wrap =
+                                        // option or optgroup
+                                       !s.indexOf("<opt") &&
+                                       [1, "<select>", "</select>"] ||
+                                       
+                                       (!s.indexOf("<thead") || !s.indexOf("<tbody") || !s.indexOf("<tfoot")) &&
+                                       [1, "<table>", "</table>"] ||
+                                       
+                                       !s.indexOf("<tr") &&
+                                       [2, "<table><tbody>", "</tbody></table>"] ||
+                                       
+                                       // <thead> matched above
+                                       (!s.indexOf("<td") || !s.indexOf("<th")) &&
+                                       [3, "<table><tbody><tr>", "</tr></tbody></table>"] ||
+                                       
+                                       [0,"",""];
 
                                // Go to html and back, then peel off extra wrappers
                                div.innerHTML = wrap[1] + s + wrap[2];
-                               while ( wrap[0]-- ) div = div.firstChild;
+                               
+                               // Move to the right depth
+                               while ( wrap[0]-- )
+                                       div = div.firstChild;
                                
                                // Remove IE's autoinserted <tbody> from table fragments
                                if ( jQuery.browser.msie ) {
-                                       var tb = null;
+                                       
                                        // String was a <table>, *may* have spurious <tbody>
-                                       if ( s6 == "<table" && s.indexOf("<tbody") < 0 ) 
+                                       if ( !s.indexOf("<table") && s.indexOf("<tbody") < 0 ) 
                                                tb = div.firstChild && div.firstChild.childNodes;
+                                               
                                        // String was a bare <thead> or <tfoot>
                                        else if ( wrap[1] == "<table>" && s.indexOf("<tbody") < 0 )
                                                tb = div.childNodes;
-                                       if ( tb ) {
-                                               for ( var n = tb.length-1; n >= 0 ; --n )
-                                                       if ( tb[n].nodeName.toUpperCase() == "TBODY" && !tb[n].childNodes.length )
-                                                               tb[n].parentNode.removeChild(tb[n]);
-                                       }
+
+                                       for ( var n = tb.length-1; n >= 0 ; --n )
+                                               if ( tb[n].nodeName.toUpperCase() == "TBODY" && !tb[n].childNodes.length )
+                                                       tb[n].parentNode.removeChild(tb[n]);
+                                       
                                }
                                
                                arg = div.childNodes;
-                       } 
-                       
+                       }
                        
-                       if ( arg.length != undefined && ( (jQuery.browser.safari && typeof arg == 'function') || !arg.nodeType ) ) // Safari reports typeof on a DOM NodeList to be a function
-                               for ( var n = 0, argl = arg.length; n < argl; n++ ) // Handles Array, jQuery, DOM NodeList collections
-                                       r.push(arg[n]);
+                       if ( arg.nodeType )
+                               r.push( arg );
                        else
-                               r.push( arg.nodeType ? arg : document.createTextNode(arg.toString()) );
+                               r = jQuery.merge( r, arg );
+
                }
 
                return r;
@@ -1428,10 +1432,9 @@ jQuery.extend({
                        return elem.filter = elem.filter.replace(/alpha\([^\)]*\)/gi,"") +
                                ( value == 1 ? "" : "alpha(opacity=" + value * 100 + ")" );
 
-               } else if ( name == "opacity" && jQuery.browser.msie ) {
+               } else if ( name == "opacity" && jQuery.browser.msie )
                        return elem.filter ? 
                                parseFloat( elem.filter.match(/alpha\(opacity=(.*)\)/)[1] ) / 100 : 1;
-               }
                
                // Mozilla doesn't play well with opacity 1
                if ( name == "opacity" && jQuery.browser.mozilla && value == 1 )
@@ -1442,11 +1445,11 @@ jQuery.extend({
                        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 && elem.nodeName && elem.nodeName.toUpperCase() == 'FORM' && (name == 'action' || name == 'method') )
                        return elem.getAttributeNode(name).nodeValue;
 
                // IE elem.getAttribute passes even for style
-               } else if ( elem.tagName ) {
+               else if ( elem.tagName ) {
                        if ( value != undefined ) elem.setAttribute( name, value );
                        return elem.getAttribute( name );
 
@@ -1475,10 +1478,10 @@ jQuery.extend({
        makeArray: function( a ) {
                var r = [];
 
-               if ( a.constructor != Array ) {
+               if ( a.constructor != Array )
                        for ( var i = 0, al = a.length; i < al; i++ )
                                r.push( a[i] );
-               } else
+               else
                        r = a.slice( 0 );
 
                return r;
@@ -1516,12 +1519,11 @@ jQuery.extend({
 
                // Now check for duplicates between the two arrays
                // and only add the unique items
-               for ( var i = 0, sl = second.length; i < sl; i++ ) {
+               for ( var i = 0, sl = second.length; i < sl; i++ )
                        // Check for duplicates
                        if ( jQuery.inArray( second[i], r ) == -1 )
                                // The item is unique, add it
                                first.push( second[i] );
-               }
 
                return first;
        },
@@ -1817,9 +1819,9 @@ new function() {
  */
 jQuery.each({
        parent: "a.parentNode",
-       parents: jQuery.parents,
-       next: "jQuery.nth(a,1,'nextSibling')",
-       prev: "jQuery.nth(a,1,'previousSibling')",
+       parents: "jQuery.parents(a)",
+       next: "jQuery.nth(a,2,'nextSibling')",
+       prev: "jQuery.nth(a,2,'previousSibling')",
        siblings: "jQuery.sibling(a.parentNode.firstChild,a)",
        children: "jQuery.sibling(a.firstChild)"
 }, function(i,n){
@@ -1928,50 +1930,6 @@ jQuery.each({
  */
 
 /**
- * Displays each of the set of matched elements if they are hidden.
- *
- * @example $("p").show()
- * @before <p style="display: none">Hello</p>
- * @result [ <p style="display: block">Hello</p> ]
- *
- * @name show
- * @type jQuery
- * @cat Effects
- */
-
-/**
- * Hides each of the set of matched elements if they are shown.
- *
- * @example $("p").hide()
- * @before <p>Hello</p>
- * @result [ <p style="display: none">Hello</p> ]
- *
- * var pass = true, div = $("div");
- * div.hide().each(function(){
- *   if ( this.style.display != "none" ) pass = false;
- * });
- * ok( pass, "Hide" );
- *
- * @name hide
- * @type jQuery
- * @cat Effects
- */
-
-/**
- * Toggles each of the set of matched elements. If they are shown,
- * toggle makes them hidden. If they are hidden, toggle
- * makes them shown.
- *
- * @example $("p").toggle()
- * @before <p>Hello</p><p style="display: none">Hello Again</p>
- * @result [ <p style="display: none">Hello</p>, <p style="display: block">Hello Again</p> ]
- *
- * @name toggle
- * @type jQuery
- * @cat Effects
- */
-
-/**
  * Adds the specified class to each of the set of matched elements.
  *
  * @example $("p").addClass("selected")
@@ -2054,20 +2012,6 @@ jQuery.each( {
                jQuery.attr( this, key, "" );
                this.removeAttribute( key );
        },
-       show: function(){
-               this.style.display = this.oldblock ? this.oldblock : "";
-               if ( jQuery.css(this,"display") == "none" )
-                       this.style.display = "block";
-       },
-       hide: function(){
-               this.oldblock = this.oldblock || jQuery.css(this,"display");
-               if ( this.oldblock == "none" )
-                       this.oldblock = "block";
-               this.style.display = "none";
-       },
-       toggle: function(){
-               jQuery(this)[ jQuery(this).is(":hidden") ? "show" : "hide" ].apply( jQuery(this), arguments );
-       },
        addClass: function(c){
                jQuery.className.add(this,c);
        },