Changed $.sub internals to match sub naming, added some more spacing in some areas for readability

This commit is contained in:
Timmy Willison 2011-02-06 19:34:57 -05:00
parent d6fbbe1080
commit 82626799ca
3 changed files with 27 additions and 29 deletions

View File

@ -441,7 +441,7 @@ jQuery.extend({
} else if ( document.attachEvent ) { } else if ( document.attachEvent ) {
// ensure firing before onload, // ensure firing before onload,
// maybe late but safe also for iframes // maybe late but safe also for iframes
document.attachEvent("onreadystatechange", DOMContentLoaded); document.attachEvent( "onreadystatechange", DOMContentLoaded );
// A fallback to window.onload, that will always work // A fallback to window.onload, that will always work
window.attachEvent( "onload", jQuery.ready ); window.attachEvent( "onload", jQuery.ready );
@ -598,7 +598,7 @@ jQuery.extend({
each: function( object, callback, args ) { each: function( object, callback, args ) {
var name, i = 0, var name, i = 0,
length = object.length, length = object.length,
isObj = length === undefined || jQuery.isFunction(object); isObj = length === undefined || jQuery.isFunction( object );
if ( args ) { if ( args ) {
if ( isObj ) { if ( isObj ) {
@ -625,7 +625,7 @@ jQuery.extend({
} }
} else { } else {
for ( var value = object[0]; for ( var value = object[0];
i < length && callback.call( value, i, value ) !== false; value = object[++i] ) {} i < length && callback.call( value, i, value ) !== false; value = object[ ++i ] ) {}
} }
} }
@ -656,7 +656,7 @@ jQuery.extend({
// The extra typeof function check is to prevent crashes // The extra typeof function check is to prevent crashes
// in Safari 2 (See: #3039) // in Safari 2 (See: #3039)
// Tweaked logic slightly to handle Blackberry 4.7 RegExp issues #6930 // Tweaked logic slightly to handle Blackberry 4.7 RegExp issues #6930
var type = jQuery.type(array); var type = jQuery.type( array );
if ( array.length == null || type === "string" || type === "function" || type === "regexp" || jQuery.isWindow( array ) ) { if ( array.length == null || type === "string" || type === "function" || type === "regexp" || jQuery.isWindow( array ) ) {
push.call( ret, array ); push.call( ret, array );
@ -982,24 +982,24 @@ jQuery.extend({
}, },
sub: function() { sub: function() {
function jQuerySubclass( selector, context ) { function jQuerySub( selector, context ) {
return new jQuerySubclass.fn.init( selector, context ); return new jQuerySub.fn.init( selector, context );
} }
jQuery.extend( true, jQuerySubclass, this ); jQuery.extend( true, jQuerySub, this );
jQuerySubclass.superclass = this; jQuerySub.superclass = this;
jQuerySubclass.fn = jQuerySubclass.prototype = this(); jQuerySub.fn = jQuerySub.prototype = this();
jQuerySubclass.fn.constructor = jQuerySubclass; jQuerySub.fn.constructor = jQuerySub;
jQuerySubclass.subclass = this.subclass; jQuerySub.sub = this.sub;
jQuerySubclass.fn.init = function init( selector, context ) { jQuerySub.fn.init = function init( selector, context ) {
if ( context && context instanceof jQuery && !(context instanceof jQuerySubclass) ) { if ( context && context instanceof jQuery && !(context instanceof jQuerySub) ) {
context = jQuerySubclass(context); context = jQuerySub( context );
} }
return jQuery.fn.init.call( this, selector, context, rootjQuerySubclass ); return jQuery.fn.init.call( this, selector, context, rootjQuerySub );
}; };
jQuerySubclass.fn.init.prototype = jQuerySubclass.fn; jQuerySub.fn.init.prototype = jQuerySub.fn;
var rootjQuerySubclass = jQuerySubclass(document); var rootjQuerySub = jQuerySub(document);
return jQuerySubclass; return jQuerySub;
}, },
browser: {} browser: {}

View File

@ -215,8 +215,8 @@ if ( !jQuery.support.opacity ) {
jQuery.cssHooks.opacity = { jQuery.cssHooks.opacity = {
get: function( elem, computed ) { get: function( elem, computed ) {
// IE uses filters for opacity // IE uses filters for opacity
return ropacity.test((computed && elem.currentStyle ? elem.currentStyle.filter : elem.style.filter) || "") ? return ropacity.test( (computed && elem.currentStyle ? elem.currentStyle.filter : elem.style.filter) || "" ) ?
(parseFloat(RegExp.$1) / 100) + "" : ( parseFloat( RegExp.$1 ) / 100 ) + "" :
computed ? "1" : ""; computed ? "1" : "";
}, },
@ -228,13 +228,13 @@ if ( !jQuery.support.opacity ) {
style.zoom = 1; style.zoom = 1;
// Set the alpha filter to set the opacity // Set the alpha filter to set the opacity
var opacity = jQuery.isNaN(value) ? var opacity = jQuery.isNaN( value ) ?
"" : "" :
"alpha(opacity=" + value * 100 + ")", "alpha(opacity=" + value * 100 + ")",
filter = style.filter || ""; filter = style.filter || "";
style.filter = ralpha.test(filter) ? style.filter = ralpha.test( filter ) ?
filter.replace(ralpha, opacity) : filter.replace( ralpha, opacity ) :
style.filter + ' ' + opacity; style.filter + ' ' + opacity;
} }
}; };

View File

@ -294,11 +294,9 @@ jQuery.event = {
} }
}, },
// bubbling is internal trigger: function( event, data, elem, bubbling /* For Internal Use Only */ ) {
trigger: function( event, data, elem /*, bubbling */ ) {
// Event object or event type // Event object or event type
var type = event.type || event, var type = event.type || event;
bubbling = arguments[3];
if ( !bubbling ) { if ( !bubbling ) {
event = typeof event === "object" ? event = typeof event === "object" ?