Skip to content

Commit

Permalink
Core: Adjust comments & tests after dropping Safari 6 support
Browse files Browse the repository at this point in the history
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.

Refs gh-2482
  • Loading branch information
mgol committed Jul 27, 2015
1 parent bf48c21 commit 93bee47
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion src/event.js
Original file line number Diff line number Diff line change
Expand Up @@ -593,7 +593,7 @@ jQuery.event = {
event[ prop ] = originalEvent[ prop ];
}

// Support: Safari 6.0+
// Support: Safari 6-8+
// Target should not be a text node (#504, #13143)
if ( event.target.nodeType === 3 ) {
event.target = event.target.parentNode;
Expand Down
2 changes: 1 addition & 1 deletion src/offset.js
Original file line number Diff line number Diff line change
Expand Up @@ -203,7 +203,7 @@ jQuery.each( { scrollLeft: "pageXOffset", scrollTop: "pageYOffset" }, function(
};
});

// Support: Safari<7+, Chrome<37+
// Support: Safari<7-8+, Chrome<37-44+
// Add the top/left cssHooks using jQuery.fn.position
// Webkit bug: https://bugs.webkit.org/show_bug.cgi?id=29084
// Blink bug: https://code.google.com/p/chromium/issues/detail?id=229280
Expand Down
2 changes: 1 addition & 1 deletion test/unit/css.js
Original file line number Diff line number Diff line change
Expand Up @@ -1121,7 +1121,7 @@ test( "show() after hide() should always set display to initial value (#14750)",
equal( div.css( "display" ), "list-item", "should get last set display value" );
});

// Support: IE < 11, Safari < 7
// Support: IE < 11
// We have to jump through the hoops here in order to test work with "order" CSS property,
// that some browsers do not support. This test is not, strictly speaking, correct,
// but it's the best that we can do.
Expand Down
2 changes: 1 addition & 1 deletion test/unit/support.js
Original file line number Diff line number Diff line change
Expand Up @@ -143,7 +143,7 @@ testIframeWithCallback( "Check CSP (https://developer.mozilla.org/en-US/docs/Sec
"radioValue": true,
"reliableMarginRight": true
};
} else if ( /(6|7)\.0(\.\d+|) safari/i.test( userAgent ) ) {
} else if ( /7\.0(\.\d+|) safari/i.test( userAgent ) ) {
expected = {
"ajax": true,
"boxSizingReliable": true,
Expand Down

0 comments on commit 93bee47

Please sign in to comment.