diff options
author | Michał Z. Gołębiowski <michal.golebiowski@laboratorium.ee> | 2013-03-12 21:06:25 -0400 |
---|---|---|
committer | Timmy Willison <timmywillisn@gmail.com> | 2013-03-12 21:06:25 -0400 |
commit | 79992d752393d80822a4f8be2b7bec3ecdd60f6a (patch) | |
tree | 2c97c4bda379e70501b15f106f5efa241c220fe3 /src | |
parent | ae3524642c54ec4df09733de03c169f4fd43ac34 (diff) | |
download | jquery-79992d752393d80822a4f8be2b7bec3ecdd60f6a.tar.gz jquery-79992d752393d80822a4f8be2b7bec3ecdd60f6a.zip |
Edited comments and minor adjustments. Fixes #13553. Closes gh-1195.
Diffstat (limited to 'src')
-rw-r--r-- | src/ajax.js | 9 | ||||
-rw-r--r-- | src/dimensions.js | 4 | ||||
-rw-r--r-- | src/effects.js | 4 | ||||
-rw-r--r-- | src/event.js | 2 |
4 files changed, 10 insertions, 9 deletions
diff --git a/src/ajax.js b/src/ajax.js index f12b231a8..3ceb82c6d 100644 --- a/src/ajax.js +++ b/src/ajax.js @@ -2,7 +2,7 @@ var // Document location ajaxLocParts, ajaxLocation, - + ajax_nonce = jQuery.now(), ajax_rquery = /\?/, @@ -140,7 +140,7 @@ jQuery.fn.load = function( url, params, callback ) { off = url.indexOf(" "); if ( off >= 0 ) { - selector = url.slice( off, url.length ); + selector = url.slice( off ); url = url.slice( 0, off ); } @@ -427,10 +427,11 @@ jQuery.extend({ jqXHR.error = jqXHR.fail; // Remove hash character (#7531: and string promotion) - // Add protocol if not provided (#5866: IE7 issue with protocol-less urls) + // Add protocol if not provided (prefilters might expect it) // Handle falsy url in the settings object (#10093: consistency with old signature) // We also use the url parameter if available - s.url = ( ( url || s.url || ajaxLocation ) + "" ).replace( rhash, "" ).replace( rprotocol, ajaxLocParts[ 1 ] + "//" ); + s.url = ( ( url || s.url || ajaxLocation ) + "" ).replace( rhash, "" ) + .replace( rprotocol, ajaxLocParts[ 1 ] + "//" ); // Alias method option to type as per ticket #12004 s.type = options.method || options.type || s.method || s.type; diff --git a/src/dimensions.js b/src/dimensions.js index 9e4d5729d..ae59fb05d 100644 --- a/src/dimensions.js +++ b/src/dimensions.js @@ -20,8 +20,8 @@ jQuery.each( { Height: "height", Width: "width" }, function( name, type ) { if ( elem.nodeType === 9 ) { doc = elem.documentElement; - // Either scroll[Width/Height] or offset[Width/Height] or client[Width/Height], whichever is greatest - // unfortunately, this causes bug #3838 in IE6/8 only, but there is currently no good, small way to fix it. + // Either scroll[Width/Height] or offset[Width/Height] or client[Width/Height], + // whichever is greatest return Math.max( elem.body[ "scroll" + name ], doc[ "scroll" + name ], elem.body[ "offset" + name ], doc[ "offset" + name ], diff --git a/src/effects.js b/src/effects.js index 764895200..c0f558aec 100644 --- a/src/effects.js +++ b/src/effects.js @@ -440,8 +440,8 @@ Tween.propHooks = { } }; -// Remove in 2.0 - this supports IE8's panic based approach -// to setting things on disconnected nodes +// Support: IE9 +// Panic based approach to setting things on disconnected nodes Tween.propHooks.scrollTop = Tween.propHooks.scrollLeft = { set: function( tween ) { diff --git a/src/event.js b/src/event.js index c9de8e617..d784e1c66 100644 --- a/src/event.js +++ b/src/event.js @@ -66,7 +66,7 @@ jQuery.event = { tmp = rtypenamespace.exec( types[t] ) || []; type = origType = tmp[1]; namespaces = ( tmp[2] || "" ).split( "." ).sort(); - + // There *must* be a type, no attaching namespace-only handlers if ( !type ) { continue; |