github/jquery
1
1
Fork 0
mirror of https://github.com/jquery/jquery.git synced 2026-02-01 15:15:00 -05:00
Code Issues Packages Projects Releases Wiki Activity

Core:CSS:Event: simplification of native method signatures

Browse Source 
* Remove third argument from "addEventListener"

* Remove third argument from "removeEventListener"

* Remove second argument from "getComputedStyle"

Ref gh-2047
Ref 85577a348a
This commit is contained in:
Oleg Gaidarenko
2015-05-02 20:54:11 +03:00
parent 4632e55870
commit 49bce47124
4 changed files with 9 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
src/core/ready.js
View File

@@ -68,8 +68,8 @@ jQuery.extend({
*/
function detach() {
if ( document.addEventListener ) {
document.removeEventListener( "DOMContentLoaded", completed, false );
window.removeEventListener( "load", completed, false );
document.removeEventListener( "DOMContentLoaded", completed );
window.removeEventListener( "load", completed );
// Support: IE<9
} else {
@@ -110,10 +110,10 @@ jQuery.ready.promise = function( obj ) {
// Standards-based browsers support DOMContentLoaded
} else if ( document.addEventListener ) {
// Use the handy event callback
document.addEventListener( "DOMContentLoaded", completed, false );
document.addEventListener( "DOMContentLoaded", completed );
// A fallback to window.onload, that will always work
window.addEventListener( "load", completed, false );
window.addEventListener( "load", completed );
// Support: IE<9
// If IE event model is used

4
src/css/curCSS.js
View File

@@ -16,10 +16,10 @@ if ( window.getComputedStyle ) {
// IE throws on elements created in popups
// FF meanwhile throws on frame elements through "defaultView.getComputedStyle"
if ( elem.ownerDocument.defaultView.opener ) {
return elem.ownerDocument.defaultView.getComputedStyle( elem, null );
return elem.ownerDocument.defaultView.getComputedStyle( elem );
}
return window.getComputedStyle( elem, null );
return window.getComputedStyle( elem );
};
curCSS = function( elem, name, computed ) {

4
src/css/support.js
View File

@@ -107,7 +107,7 @@ define([
// Check for getComputedStyle so that this code is not run in IE<9.
if ( window.getComputedStyle ) {
divStyle = window.getComputedStyle( div, null );
divStyle = window.getComputedStyle( div );
pixelPositionVal = ( divStyle || {} ).top !== "1%";
boxSizingReliableVal = ( divStyle || { height: "4px" } ).height === "4px";
pixelMarginRightVal = ( divStyle || { marginRight: "4px" } ).marginRight === "4px";
@@ -128,7 +128,7 @@ define([
div.style.width = "1px";
reliableMarginRightVal =
!parseFloat( ( window.getComputedStyle( contents, null ) || {} ).marginRight );
!parseFloat( ( window.getComputedStyle( contents ) || {} ).marginRight );
div.removeChild( contents );
}

2
src/event.js
View File

@@ -677,7 +677,7 @@ jQuery.removeEvent = document.removeEventListener ?
// This "if" is needed for plain objects
if ( elem.removeEventListener ) {
elem.removeEventListener( type, handle, false );
elem.removeEventListener( type, handle );
}
} :
function( elem, type, handle ) {