Skip to content

Commit 5fce498

Browse files
committed
Core: Adjust comments & tests after dropping Safari 6 support
Support comments that mentioned only Safari < 7 were checked & updated to account for bugs existing in newer versions as well; Safari 6 support test results were removed. (cherry-picked from 93bee47) Refs gh-2482
1 parent 15f4804 commit 5fce498

File tree

4 files changed

+4
-3
lines changed

4 files changed

+4
-3
lines changed

src/event.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -572,7 +572,7 @@ jQuery.event = {
572572
event.target = originalEvent.srcElement || document;
573573
}
574574

575-
// Support: Safari 6.0+
575+
// Support: Safari 6-8+
576576
// Target should not be a text node (#504, #13143)
577577
if ( event.target.nodeType === 3 ) {
578578
event.target = event.target.parentNode;

src/offset.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -215,6 +215,7 @@ jQuery.each( { scrollLeft: "pageXOffset", scrollTop: "pageYOffset" }, function(
215215
};
216216
});
217217

218+
// Support: Safari<7-8+, Chrome<37-44+
218219
// Add the top/left cssHooks using jQuery.fn.position
219220
// Webkit bug: https://bugs.webkit.org/show_bug.cgi?id=29084
220221
// getComputedStyle returns percent when specified for top/left/bottom/right

test/unit/css.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1180,7 +1180,7 @@ test( "show() after hide() should always set display to initial value (#14750)",
11801180
equal( div.css( "display" ), "list-item", "should get last set display value" );
11811181
});
11821182

1183-
// Support: IE < 11, Safari < 7
1183+
// Support: IE < 11
11841184
// We have to jump through the hoops here in order to test work with "order" CSS property,
11851185
// that some browsers do not support. This test is not, strictly speaking, correct,
11861186
// but it's the best that we can do.

test/unit/support.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -268,7 +268,7 @@ testIframeWithCallback( "Check CSP (https://developer.mozilla.org/en-US/docs/Sec
268268
"style": true,
269269
"submit": true
270270
};
271-
} else if ( /(6|7)\.0(\.\d+|) safari/i.test( userAgent ) ) {
271+
} else if ( /7\.0(\.\d+|) safari/i.test( userAgent ) ) {
272272
expected = {
273273
"ajax": true,
274274
"attributes": true,

0 commit comments

Comments
 (0)