Skip to content

Commit cf84696

Browse files
mgolmgol
authored
Core: Drop support for IE <11, iOS <11, Firefox <65, Android Browser & PhantomJS
Also, update support comments format to match format described in: jquery/contribute.jquery.org#95 (comment) with the change from: jquery/contribute.jquery.org#95 (comment) (open-ended ranges end with `+`). Fixes gh-3950 Fixes gh-4299 Closes gh-4347
1 parent bde53ed commit cf84696

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

67 files changed

+515
-1431
lines changed

‎.eslintrc-browser.json

+1-4
Original file line numberDiff line numberDiff line change
@@ -3,11 +3,8 @@
33

44
"extends": "jquery",
55

6-
// Support: IE <=9 only, Android <=4.0 only
7-
// The above browsers are failing a lot of tests in the ES5
8-
// test suite at http://test262.ecmascript.org.
96
"parserOptions": {
10-
"ecmaVersion": 3
7+
"ecmaVersion": 5
118
},
129

1310
// The browser env is not enabled on purpose so that code takes

‎Gruntfile.js

+7-36
Original file line numberDiff line numberDiff line change
@@ -14,15 +14,7 @@ module.exports = function( grunt ) {
1414

1515
var fs = require( "fs" ),
1616
gzip = require( "gzip-js" ),
17-
isTravis = process.env.TRAVIS,
18-
oldNode = /^v6\./.test( process.version );
19-
20-
// Support: Node.js <8
21-
// Skip running tasks that dropped support for Node.js 6
22-
// in those Node versions.
23-
function runIfNewNode( task ) {
24-
return oldNode ? "print_old_node_message:" + task : task;
25-
}
17+
isTravis = process.env.TRAVIS;
2618

2719
if ( !grunt.option( "filename" ) ) {
2820
grunt.option( "filename", "jquery.js" );
@@ -123,10 +115,9 @@ module.exports = function( grunt ) {
123115
testswarm: {
124116
tests: [
125117

126-
// A special module with basic tests, meant for
127-
// not fully supported environments like Android 2.3,
128-
// jsdom or PhantomJS. We run it everywhere, though,
129-
// to make sure tests are not broken.
118+
// A special module with basic tests, meant for not fully
119+
// supported environments like jsdom. We run it everywhere,
120+
// though, to make sure tests are not broken.
130121
"basic",
131122

132123
"ajax",
@@ -291,23 +282,13 @@ module.exports = function( grunt ) {
291282
"dist/<%= grunt.option('filename').replace('.js', '.min.map') %>",
292283
report: "min",
293284
output: {
294-
"ascii_only": true,
295-
296-
// Support: Android 4.0 only
297-
// UglifyJS 3 breaks Android 4.0 if this option is not enabled.
298-
// This is in lieu of setting ie8 for all of mangle, compress, and output
299-
"ie8": true
285+
"ascii_only": true
300286
},
301287
banner: "/*! jQuery v<%= pkg.version %> | " +
302288
"(c) JS Foundation and other contributors | jquery.org/license */",
303289
compress: {
304290
"hoist_funs": false,
305-
loops: false,
306-
307-
// Support: IE <11
308-
// typeofs transformation is unsafe for IE9-10
309-
// See https://github.com/mishoo/UglifyJS2/issues/2198
310-
typeofs: false
291+
loops: false
311292
}
312293
}
313294
}
@@ -320,13 +301,6 @@ module.exports = function( grunt ) {
320301
// Integrate jQuery specific tasks
321302
grunt.loadTasks( "build/tasks" );
322303

323-
// Support: Node.js <8
324-
// Print a message on Node.js <8 notifying the task is skipped there.
325-
grunt.registerTask( "print_old_node_message", function() {
326-
var task = [].slice.call( arguments ).join( ":" );
327-
grunt.log.writeln( "Old Node.js detected, running the task \"" + task + "\" skipped..." );
328-
} );
329-
330304
grunt.registerTask( "lint", [
331305
"jsonlint",
332306

@@ -349,10 +323,7 @@ module.exports = function( grunt ) {
349323
grunt.registerTask( "test:fast", "node_smoke_tests" );
350324
grunt.registerTask( "test:slow", [
351325
"promises_aplus_tests",
352-
353-
// Support: Node.js <8
354-
// Karma no longer supports Node.js <8 as it relies on async-await internally.
355-
runIfNewNode( "karma:jsdom" )
326+
"karma:jsdom"
356327
] );
357328

358329
grunt.registerTask( "test", [

‎src/ajax.js

+5-6
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
define( [
22
"./core",
33
"./var/document",
4-
"./var/isFunction",
54
"./var/rnothtmlwhite",
65
"./ajax/var/location",
76
"./ajax/var/nonce",
@@ -12,7 +11,7 @@ define( [
1211
"./event/trigger",
1312
"./deferred",
1413
"./serialize" // jQuery.param
15-
], function( jQuery, document, isFunction, rnothtmlwhite, location, nonce, rquery ) {
14+
], function( jQuery, document, rnothtmlwhite, location, nonce, rquery ) {
1615

1716
"use strict";
1817

@@ -67,7 +66,7 @@ function addToPrefiltersOrTransports( structure ) {
6766
i = 0,
6867
dataTypes = dataTypeExpression.toLowerCase().match( rnothtmlwhite ) || [];
6968

70-
if ( isFunction( func ) ) {
69+
if ( typeof func === "function" ) {
7170

7271
// For each dataType in the dataTypeExpression
7372
while ( ( dataType = dataTypes[ i++ ] ) ) {
@@ -541,13 +540,13 @@ jQuery.extend( {
541540
if ( s.crossDomain == null ) {
542541
urlAnchor = document.createElement( "a" );
543542

544-
// Support: IE <=8 - 11, Edge 12 - 15
543+
// Support: IE <=8 - 11+, Edge 12 - 17 only
545544
// IE throws exception on accessing the href property if url is malformed,
546545
// e.g. http://example.com:80x/
547546
try {
548547
urlAnchor.href = s.url;
549548

550-
// Support: IE <=8 - 11 only
549+
// Support: IE <=8 - 11+
551550
// Anchor's host property isn't correctly set when s.url is relative
552551
urlAnchor.href = urlAnchor.href;
553552
s.crossDomain = originAnchor.protocol + "//" + originAnchor.host !==
@@ -837,7 +836,7 @@ jQuery.each( [ "get", "post" ], function( i, method ) {
837836
jQuery[ method ] = function( url, data, callback, type ) {
838837

839838
// Shift arguments if data argument was omitted
840-
if ( isFunction( data ) ) {
839+
if ( typeof data === "function" ) {
841840
type = type || callback;
842841
callback = data;
843842
data = undefined;

‎src/ajax/jsonp.js

+3-4
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,9 @@
11
define( [
22
"../core",
3-
"../var/isFunction",
43
"./var/nonce",
54
"./var/rquery",
65
"../ajax"
7-
], function( jQuery, isFunction, nonce, rquery ) {
6+
], function( jQuery, nonce, rquery ) {
87

98
"use strict";
109

@@ -37,7 +36,7 @@ jQuery.ajaxPrefilter( "json jsonp", function( s, originalSettings, jqXHR ) {
3736
if ( jsonProp || s.dataTypes[ 0 ] === "jsonp" ) {
3837

3938
// Get callback name, remembering preexisting value associated with it
40-
callbackName = s.jsonpCallback = isFunction( s.jsonpCallback ) ?
39+
callbackName = s.jsonpCallback = typeof s.jsonpCallback === "function" ?
4140
s.jsonpCallback() :
4241
s.jsonpCallback;
4342

@@ -88,7 +87,7 @@ jQuery.ajaxPrefilter( "json jsonp", function( s, originalSettings, jqXHR ) {
8887
}
8988

9089
// Call if it was a function and we have a response
91-
if ( responseContainer && isFunction( overwritten ) ) {
90+
if ( responseContainer && typeof overwritten === "function" ) {
9291
overwritten( responseContainer[ 0 ] );
9392
}
9493

‎src/ajax/load.js

+2-3
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,12 @@
11
define( [
22
"../core",
33
"../core/stripAndCollapse",
4-
"../var/isFunction",
54
"../core/parseHTML",
65
"../ajax",
76
"../traversing",
87
"../manipulation",
98
"../selector"
10-
], function( jQuery, stripAndCollapse, isFunction ) {
9+
], function( jQuery, stripAndCollapse ) {
1110

1211
"use strict";
1312

@@ -25,7 +24,7 @@ jQuery.fn.load = function( url, params, callback ) {
2524
}
2625

2726
// If it's a function
28-
if ( isFunction( params ) ) {
27+
if ( typeof params === "function" ) {
2928

3029
// We assume that it's the callback
3130
callback = params;

‎src/ajax/parseXML.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ jQuery.parseXML = function( data ) {
1111
return null;
1212
}
1313

14-
// Support: IE 9 - 11 only
14+
// Support: IE 9 - 11+
1515
// IE throws on parseFromString with invalid input.
1616
try {
1717
xml = ( new window.DOMParser() ).parseFromString( data, "text/xml" );

0 commit comments

Comments
 (0)