Permalink
Browse files

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
  • Loading branch information...
mgol committed Jul 27, 2015
1 parent 15f4804 commit 5fce498e4263f11ed4c688e5304999e63c395367
Showing with 4 additions and 3 deletions.
  1. +1 −1 src/event.js
  2. +1 −0 src/offset.js
  3. +1 −1 test/unit/css.js
  4. +1 −1 test/unit/support.js
View
@@ -572,7 +572,7 @@ jQuery.event = {
event.target = originalEvent.srcElement || document;
}
// 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;
View
@@ -215,6 +215,7 @@ jQuery.each( { scrollLeft: "pageXOffset", scrollTop: "pageYOffset" }, function(
};
});
// 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
// getComputedStyle returns percent when specified for top/left/bottom/right
View
@@ -1180,7 +1180,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.
View
@@ -268,7 +268,7 @@ testIframeWithCallback( "Check CSP (https://developer.mozilla.org/en-US/docs/Sec
"style": true,
"submit": true
};
} else if ( /(6|7)\.0(\.\d+|) safari/i.test( userAgent ) ) {
} else if ( /7\.0(\.\d+|) safari/i.test( userAgent ) ) {
expected = {
"ajax": true,
"attributes": true,

0 comments on commit 5fce498

Please sign in to comment.