]> source.dussan.org Git - jquery.git/commitdiff
Completely overhauled the syntax of core - re-worked variable names, formatting,...
authorJohn Resig <jeresig@gmail.com>
Sun, 23 Sep 2007 16:55:19 +0000 (16:55 +0000)
committerJohn Resig <jeresig@gmail.com>
Sun, 23 Sep 2007 16:55:19 +0000 (16:55 +0000)
src/core.js

index 92945a4ca5d8cf76ac3d7a08c01891e7447d8e06..3b0c46567edfcd1aa58421d2f4765042a7573606 100644 (file)
 if ( typeof jQuery != "undefined" )
        var _jQuery = jQuery;
 
-var jQuery = window.jQuery = function(selector, context) {
+var jQuery = window.jQuery = function( selector, context ) {
        // If the context is a namespace object, return a new object
        return this instanceof jQuery ?
-               this.init(selector, context) :
-               new jQuery(selector, context);
+               this.init( selector, context ) :
+               new jQuery( selector, context );
 };
 
 // Map over the $ in case of overwrite
@@ -27,67 +27,84 @@ if ( typeof $ != "undefined" )
 // Map the jQuery namespace to the '$' one
 window.$ = jQuery;
 
+// A simple way to check for HTML strings or ID strings
+// (both of which we optimize for)
 var quickExpr = /^[^<]*(<(.|\s)+>)[^>]*$|^#(\w+)$/;
 
 jQuery.fn = jQuery.prototype = {
-       init: function(selector, context) {
+       init: function( selector, context ) {
                // Make sure that a selection was provided
                selector = selector || document;
 
                // Handle HTML strings
                if ( typeof selector  == "string" ) {
-                       var m = quickExpr.exec(selector);
-                       if ( m && (m[1] || !context) ) {
+                       // Are we dealing with HTML string or an ID?
+                       var match = quickExpr.exec( selector );
+
+                       // Verify a match, and that no context was specified for #id
+                       if ( match && (match[1] || !context) ) {
+
                                // HANDLE: $(html) -> $(array)
-                               if ( m[1] )
-                                       selector = jQuery.clean( [ m[1] ], context );
+                               if ( match[1] )
+                                       selector = jQuery.clean( [ match[1] ], context );
 
                                // HANDLE: $("#id")
                                else {
-                                       var tmp = document.getElementById( m[3] );
-                                       if ( tmp )
+                                       var elem = document.getElementById( match[3] );
+
+                                       // Make sure an element was located
+                                       if ( elem )
                                                // Handle the case where IE and Opera return items
                                                // by name instead of ID
-                                               if ( tmp.id != m[3] )
+                                               if ( elem.id != match[3] )
                                                        return jQuery().find( selector );
+
+                                               // Otherwise, we inject the element directly into the jQuery object
                                                else {
-                                                       this[0] = tmp;
+                                                       this[0] = elem;
                                                        this.length = 1;
                                                        return this;
                                                }
+
                                        else
                                                selector = [];
                                }
 
-                       // HANDLE: $(expr)
+                       // HANDLE: $(expr, [context])
+                       // (which is just equivalent to: $(content).find(expr)
                        } else
                                return new jQuery( context ).find( selector );
 
                // HANDLE: $(function)
                // Shortcut for document ready
-               } else if ( jQuery.isFunction(selector) )
-                       return new jQuery(document)[ jQuery.fn.ready ? "ready" : "load" ]( selector );
+               } else if ( jQuery.isFunction( selector ) )
+                       return new jQuery( document )[ jQuery.fn.ready ? "ready" : "load" ]( selector );
 
                return this.setArray(
                        // HANDLE: $(array)
                        selector.constructor == Array && selector ||
 
                        // HANDLE: $(arraylike)
-                       // Watch for when an array-like object is passed as the selector
+                       // Watch for when an array-like object, contains DOM nodes, is passed in as the selector
                        (selector.jquery || selector.length && selector != window && !selector.nodeType && selector[0] != undefined && selector[0].nodeType) && jQuery.makeArray( selector ) ||
 
                        // HANDLE: $(*)
                        [ selector ] );
        },
        
+       // The current version of jQuery being used
        jquery: "@VERSION",
 
+       // The number of elements contained in the matched element set
        size: function() {
                return this.length;
        },
        
+       // The number of elements contained in the matched element set
        length: 0,
 
+       // Get the Nth element in the matched element set OR
+       // Get the whole matched element set as a clean array
        get: function( num ) {
                return num == undefined ?
 
@@ -95,52 +112,77 @@ jQuery.fn = jQuery.prototype = {
                        jQuery.makeArray( this ) :
 
                        // Return just the object
-                       this[num];
+                       this[ num ];
        },
        
-       pushStack: function( a ) {
-               var ret = jQuery(a);
+       // Take an array of elements and push it onto the stack
+       // (returning the new matched element set)
+       pushStack: function( elems ) {
+               // Build a new jQuery matched element set
+               var ret = jQuery( elems );
+
+               // Add the old object onto the stack (as a reference)
                ret.prevObject = this;
+
+               // Return the newly-formed element set
                return ret;
        },
        
-       setArray: function( a ) {
+       // Force the current matched set of elements to become
+       // the specified array of elements (destroying the stack in the process)
+       // You should use pushStack() in order to do this, but maintain the stack
+       setArray: function( elems ) {
+               // Resetting the length to 0, then using the native Array push
+               // is a super-fast way to populate an object with array-like properties
                this.length = 0;
-               Array.prototype.push.apply( this, a );
+               Array.prototype.push.apply( this, elems );
+               
                return this;
        },
 
-       each: function( fn, args ) {
-               return jQuery.each( this, fn, args );
+       // Execute a callback for every element in the matched set.
+       // (You can seed the arguments with an array of args, but this is
+       // only used internally.)
+       each: function( callback, args ) {
+               return jQuery.each( this, callback, args );
        },
 
-       index: function( obj ) {
-               var pos = -1;
+       // Determine the position of an element within 
+       // the matched set of elements
+       index: function( elem ) {
+               var ret = -1;
+
+               // Locate the position of the desired element
                this.each(function(i){
-                       if ( this == obj ) pos = i;
+                       if ( this == elem )
+                               ret = i;
                });
-               return pos;
+
+               return ret;
        },
 
-       attr: function( key, value, type ) {
-               var obj = key;
+       attr: function( name, value, type ) {
+               var options = name;
                
                // Look for the case where we're accessing a style value
-               if ( key.constructor == String )
+               if ( name.constructor == String )
                        if ( value == undefined )
-                               return this.length && jQuery[ type || "attr" ]( this[0], key ) || undefined;
+                               return this.length && jQuery[ type || "attr" ]( this[0], name ) || undefined;
+
                        else {
-                               obj = {};
-                               obj[ key ] = value;
+                               options = {};
+                               options[ name ] = value;
                        }
                
                // Check to see if we're setting style values
-               return this.each(function(index){
+               return this.each(function(i){
                        // Set all the styles
-                       for ( var prop in obj )
+                       for ( name in options )
                                jQuery.attr(
-                                       type ? this.style : this,
-                                       prop, jQuery.prop(this, obj[prop], type, index, prop)
+                                       type ?
+                                               this.style :
+                                               this,
+                                       name, jQuery.prop( this, options[ name ], type, i, name )
                                );
                });
        },
@@ -149,31 +191,36 @@ jQuery.fn = jQuery.prototype = {
                return this.attr( key, value, "curCSS" );
        },
 
-       text: function(e) {
-               if ( typeof e != "object" && e != null )
-                       return this.empty().append( document.createTextNode( e ) );
+       text: function( text ) {
+               if ( typeof text != "object" && text != null )
+                       return this.empty().append( document.createTextNode( text ) );
+
+               var ret = "";
 
-               var t = "";
-               jQuery.each( e || this, function(){
+               jQuery.each( text || this, function(){
                        jQuery.each( this.childNodes, function(){
                                if ( this.nodeType != 8 )
-                                       t += this.nodeType != 1 ?
-                                               this.nodeValue : jQuery.fn.text([ this ]);
+                                       ret += this.nodeType != 1 ?
+                                               this.nodeValue :
+                                               jQuery.fn.text( [ this ] );
                        });
                });
-               return t;
+
+               return ret;
        },
 
-       wrapAll: function(html) {
+       wrapAll: function( html ) {
                if ( this[0] )
                        // The elements to wrap the target around
-                       jQuery(html, this[0].ownerDocument)
+                       jQuery( html, this[0].ownerDocument )
                                .clone()
-                               .insertBefore(this[0])
+                               .insertBefore( this[0] )
                                .map(function(){
                                        var elem = this;
+
                                        while ( elem.firstChild )
                                                elem = elem.firstChild;
+
                                        return elem;
                                })
                                .append(this);
@@ -181,56 +228,62 @@ jQuery.fn = jQuery.prototype = {
                return this;
        },
 
-       wrapInner: function(html) {
+       wrapInner: function( html ) {
                return this.each(function(){
-                       jQuery(this).contents().wrapAll(html);
+                       jQuery( this ).contents().wrapAll( html );
                });
        },
 
-       wrap: function(html) {
+       wrap: function( html ) {
                return this.each(function(){
-                       jQuery(this).wrapAll(html);
+                       jQuery( this ).wrapAll( html );
                });
        },
 
        append: function() {
-               return this.domManip(arguments, true, 1, function(a){
-                       this.appendChild( a );
+               return this.domManip(arguments, true, false, function(elem){
+                       this.appendChild( elem );
                });
        },
 
        prepend: function() {
-               return this.domManip(arguments, true, -1, function(a){
-                       this.insertBefore( a, this.firstChild );
+               return this.domManip(arguments, true, true, function(elem){
+                       this.insertBefore( elem, this.firstChild );
                });
        },
        
        before: function() {
-               return this.domManip(arguments, false, 1, function(a){
-                       this.parentNode.insertBefore( a, this );
+               return this.domManip(arguments, false, false, function(elem){
+                       this.parentNode.insertBefore( elem, this );
                });
        },
 
        after: function() {
-               return this.domManip(arguments, false, -1, function(a){
-                       this.parentNode.insertBefore( a, this.nextSibling );
+               return this.domManip(arguments, false, true, function(elem){
+                       this.parentNode.insertBefore( elem, this.nextSibling );
                });
        },
 
        end: function() {
-               return this.prevObject || jQuery([]);
+               return this.prevObject || jQuery( [] );
        },
 
-       find: function(t) {
-               var data = jQuery.map(this, function(a){ return jQuery.find(t,a); });
-               return this.pushStack( /[^+>] [^+>]/.test( t ) || t.indexOf("..") > -1 ?
-                       jQuery.unique( data ) : data );
+       find: function( selector ) {
+               var elems = jQuery.map(this, function(elem){
+                       return jQuery.find( selector, elem );
+               });
+
+               return this.pushStack( /[^+>] [^+>]/.test( selector ) || selector.indexOf("..") > -1 ?
+                       jQuery.unique( elems ) :
+                       elems );
        },
 
-       clone: function(events) {
+       clone: function( events ) {
                // Do the clone
                var ret = this.map(function(){
-                       return this.outerHTML ? jQuery(this.outerHTML)[0] : this.cloneNode(true);
+                       return this.outerHTML ?
+                               jQuery( this.outerHTML )[0] :
+                               this.cloneNode( true );
                });
 
                // Need to set the expando to null on the cloned set if it exists
@@ -242,68 +295,65 @@ jQuery.fn = jQuery.prototype = {
                });
                
                // Copy the events from the original to the clone
-               if (events === true)
-                       this.find("*").andSelf().each(function(i) {
-                               var events = jQuery.data(this, "events");
+               if ( events === true )
+                       this.find("*").andSelf().each(function(i){
+                               var events = jQuery.data( this, "events" );
+
                                for ( var type in events )
-                                       for ( var handler in events[type] )
-                                               jQuery.event.add(clone[i], type, events[type][handler], events[type][handler].data);
+                                       for ( var handler in events[ type ] )
+                                               jQuery.event.add( clone[ i ], type, events[ type ][ handler ], events[ type ][ handler ].data );
                        });
 
                // Return the cloned set
                return ret;
        },
 
-       filter: function(t) {
+       filter: function( selector ) {
                return this.pushStack(
-                       jQuery.isFunction( t ) &&
-                       jQuery.grep(this, function(el, index){
-                               return t.apply(el, [index]);
+                       jQuery.isFunction( selector ) &&
+                       jQuery.grep(this, function(elem, i){
+                               return selector.call( elem, i );
                        }) ||
 
-                       jQuery.multiFilter(t,this) );
+                       jQuery.multiFilter( selector, this ) );
        },
 
-       not: function(t) {
+       not: function( selector ) {
                return this.pushStack(
-                       t.constructor == String &&
-                       jQuery.multiFilter(t, this, true) ||
-
-                       jQuery.grep(this, function(a) {
-                               return ( t.constructor == Array || t.jquery )
-                                       ? jQuery.inArray( a, t ) < 0
-                                       : a != t;
-                       })
-               );
+                       selector.constructor == String &&
+                       jQuery.multiFilter( selector, this, true ) ||
+
+                       jQuery.grep(this, function(elem) {
+                               return selector.constructor == Array || selector.jquery ?
+                                       jQuery.inArray( elem, selector ) < 0 :
+                                       elem != selector;
+                       }) );
        },
 
-       add: function(t) {
-               return this.pushStack( jQuery.merge(
-                       this.get(),
-                       t.constructor == String ?
-                               jQuery(t).get() :
-                               t.length != undefined && (!t.nodeName || jQuery.nodeName(t, "form")) ?
-                                       t : [t] )
-               );
+       add: function( selector ) {
+               return this.pushStack( jQuery.merge( this.get(), jQuery( selector ) ) );
        },
 
-       is: function(expr) {
-               return expr ? jQuery.multiFilter(expr,this).length > 0 : false;
+       is: function( selector ) {
+               return selector ?
+                       jQuery.multiFilter( selector, this ).length > 0 :
+                       false;
        },
 
-       hasClass: function(expr) {
-               return this.is("." + expr);
+       hasClass: function( selector ) {
+               return this.is( "." + selector );
        },
        
-       val: function( val ) {
-               if ( val == undefined ) {
+       val: function( value ) {
+               if ( value == undefined ) {
+
                        if ( this.length ) {
                                var elem = this[0];
                        
                                // We need to handle select boxes special
-                               if ( jQuery.nodeName(elem, "select") ) {
+                               if ( jQuery.nodeName( elem, "select" ) ) {
                                        var index = elem.selectedIndex,
-                                               a = [],
+                                               values = [],
                                                options = elem.options,
                                                one = elem.type == "select-one";
                                        
@@ -313,67 +363,76 @@ jQuery.fn = jQuery.prototype = {
 
                                        // Loop through all the selected options
                                        for ( var i = one ? index : 0, max = one ? index + 1 : options.length; i < max; i++ ) {
-                                               var option = options[i];
+                                               var option = options[ i ];
+
                                                if ( option.selected ) {
                                                        // Get the specifc value for the option
-                                                       var val = jQuery.browser.msie && !option.attributes["value"].specified ? option.text : option.value;
+                                                       value = jQuery.browser.msie && !option.attributes.value.specified ? option.text : option.value;
                                                        
                                                        // We don't need an array for one selects
                                                        if ( one )
-                                                               return val;
+                                                               return value;
                                                        
                                                        // Multi-Selects return an array
-                                                       a.push(val);
+                                                       values.push( value );
                                                }
                                        }
                                        
-                                       return a;
+                                       return values;
                                        
                                // Everything else, we just grab the value
                                } else
                                        return this[0].value.replace(/\r/g, "");
+
                        }
+
                } else
                        return this.each(function(){
-                               if ( val.constructor == Array && /radio|checkbox/.test(this.type) )
-                                       this.checked = (jQuery.inArray(this.value, val) >= 0 ||
-                                               jQuery.inArray(this.name, val) >= 0);
-                               else if ( jQuery.nodeName(this, "select") ) {
-                                       var tmp = val.constructor == Array ? val : [val];
-
-                                       jQuery("option", this).each(function(){
-                                               this.selected = (jQuery.inArray(this.value, tmp) >= 0 ||
-                                               jQuery.inArray(this.text, tmp) >= 0);
+                               if ( value.constructor == Array && /radio|checkbox/.test( this.type ) )
+                                       this.checked = (jQuery.inArray(this.value, value) >= 0 ||
+                                               jQuery.inArray(this.name, value) >= 0);
+
+                               else if ( jQuery.nodeName( this, "select" ) ) {
+                                       var values = value.constructor == Array ?
+                                               value :
+                                               [ value ];
+
+                                       jQuery( "option", this ).each(function(){
+                                               this.selected = (jQuery.inArray( this.value, values ) >= 0 ||
+                                                       jQuery.inArray( this.text, values ) >= 0);
                                        });
 
                                        if ( !tmp.length )
                                                this.selectedIndex = -1;
+
                                } else
-                                       this.value = val;
+                                       this.value = value;
                        });
        },
        
-       html: function( val ) {
-               return val == undefined ?
-                       ( this.length ? this[0].innerHTML : null ) :
-                       this.empty().append( val );
+       html: function( value ) {
+               return value == undefined ?
+                       (this.length ?
+                               this[0].innerHTML :
+                               null) :
+                       this.empty().append( value );
        },
 
-       replaceWith: function( val ) {
-               return this.after( val ).remove();
+       replaceWith: function( value ) {
+               return this.after( value ).remove();
        },
 
-       eq: function(i){
-               return this.slice(i, i+1);
+       eq: function( i ) {
+               return this.slice( i, i + 1 );
        },
 
        slice: function() {
                return this.pushStack( Array.prototype.slice.apply( this, arguments ) );
        },
 
-       map: function(fn) {
-               return this.pushStack(jQuery.map( this, function(elem,i){
-                       return fn.call( elem, i, elem );
+       map: function( callback ) {
+               return this.pushStack( jQuery.map(this, function(elem, i){
+                       return callback.call( elem, i, elem );
                }));
        },
 
@@ -381,51 +440,60 @@ jQuery.fn = jQuery.prototype = {
                return this.add( this.prevObject );
        },
        
-       domManip: function(args, table, dir, fn) {
-               var clone = this.length > 1, a
+       domManip: function( args, table, reverse, callback ) {
+               var clone = this.length > 1, elems
 
                return this.each(function(){
-                       if ( !a ) {
-                               a = jQuery.clean(args, this.ownerDocument);
-                               if ( dir < 0 )
-                                       a.reverse();
+                       if ( !elems ) {
+                               elems = jQuery.clean( args, this.ownerDocument );
+
+                               if ( reverse )
+                                       elems.reverse();
                        }
 
                        var obj = this;
 
-                       if ( table && jQuery.nodeName(this, "table") && jQuery.nodeName(a[0], "tr") )
-                               obj = this.getElementsByTagName("tbody")[0] || this.appendChild(document.createElement("tbody"));
+                       if ( table && jQuery.nodeName( this, "table" ) && jQuery.nodeName( elems[0], "tr" ) )
+                               obj = this.getElementsByTagName("tbody")[0] || this.appendChild( document.createElement("tbody") );
+
+                       jQuery.each(elems, function(){
+                               var elem = clone ?
+                                       this.cloneNode( true ) :
+                                       this;
 
-                       jQuery.each( a, function(){
-                               var elem = clone ? this.cloneNode(true) : this;
-                               if ( !evalScript(0, elem) )
-                                       fn.call( obj, elem );
+                               if ( !evalScript( 0, elem ) )
+                                       callback.call( obj, elem );
                        });
                });
        }
 };
 
-function evalScript(i, elem){
-       var script = jQuery.nodeName(elem, "script");
+function evalScript( i, elem ) {
+       var script = jQuery.nodeName( elem, "script" );
 
        if ( script ) {
                if ( elem.src )
-                       jQuery.ajax({ url: elem.src, async: false, dataType: "script" });
+                       jQuery.ajax({
+                               url: elem.src,
+                               async: false,
+                               dataType: "script"
+                       });
+
                else
                        jQuery.globalEval( elem.text || elem.textContent || elem.innerHTML || "" );
        
                if ( elem.parentNode )
-                       elem.parentNode.removeChild(elem);
+                       elem.parentNode.removeChild( elem );
 
        } else if ( elem.nodeType == 1 )
-    jQuery("script", elem).each(evalScript);
+               jQuery( "script", elem ).each( evalScript );
 
        return script;
 }
 
 jQuery.extend = jQuery.fn.extend = function() {
        // copy reference to target object
-       var target = arguments[0] || {}, a = 1, al = arguments.length, deep = false;
+       var target = arguments[0] || {}, i = 1, length = arguments.length, deep = false, options;
 
        // Handle a deep copy situation
        if ( target.constructor == Boolean ) {
@@ -434,42 +502,46 @@ jQuery.extend = jQuery.fn.extend = function() {
        }
 
        // extend jQuery itself if only one argument is passed
-       if ( al == 1 ) {
+       if ( length == 1 ) {
                target = this;
-               a = 0;
+               i = 0;
        }
 
-       var prop;
-
-       for ( ; a < al; a++ )
+       for ( ; i < length; i++ )
                // Only deal with non-null/undefined values
-               if ( (prop = arguments[a]) != null )
+               if ( (options = arguments[ i ]) != null )
                        // Extend the base object
-                       for ( var i in prop ) {
+                       for ( var name in options ) {
                                // Prevent never-ending loop
-                               if ( target == prop[i] )
+                               if ( target == options[ name ] )
                                        continue;
 
                                // Recurse if we're merging object values
-                               if ( deep && typeof prop[i] == 'object' && target[i] )
-                                       jQuery.extend( target[i], prop[i] );
+                               if ( deep && typeof options[ name ] == "object" && target[ name ] )
+                                       jQuery.extend( target[ name ], options[ name ] );
 
                                // Don't bring in undefined values
-                               else if ( prop[i] != undefined )
-                                       target[i] = prop[i];
+                               else if ( options[ name ] != undefined )
+                                       target[ name ] = options[ name ];
+
                        }
 
        // Return the modified object
        return target;
 };
 
-var expando = "jQuery" + (new Date()).getTime(), uuid = 0, win = {};
+var expando = "jQuery" + (new Date()).getTime(), uuid = 0, windowData = {};
+
+// exclude the following css properties to add px
+var exclude = /z-?index|font-?weight|opacity|zoom|line-?height/i;
 
 jQuery.extend({
-       noConflict: function(deep) {
+       noConflict: function( deep ) {
                window.$ = _$;
+
                if ( deep )
                        window.jQuery = _jQuery;
+
                return jQuery;
        },
 
@@ -480,8 +552,8 @@ jQuery.extend({
                        fn.constructor != Array && /function/i.test( fn + "" );
        },
        
-       // check if an element is in a XML document
-       isXMLDoc: function(elem) {
+       // check if an element is in a (or is an) XML document
+       isXMLDoc: function( elem ) {
                return elem.documentElement && !elem.body ||
                        elem.tagName && elem.ownerDocument && !elem.ownerDocument.body;
        },
@@ -490,12 +562,15 @@ jQuery.extend({
        // Evaluates Async. in Safari 2 :-(
        globalEval: function( data ) {
                data = jQuery.trim( data );
+
                if ( data ) {
                        if ( window.execScript )
                                window.execScript( data );
+
                        else if ( jQuery.browser.safari )
                                // safari doesn't provide a synchronous global eval
                                window.setTimeout( data, 0 );
+
                        else
                                eval.call( window, data );
                }
@@ -508,7 +583,9 @@ jQuery.extend({
        cache: {},
        
        data: function( elem, name, data ) {
-               elem = elem == window ? win : elem;
+               elem = elem == window ?
+                       windowData :
+                       elem;
 
                var id = elem[ expando ];
 
@@ -526,11 +603,15 @@ jQuery.extend({
                        jQuery.cache[ id ][ name ] = data;
                
                // Return the named cache data, or the ID for the element       
-               return name ? jQuery.cache[ id ][ name ] : id;
+               return name ?
+                       jQuery.cache[ id ][ name ] :
+                       id;
        },
        
        removeData: function( elem, name ) {
-               elem = elem == window ? win : elem;
+               elem = elem == window ?
+                       windowData :
+                       elem;
 
                var id = elem[ expando ];
 
@@ -542,7 +623,10 @@ jQuery.extend({
 
                                // If we've removed all the data, remove the element's cache
                                name = "";
-                               for ( name in jQuery.cache[ id ] ) break;
+
+                               for ( name in jQuery.cache[ id ] )
+                                       break;
+
                                if ( !name )
                                        jQuery.removeData( elem );
                        }
@@ -565,194 +649,228 @@ jQuery.extend({
        },
 
        // args is for internal usage only
-       each: function( obj, fn, args ) {
+       each: function( object, callback, args ) {
                if ( args ) {
-                       if ( obj.length == undefined )
-                               for ( var i in obj )
-                                       fn.apply( obj[i], args );
+                       if ( object.length == undefined )
+                               for ( var name in object )
+                                       callback.apply( object[ name ], args );
                        else
-                               for ( var i = 0, ol = obj.length; i < ol; i++ )
-                                       if ( fn.apply( obj[i], args ) === false ) break;
+                               for ( var i = 0, length = object.length; i < length; i++ )
+                                       if ( callback.apply( object[ i ], args ) === false )
+                                               break;
 
                // A special, fast, case for the most common use of each
                } else {
-                       if ( obj.length == undefined )
-                               for ( var i in obj )
-                                       fn.call( obj[i], i, obj[i] );
+                       if ( object.length == undefined )
+                               for ( var name in object )
+                                       callback.call( object[ name ], name, object[ name ] );
                        else
-                               for ( var i = 0, ol = obj.length, val = obj[0]; 
-                                       i < ol && fn.call(val,i,val) !== false; val = obj[++i] ){}
+                               for ( var i = 0, length = object.length, value = object[0]; 
+                                       i < length && callback.call( value, i, value ) !== false; value = object[++i] ){}
                }
 
-               return obj;
+               return object;
        },
        
-       prop: function(elem, value, type, index, prop){
+       prop: function( elem, value, type, i, name ) {
                        // Handle executable functions
                        if ( jQuery.isFunction( value ) )
-                               value = value.call( elem, [index] );
+                               value = value.call( elem, i );
                                
-                       // 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
-                       return value && value.constructor == Number && type == "curCSS" && !exclude.test(prop) ?
+                       return value && value.constructor == Number && type == "curCSS" && !exclude.test( name ) ?
                                value + "px" :
                                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;
+               add: function( elem, classNames ) {
+                       jQuery.each((classNames || "").split(/\s+/), function(i, className){
+                               if ( !jQuery.className.has( elem.className, className ) )
+                                       elem.className += (elem.className ? " " : "") + className;
                        });
                },
 
                // internal only, use removeClass("class")
-               remove: function( elem, c ){
-                       elem.className = c != undefined ?
-                               jQuery.grep( elem.className.split(/\s+/), function(cur){
-                                       return !jQuery.className.has( c, cur ); 
-                               }).join(" ") : "";
+               remove: function( elem, classNames ) {
+                       elem.className = classNames != undefined ?
+                               jQuery.grep(elem.className.split(/\s+/), function(className){
+                                       return !jQuery.className.has( classNames, className );  
+                               }).join(" ") :
+                               "";
                },
 
                // internal only, use is(".class")
-               has: function( t, c ) {
-                       return jQuery.inArray( c, (t.className || t).toString().split(/\s+/) ) > -1;
+               has: function( elem, className ) {
+                       return jQuery.inArray( className, (elem.className || elem).toString().split(/\s+/) ) > -1;
                }
        },
 
-       swap: function(e,o,f) {
-               for ( var i in o ) {
-                       e.style["old"+i] = e.style[i];
-                       e.style[i] = o[i];
+       // A method for quickly swapping in/out CSS properties to get correct calculations
+       swap: function( elem, options, callback ) {
+               // Remember the old values, and insert the new ones
+               for ( var name in options ) {
+                       elem.style[ "old" + name ] = elem.style[ name ];
+                       elem.style[ name ] = options[ name ];
                }
-               f.apply( e, [] );
-               for ( var i in o )
-                       e.style[i] = e.style["old"+i];
+
+               callback.call( elem );
+
+               // Revert the old values
+               for ( var name in options )
+                       elem.style[ name ] = elem.style[ "old" + name ];
        },
 
-       css: function(e,p) {
-               if ( p == "height" || p == "width" ) {
-                       var old = {}, oHeight, oWidth, d = ["Top","Bottom","Right","Left"];
+       css: function( elem, name ) {
+               if ( name == "height" || name == "width" ) {
+                       var old = {}, height, width;
 
-                       jQuery.each( d, function(){
-                               old["padding" + this] = 0;
-                               old["border" + this + "Width"] = 0;
+                       // Revert the padding and border widths to get the
+                       // correct height/width values
+                       jQuery.each([ "Top", "Bottom", "Right", "Left" ], function(){
+                               old[ "padding" + this ] = 0;
+                               old[ "border" + this + "Width" ] = 0;
                        });
 
-                       jQuery.swap( e, old, function() {
-                               if ( jQuery(e).is(':visible') ) {
-                                       oHeight = e.offsetHeight;
-                                       oWidth = e.offsetWidth;
+                       // Swap out the padding/border values temporarily
+                       jQuery.swap( elem, old, function() {
+
+                               // If the element is visible, then the calculation is easy
+                               if ( jQuery( elem ).is(":visible") ) {
+                                       height = elem.offsetHeight;
+                                       width = elem.offsetWidth;
+
+                               // Otherwise, we need to flip out more values
                                } else {
-                                       e = jQuery(e.cloneNode(true))
+                                       elem = jQuery( elem.cloneNode(true) )
                                                .find(":radio").removeAttr("checked").end()
                                                .css({
-                                                       visibility: "hidden", position: "absolute", display: "block", right: "0", left: "0"
-                                               }).appendTo(e.parentNode)[0];
+                                                       visibility: "hidden",
+                                                       position: "absolute",
+                                                       display: "block",
+                                                       right: "0",
+                                                       left: "0"
+                                               }).appendTo( elem.parentNode )[0];
 
-                                       var parPos = jQuery.css(e.parentNode,"position") || "static";
-                                       if ( parPos == "static" )
-                                               e.parentNode.style.position = "relative";
+                                       var position = jQuery.css( elem.parentNode, "position" ) || "static";
+                                       if ( position == "static" )
+                                               elem.parentNode.style.position = "relative";
 
-                                       oHeight = e.clientHeight;
-                                       oWidth = e.clientWidth;
+                                       height = elem.clientHeight;
+                                       width = elem.clientWidth;
 
-                                       if ( parPos == "static" )
-                                               e.parentNode.style.position = "static";
+                                       if ( position == "static" )
+                                               elem.parentNode.style.position = "static";
 
-                                       e.parentNode.removeChild(e);
+                                       elem.parentNode.removeChild( elem );
                                }
                        });
 
-                       return p == "height" ? oHeight : oWidth;
+                       return name == "height" ?
+                               height :
+                               width;
                }
 
-               return jQuery.curCSS( e, p );
+               return jQuery.curCSS( elem, name );
        },
 
-       curCSS: function(elem, prop, force) {
-               var ret, stack = [], swap = [];
+       curCSS: function( elem, name, force ) {
+               var ret;
 
                // A helper method for determining if an element's values are broken
-               function color(a){
+               function color( elem ) {
                        if ( !jQuery.browser.safari )
                                return false;
 
-                       var ret = document.defaultView.getComputedStyle(a,null);
+                       var ret = document.defaultView.getComputedStyle( elem, null );
                        return !ret || ret.getPropertyValue("color") == "";
                }
 
-               if (prop == "opacity" && jQuery.browser.msie) {
-                       ret = jQuery.attr(elem.style, "opacity");
-                       return ret == "" ? "1" : ret;
+               // We need to handle opacity special in IE
+               if ( name == "opacity" && jQuery.browser.msie ) {
+                       ret = jQuery.attr( elem.style, "opacity" );
+
+                       return ret == "" ?
+                               "1" :
+                               ret;
                }
                
-               if (prop.match(/float/i))
-                       prop = styleFloat;
+               // Make sure we're using the right name for getting the float value
+               if ( name.match( /float/i ) )
+                       name = styleFloat;
+
+               if ( !force && elem.style[ name ] )
+                       ret = elem.style[ name ];
 
-               if (!force && elem.style[prop])
-                       ret = elem.style[prop];
+               else if ( document.defaultView && document.defaultView.getComputedStyle ) {
 
-               else if (document.defaultView && document.defaultView.getComputedStyle) {
+                       // Only "float" is needed here
+                       if ( name.match( /float/i ) )
+                               name = "float";
 
-                       if (prop.match(/float/i))
-                               prop = "float";
+                       name = name.replace( /([A-Z])/g, "-$1" ).toLowerCase();
 
-                       prop = prop.replace(/([A-Z])/g,"-$1").toLowerCase();
-                       var cur = document.defaultView.getComputedStyle(elem, null);
+                       var getComputedStyle = document.defaultView.getComputedStyle( elem, null );
 
-                       if ( cur && !color(elem) )
-                               ret = cur.getPropertyValue(prop);
+                       if ( getComputedStyle && !color( elem ) )
+                               ret = getComputedStyle.getPropertyValue( name );
 
                        // If the element isn't reporting its values properly in Safari
                        // then some display: none elements are involved
                        else {
+                               var swap = [], stack = [];
+
                                // Locate all of the parent display: none elements
                                for ( var a = elem; a && color(a); a = a.parentNode )
                                        stack.unshift(a);
 
                                // Go through and make them visible, but in reverse
                                // (It would be better if we knew the exact display type that they had)
-                               for ( a = 0; a < stack.length; a++ )
-                                       if ( color(stack[a]) ) {
-                                               swap[a] = stack[a].style.display;
-                                               stack[a].style.display = "block";
+                               for ( var i = 0; i < stack.length; i++ )
+                                       if ( color( stack[ i ] ) ) {
+                                               swap[ i ] = stack[ i ].style.display;
+                                               stack[ i ].style.display = "block";
                                        }
 
                                // Since we flip the display style, we have to handle that
                                // one special, otherwise get the value
-                               ret = prop == "display" && swap[stack.length-1] != null ?
+                               ret = name == "display" && swap[ stack.length - 1 ] != null ?
                                        "none" :
-                                       document.defaultView.getComputedStyle(elem,null).getPropertyValue(prop) || "";
+                                       document.defaultView.getComputedStyle( elem, null ).getPropertyValue( name ) || "";
 
                                // Finally, revert the display styles back
-                               for ( a = 0; a < swap.length; a++ )
-                                       if ( swap[a] != null )
-                                               stack[a].style.display = swap[a];
+                               for ( var i = 0; i < swap.length; i++ )
+                                       if ( swap[ i ] != null )
+                                               stack[ i ].style.display = swap[ i ];
                        }
 
-                       if ( prop == "opacity" && ret == "" )
+                       // We should always get a number back from opacity
+                       if ( name == "opacity" && ret == "" )
                                ret = "1";
 
-               } else if (elem.currentStyle) {
-                       var newProp = prop.replace(/\-(\w)/g,function(m,c){return c.toUpperCase();});
-                       ret = elem.currentStyle[prop] || elem.currentStyle[newProp];
+               } else if ( elem.currentStyle ) {
+                       var camelCase = name.replace(/\-(\w)/g, function(all, letter){
+                               return letter.toUpperCase();
+                       });
+
+                       ret = elem.currentStyle[ name ] || elem.currentStyle[ camelCase ];
 
                        // From the awesome hack by Dean Edwards
                        // http://erik.eae.net/archives/2007/07/27/18.54.15/#comment-102291
 
                        // If we're not dealing with a regular pixel number
                        // but a number that has a weird ending, we need to convert it to pixels
-                       if ( !/^\d+(px)?$/i.test(ret) && /^\d/.test(ret) ) {
-                               var style = elem.style.left;
-                               var runtimeStyle = elem.runtimeStyle.left;
+                       if ( !/^\d+(px)?$/i.test( ret ) && /^\d/.test( ret ) ) {
+                               // Remember the original values
+                               var style = elem.style.left, runtimeStyle = elem.runtimeStyle.left;
+
+                               // Put in the new values to get a computed value out
                                elem.runtimeStyle.left = elem.currentStyle.left;
                                elem.style.left = ret || 0;
                                ret = elem.style.pixelLeft + "px";
+
+                               // Revert the changed values
                                elem.style.left = style;
                                elem.runtimeStyle.left = runtimeStyle;
                        }
@@ -761,55 +879,58 @@ jQuery.extend({
                return ret;
        },
        
-       clean: function(a, doc) {
-               var r = [];
-               doc = doc || document;
+       clean: function( elems, context ) {
+               var ret = [];
+               context = context || document;
 
-               jQuery.each( a, function(i,arg){
-                       if ( !arg ) return;
+               jQuery.each(elems, function(i, elem){
+                       if ( !elem )
+                               return;
 
-                       if ( arg.constructor == Number )
-                               arg = arg.toString();
+                       if ( elem.constructor == Number )
+                               elem = elem.toString();
                        
                        // Convert html string into DOM nodes
-                       if ( typeof arg == "string" ) {
+                       if ( typeof elem == "string" ) {
                                // Fix "XHTML"-style tags in all browsers
-                               arg = arg.replace(/(<(\w+)[^>]*?)\/>/g, function(m, all, tag){
-                                       return tag.match(/^(abbr|br|col|img|input|link|meta|param|hr|area)$/i)? m : all+"></"+tag+">";
+                               elem = elem.replace(/(<(\w+)[^>]*?)\/>/g, function(all, front, tag){
+                                       return tag.match(/^(abbr|br|col|img|input|link|meta|param|hr|area)$/i) ?
+                                               all :
+                                               front + "></" + tag + ">";
                                });
 
                                // Trim whitespace, otherwise indexOf won't work as expected
-                               var s = jQuery.trim(arg).toLowerCase(), div = doc.createElement("div"), tb = [];
+                               var tags = jQuery.trim( elem ).toLowerCase(), div = context.createElement("div");
 
                                var wrap =
                                        // option or optgroup
-                                       !s.indexOf("<opt") &&
-                                       [1, "<select>", "</select>"] ||
+                                       !tags.indexOf("<opt") &&
+                                       [ 1, "<select>", "</select>" ] ||
                                        
-                                       !s.indexOf("<leg") &&
-                                       [1, "<fieldset>", "</fieldset>"] ||
+                                       !tags.indexOf("<leg") &&
+                                       [ 1, "<fieldset>", "</fieldset>" ] ||
                                        
-                                       s.match(/^<(thead|tbody|tfoot|colg|cap)/) &&
-                                       [1, "<table>", "</table>"] ||
+                                       tags.match(/^<(thead|tbody|tfoot|colg|cap)/) &&
+                                       [ 1, "<table>", "</table>" ] ||
                                        
-                                       !s.indexOf("<tr") &&
-                                       [2, "<table><tbody>", "</tbody></table>"] ||
+                                       !tags.indexOf("<tr") &&
+                                       [ 2, "<table><tbody>", "</tbody></table>" ] ||
                                        
                                        // <thead> matched above
-                                       (!s.indexOf("<td") || !s.indexOf("<th")) &&
-                                       [3, "<table><tbody><tr>", "</tr></tbody></table>"] ||
+                                       (!tags.indexOf("<td") || !tags.indexOf("<th")) &&
+                                       [ 3, "<table><tbody><tr>", "</tr></tbody></table>" ] ||
                                        
-                                       !s.indexOf("<col") &&
-                                       [2, "<table><tbody></tbody><colgroup>", "</colgroup></table>"] ||
+                                       !tags.indexOf("<col") &&
+                                       [ 2, "<table><tbody></tbody><colgroup>", "</colgroup></table>" ] ||
 
                                        // IE can't serialize <link> and <script> tags normally
                                        jQuery.browser.msie &&
-                                       [1, "div<div>", "</div>"] ||
+                                       [ 1, "div<div>", "</div>" ] ||
                                        
-                                       [0,"",""];
+                                       [ 0, "", "" ];
 
                                // Go to html and back, then peel off extra wrappers
-                               div.innerHTML = wrap[1] + arg + wrap[2];
+                               div.innerHTML = wrap[1] + elem + wrap[2];
                                
                                // Move to the right depth
                                while ( wrap[0]-- )
@@ -819,41 +940,45 @@ jQuery.extend({
                                if ( jQuery.browser.msie ) {
                                        
                                        // String was a <table>, *may* have spurious <tbody>
-                                       if ( !s.indexOf("<table") && s.indexOf("<tbody") < 0 ) 
-                                               tb = div.firstChild && div.firstChild.childNodes;
+                                       var tbody = !tags.indexOf("<table") && tags.indexOf("<tbody") < 0 ?
+                                               div.firstChild && div.firstChild.childNodes :
                                                
-                                       // String was a bare <thead> or <tfoot>
-                                       else if ( wrap[1] == "<table>" && s.indexOf("<tbody") < 0 )
-                                               tb = div.childNodes;
-
-                                       for ( var n = tb.length-1; n >= 0 ; --n )
-                                               if ( jQuery.nodeName(tb[n], "tbody") && !tb[n].childNodes.length )
-                                                       tb[n].parentNode.removeChild(tb[n]);
+                                               // String was a bare <thead> or <tfoot>
+                                               wrap[1] == "<table>" && s.indexOf("<tbody") < 0 ?
+                                                       div.childNodes :
+                                                       [];
+
+                                       for ( var i = tbody.length - 1; i >= 0 ; --i )
+                                               if ( jQuery.nodeName( tbody[ i ], "tbody" ) && !tbody[ i ].childNodes.length )
+                                                       tbody[ i ].parentNode.removeChild( tbody[ i ] );
        
                                        // IE completely kills leading whitespace when innerHTML is used        
-                                       if ( /^\s/.test(arg) )  
-                                               div.insertBefore( doc.createTextNode( arg.match(/^\s*/)[0] ), div.firstChild );
+                                       if ( /^\s/.test( elem ) )       
+                                               div.insertBefore( context.createTextNode( elem.match(/^\s*/)[0] ), div.firstChild );
 
                                }
                                
-                               arg = jQuery.makeArray( div.childNodes );
+                               elem = jQuery.makeArray( div.childNodes );
                        }
 
-                       if ( 0 === arg.length && (!jQuery.nodeName(arg, "form") && !jQuery.nodeName(arg, "select")) )
+                       if ( elem.length === 0 && (!jQuery.nodeName( elem, "form" ) && !jQuery.nodeName( elem, "select" )) )
                                return;
 
-                       if ( arg[0] == undefined || jQuery.nodeName(arg, "form") || arg.options )
-                               r.push( arg );
+                       if ( elem[0] == undefined || jQuery.nodeName( elem, "form" ) || elem.options )
+                               ret.push( elem );
+
                        else
-                               r = jQuery.merge( r, arg );
+                               ret = jQuery.merge( ret, elem );
 
                });
 
-               return r;
+               return ret;
        },
        
-       attr: function(elem, name, value){
-               var fix = jQuery.isXMLDoc(elem) ? {} : jQuery.props;
+       attr: function( elem, name, value ) {
+               var fix = jQuery.isXMLDoc( elem ) ?
+                       {} :
+                       jQuery.props;
 
                // Safari mis-reports the default selected property of a hidden option
                // Accessing the parent's selectedIndex property fixes it
@@ -861,25 +986,30 @@ jQuery.extend({
                        elem.parentNode.selectedIndex;
                
                // 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]];
+               if ( fix[ name ] ) {
+                       if ( value != undefined )
+                               elem[ fix[ name ] ] = value;
+
+                       return elem[ fix[ name ] ];
+
                } else if ( jQuery.browser.msie && name == "style" )
                        return jQuery.attr( elem.style, "cssText", value );
 
-               else if ( value == undefined && jQuery.browser.msie && jQuery.nodeName(elem, "form") && (name == "action" || name == "method") )
-                       return elem.getAttributeNode(name).nodeValue;
+               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 ) {
-                               if ( name == "type" && jQuery.nodeName(elem,"input") && elem.parentNode )
+                               // We can't allow the type property to be changed (since it causes problems in IE)
+                               if ( name == "type" && jQuery.nodeName( elem, "input" ) && elem.parentNode )
                                        throw "type property can't be changed";
+
                                elem.setAttribute( name, value );
                        }
 
-                       if ( jQuery.browser.msie && /href|src/.test(name) && !jQuery.isXMLDoc(elem) ) 
+                       if ( jQuery.browser.msie && /href|src/.test( name ) && !jQuery.isXMLDoc( elem ) ) 
                                return elem.getAttribute( name, 2 );
 
                        return elem.getAttribute( name );
@@ -894,115 +1024,124 @@ jQuery.extend({
                                        elem.zoom = 1; 
        
                                        // Set the alpha filter to set the opacity
-                                       elem.filter = (elem.filter || "").replace(/alpha\([^)]*\)/,"") +
-                                               (parseFloat(value).toString() == "NaN" ? "" : "alpha(opacity=" + value * 100 + ")");
+                                       elem.filter = (elem.filter || "").replace( /alpha\([^)]*\)/, "" ) +
+                                               (parseFloat( value ).toString() == "NaN" ? "" : "alpha(opacity=" + value * 100 + ")");
                                }
        
                                return elem.filter ? 
-                                       (parseFloat( elem.filter.match(/opacity=([^)]*)/)[1] ) / 100).toString() : "";
+                                       (parseFloat( elem.filter.match(/opacity=([^)]*)/)[1] ) / 100).toString() :
+                                       "";
                        }
-                       name = name.replace(/-([a-z])/ig,function(z,b){return b.toUpperCase();});
-                       if ( value != undefined ) elem[name] = value;
-                       return elem[name];
+
+                       name = name.replace(/-([a-z])/ig, function(all, letter){
+                               return letter.toUpperCase();
+                       });
+
+                       if ( value != undefined )
+                               elem[ name ] = value;
+
+                       return elem[ name ];
                }
        },
        
-       trim: function(t){
-               return (t||"").replace(/^\s+|\s+$/g, "");
+       trim: function( text ) {
+               return (text || "").replace( /^\s+|\s+$/g, "" );
        },
 
-       makeArray: function( a ) {
-               var r = [];
+       makeArray: function( array ) {
+               var ret = [];
 
                // Need to use typeof to fight Safari childNodes crashes
-               if ( typeof a != "array" )
-                       for ( var i = 0, al = a.length; i < al; i++ )
-                               r.push( a[i] );
+               if ( typeof array != "array" )
+                       for ( var i = 0, length = array.length; i < length; i++ )
+                               ret.push( array[ i ] );
                else
-                       r = a.slice( 0 );
+                       ret = array.slice( 0 );
 
-               return r;
+               return ret;
        },
 
-       inArray: function( b, a ) {
-               for ( var i = 0, al = a.length; i < al; i++ )
-                       if ( a[i] == b )
+       inArray: function( elem, array ) {
+               for ( var i = 0, length = array.length; i < length; i++ )
+                       if ( array[ i ] == elem )
                                return i;
+
                return -1;
        },
 
-       merge: function(first, second) {
+       merge: function( first, second ) {
                // We have to loop this way because IE & Opera overwrite the length
                // expando of getElementsByTagName
 
                // Also, we need to make sure that the correct elements are being returned
                // (IE returns comment nodes in a '*' query)
                if ( jQuery.browser.msie ) {
-                       for ( var i = 0; second[i]; i++ )
-                               if ( second[i].nodeType != 8 )
-                                       first.push(second[i]);
+                       for ( var i = 0; second[ i ]; i++ )
+                               if ( second[ i ].nodeType != 8 )
+                                       first.push( second[ i ] );
+
                } else
-                       for ( var i = 0; second[i]; i++ )
-                               first.push(second[i]);
+                       for ( var i = 0; second[ i ]; i++ )
+                               first.push( second[ i ] );
 
                return first;
        },
 
-       unique: function(first) {
-               var r = [], done = {};
+       unique: function( array ) {
+               var ret = [], done = {};
 
                try {
-                       for ( var i = 0, fl = first.length; i < fl; i++ ) {
-                               var id = jQuery.data(first[i]);
-                               if ( !done[id] ) {
-                                       done[id] = true;
-                                       r.push(first[i]);
+
+                       for ( var i = 0, length = array.length; i < length; i++ ) {
+                               var id = jQuery.data( array[ i ] );
+
+                               if ( !done[ id ] ) {
+                                       done[ id ] = true;
+                                       ret.push( array[ i ] );
                                }
                        }
-               } catch(e) {
-                       r = first;
+
+               } catch( e ) {
+                       ret = array;
                }
 
-               return r;
+               return ret;
        },
 
-       grep: function(elems, fn, inv) {
+       grep: function( elems, callback, inv ) {
                // If a string is passed in for the function, make a function
                // for it (a handy shortcut)
-               if ( typeof fn == "string" )
-                       fn = eval("false||function(a,i){return " + fn + "}");
+               if ( typeof callback == "string" )
+                       callback = eval("false||function(a,i){return " + callback + "}");
 
-               var result = [];
+               var ret = [];
 
                // Go through the array, only saving the items
                // that pass the validator function
-               for ( var i = 0, el = elems.length; i < el; i++ )
-                       if ( !inv && fn(elems[i],i) || inv && !fn(elems[i],i) )
-                               result.push( elems[i] );
+               for ( var i = 0, length = elems.length; i < length; i++ )
+                       if ( !inv && callback( elems[ i ], i ) || inv && !callback( elems[ i ], i ) )
+                               ret.push( elems[ i ] );
 
-               return result;
+               return ret;
        },
 
-       map: function(elems, fn) {
-               // If a string is passed in for the function, make a function
-               // for it (a handy shortcut)
-               if ( typeof fn == "string" )
-                       fn = eval("false||function(a){return " + fn + "}");
-
-               var result = [];
+       map: function( elems, callback ) {
+               var ret = [];
 
                // Go through the array, translating each of the items to their
                // new value (or values).
-               for ( var i = 0, el = elems.length; i < el; i++ ) {
-                       var val = fn(elems[i],i);
+               for ( var i = 0, length = elems.length; i < length; i++ ) {
+                       var value = callback( elems[ i ], i );
 
-                       if ( val !== null && val != undefined ) {
-                               if ( val.constructor != Array ) val = [val];
-                               result = result.concat( val );
+                       if ( value !== null && value != undefined ) {
+                               if ( value.constructor != Array )
+                                       value = [ value ];
+
+                               ret = ret.concat( value );
                        }
                }
 
-               return result;
+               return ret;
        }
 });
 
@@ -1010,21 +1149,21 @@ var userAgent = navigator.userAgent.toLowerCase();
 
 // Figure out what browser is being used
 jQuery.browser = {
-       version: (userAgent.match(/.+(?:rv|it|ra|ie)[\/: ]([\d.]+)/) || [])[1],
-       safari: /webkit/.test(userAgent),
-       opera: /opera/.test(userAgent),
-       msie: /msie/.test(userAgent) && !/opera/.test(userAgent),
-       mozilla: /mozilla/.test(userAgent) && !/(compatible|webkit)/.test(userAgent)
+       version: (userAgent.match( /.+(?:rv|it|ra|ie)[\/: ]([\d.]+)/ ) || [])[1],
+       safari: /webkit/.test( userAgent ),
+       opera: /opera/.test( userAgent ),
+       msie: /msie/.test( userAgent ) && !/opera/.test( userAgent ),
+       mozilla: /mozilla/.test( userAgent ) && !/(compatible|webkit)/.test( userAgent )
 };
 
-var styleFloat = jQuery.browser.msie ? "styleFloat" : "cssFloat";
+var styleFloat = jQuery.browser.msie ?
+       "styleFloat" :
+       "cssFloat";
        
 jQuery.extend({
        // Check to see if the W3C box model is being used
        boxModel: !jQuery.browser.msie || document.compatMode == "CSS1Compat",
        
-       styleFloat: jQuery.browser.msie ? "styleFloat" : "cssFloat",
-       
        props: {
                "for": "htmlFor",
                "class": "className",
@@ -1044,21 +1183,25 @@ jQuery.extend({
 });
 
 jQuery.each({
-       parent: "a.parentNode",
-       parents: "jQuery.dir(a,'parentNode')",
-       next: "jQuery.nth(a,2,'nextSibling')",
-       prev: "jQuery.nth(a,2,'previousSibling')",
-       nextAll: "jQuery.dir(a,'nextSibling')",
-       prevAll: "jQuery.dir(a,'previousSibling')",
-       siblings: "jQuery.sibling(a.parentNode.firstChild,a)",
-       children: "jQuery.sibling(a.firstChild)",
-       contents: "jQuery.nodeName(a,'iframe')?a.contentDocument||a.contentWindow.document:jQuery.makeArray(a.childNodes)"
-}, function(i,n){
-       jQuery.fn[ i ] = function(a) {
-               var ret = jQuery.map(this,n);
-               if ( a && typeof a == "string" )
-                       ret = jQuery.multiFilter(a,ret);
-               return this.pushStack( jQuery.unique(ret) );
+       parent: "elem.parentNode",
+       parents: "jQuery.dir(elem,'parentNode')",
+       next: "jQuery.nth(elem,2,'nextSibling')",
+       prev: "jQuery.nth(elem,2,'previousSibling')",
+       nextAll: "jQuery.dir(elem,'nextSibling')",
+       prevAll: "jQuery.dir(elem,'previousSibling')",
+       siblings: "jQuery.sibling(elem.parentNode.firstChild,elem)",
+       children: "jQuery.sibling(elem.firstChild)",
+       contents: "jQuery.nodeName(elem,'iframe')?elem.contentDocument||elem.contentWindow.document:jQuery.makeArray(elem.childNodes)"
+}, function(name, fn){
+       fn = eval("false||function(elem){return " + fn + "}");
+
+       jQuery.fn[ name ] = function( selector ) {
+               var ret = jQuery.map( this, fn );
+
+               if ( selector && typeof selector == "string" )
+                       ret = jQuery.multiFilter( selector, ret );
+
+               return this.pushStack( jQuery.unique( ret ) );
        };
 });
 
@@ -1068,74 +1211,83 @@ jQuery.each({
        insertBefore: "before",
        insertAfter: "after",
        replaceAll: "replaceWith"
-}, function(i,n){
-       jQuery.fn[ i ] = function(){
-               var a = arguments;
+}, function(name, original){
+       jQuery.fn[ name ] = function() {
+               var args = arguments;
+
                return this.each(function(){
-                       for ( var j = 0, al = a.length; j < al; j++ )
-                               jQuery(a[j])[n]( this );
+                       for ( var i = 0, length = args.length; i < length; i++ )
+                               jQuery( args[ i ] )[ original ]( this );
                });
        };
 });
 
-jQuery.each( {
-       removeAttr: function( key ) {
-               jQuery.attr( this, key, "" );
-               this.removeAttribute( key );
+jQuery.each({
+       removeAttr: function( name ) {
+               jQuery.attr( this, name, "" );
+               this.removeAttribute( name );
        },
-       addClass: function(c){
-               jQuery.className.add(this,c);
+
+       addClass: function( classNames ) {
+               jQuery.className.add( this, classNames );
        },
-       removeClass: function(c){
-               jQuery.className.remove(this,c);
+
+       removeClass: function( classNames ) {
+               jQuery.className.remove( this, classNames );
        },
-       toggleClass: function( c ){
-               jQuery.className[ jQuery.className.has(this,c) ? "remove" : "add" ](this, c);
+
+       toggleClass: function( classNames ) {
+               jQuery.className[ jQuery.className.has( this, classNames ) ? "remove" : "add" ]( this, classNames );
        },
-       remove: function(a){
-               if ( !a || jQuery.filter( a, [this] ).r.length ) {
+
+       remove: function( selector ) {
+               if ( !selector || jQuery.filter( selector, [ this ] ).r.length ) {
                        jQuery.removeData( this );
                        this.parentNode.removeChild( this );
                }
        },
+
        empty: function() {
                // Clean up the cache
-               jQuery("*", this).each(function(){ jQuery.removeData(this); });
+               jQuery( "*", this ).each(function(){
+                       jQuery.removeData(this);
+               });
 
                while ( this.firstChild )
                        this.removeChild( this.firstChild );
        }
-}, function(i,n){
-       jQuery.fn[ i ] = function() {
-               return this.each( n, arguments );
+}, function(name, fn){
+       jQuery.fn[ name ] = function(){
+               return this.each( fn, arguments );
        };
 });
 
-jQuery.each( [ "Height", "Width" ], function(i,name){
-       var n = name.toLowerCase();
+jQuery.each([ "Height", "Width" ], function(i, name){
+       var type = name.toLowerCase();
        
-       jQuery.fn[ n ] = function(h) {
+       jQuery.fn[ type ] = function( size ) {
                // Get window width or height
                return this[0] == window ?
                        // Opera reports document.body.client[Width/Height] properly in both quirks and standards
-                       jQuery.browser.opera && document.body["client" + name] || 
+                       jQuery.browser.opera && document.body[ "client" + name ] || 
                        
                        // Safari reports inner[Width/Height] just fine (Mozilla and Opera include scroll bar widths)
-                       jQuery.browser.safari && self["inner" + name] ||
+                       jQuery.browser.safari && self[ "inner" + name ] ||
                        
                        // Everyone else use document.documentElement or document.body depending on Quirks vs Standards mode
-                       document.compatMode == "CSS1Compat" && document.documentElement["client" + name] || document.body["client" + name] :
+                       document.compatMode == "CSS1Compat" && document.documentElement[ "client" + name ] || document.body[ "client" + name ] :
                
-               // Get document width or height
+                       // Get document width or height
                        this[0] == document ?
                                // Either scroll[Width/Height] or offset[Width/Height], whichever is greater (Mozilla reports scrollWidth the same as offsetWidth)
-                               Math.max( document.body["scroll" + name], document.body["offset" + name] ) :
+                               Math.max( document.body[ "scroll" + name ], document.body[ "offset" + name ] ) :
         
-               // Get or set width or height on the element
-                               h == undefined ?
+                               // Get or set width or height on the element
+                               size == undefined ?
                                        // Get width or height on the element
-                                       ( this.length ? jQuery.css( this[0], n ) : null ) :
+                                       (this.length ? jQuery.css( this[0], type ) : null) :
+
                                        // Set the width or height on the element (default to pixels if value is unitless)
-                                       this.css( n, h.constructor == String ? h : h + "px" );
+                                       this.css( type, size.constructor == String ? size : size + "px" );
        };
 });