]> source.dussan.org Git - jquery.git/commitdiff
Test: enable a spec testing CSS whitespace preserving in Edge 17
authorMichał Gołębiowski-Owczarek <m.goleb@gmail.com>
Mon, 18 Jun 2018 16:48:15 +0000 (18:48 +0200)
committerGitHub <noreply@github.com>
Mon, 18 Jun 2018 16:48:15 +0000 (18:48 +0200)
In Edge 14-16 setting a style property to a whitespace-only value resets it to
the default, forcing us to skip a relevant CSS test in Edge. Now that Edge 17
has fixed the issue we can re-enable this test there.

Ref gh-3204
Closes gh-4101

test/unit/css.js

index 96a231726d01d6d0fd31ccd52ea8e6266b63cd57..a21a58b9b527d88d794ed98c0b8f8e5c55a368ec 100644 (file)
@@ -1348,11 +1348,11 @@ QUnit.test( "Keep the last style if the new one isn't recognized by the browser
        assert.equal( el.css( "position" ), "absolute", "The old style is kept when setting an unrecognized value" );
 } );
 
-// Support: Edge 14 - 15
+// Support: Edge 14 - 16 only
 // Edge collapses whitespace-only values when setting a style property and
 // there is no easy way for us to work around it. Just skip the test there
 // and hope for the better future.
-QUnit[ /\bedge\//i.test( navigator.userAgent ) ? "skip" : "test" ](
+QUnit[ /\bedge\/16\./i.test( navigator.userAgent ) ? "skip" : "test" ](
        "Keep the last style if the new one is a non-empty whitespace (gh-3204)",
        function( assert ) {
        assert.expect( 1 );