Fixed typo from commit 542099a278e79dce38e814e7e7b448a1b73df82f.

[jquery.git] / src / attributes.js

diff --git a/src/attributes.js b/src/attributes.js
index 8d889fc..0a60b0d 100644 (file)
--- a/src/attributes.js
+++ b/src/attributes.js
@@ -4,11 +4,11 @@ jQuery.fn.extend({
        },
 
        addClass: function( value ) {
-    if(jQuery.isFunction(value)) {
-      return this.each(function() {
-        jQuery(this).addClass( value.call(this) );
-      });
-    }
+               if(jQuery.isFunction(value)) {
+                       return this.each(function() {
+                               jQuery(this).addClass( value.call(this) );
+                       });
+               }
 
                if ( value && typeof value === "string" ) {
                        var classNames = (value || "").split(/\s+/);
@@ -35,11 +35,11 @@ jQuery.fn.extend({
        },
 
        removeClass: function( value ) {
-         if(jQuery.isFunction(value)) {
-           return this.each(function() {
-             jQuery(this).removeClass( value.call(this) );
-           });
-         }
+               if(jQuery.isFunction(value)) {
+                       return this.each(function() {
+                               jQuery(this).removeClass( value.call(this) );
+                       });
+               }
 
                if ( (value && typeof value === "string") || value === undefined ) {
                        var classNames = (value || "").split(/\s+/);
@@ -163,20 +163,13 @@ jQuery.fn.extend({
 
 jQuery.each({
        removeAttr: function( name ) {
-               jQuery.attr( this, name, "" );
-               if (this.nodeType == 1) {
+               if ( this.nodeType === 1 ) {
+                       this[ jQuery.isXMLDoc( this ) ? name : jQuery.props[ name ] || name ] = null;
                        this.removeAttribute( name );
                }
        },
 
        toggleClass: function( classNames, state ) {
-         if( jQuery.isFunction(classNames) ) {
-           return this.each(function() {
-             console.log(this);
-             jQuery(this).toggleClass( classNames.call(this), state );
-           });
-         }
-
                var type = typeof classNames;
                if ( type === "string" ) {
                        // toggle individual class names
@@ -198,21 +191,37 @@ jQuery.each({
        }
 }, function(name, fn){
        jQuery.fn[ name ] = function(val, state){
-         if( jQuery.isFunction( val ) ) {
-           return this.each(function() { jQuery(this)[ name ]( val.call(this), state ); });
-         }
+               if( jQuery.isFunction( val ) ) {
+                       return this.each(function() { jQuery(this)[ name ]( val.call(this), state ); });
+               }
 
                return this.each( fn, arguments );
        };
 });
 
 jQuery.extend({
+       attrFn: {
+               val: true,
+               addClass: true,
+               css: true,
+               html: true,
+               text: true,
+               append: true,
+               prepend: true,
+               data: true,
+               width: true,
+               height: true,
+               offset: true,
+               bind: true
+       },
+               
        attr: function( elem, name, value ) {
                // don't set attributes on text and comment nodes
                if (!elem || elem.nodeType == 3 || elem.nodeType == 8) {
                        return undefined;
                }
-               if ( name in jQuery.fn && name !== "attr" ) {
+
+               if ( name in jQuery.attrFn ) {
                        return jQuery(elem)[name](value);
                }