library-mirrors/jquery
1
0
Fork 0
mirror of https://github.com/jquery/jquery.git synced 2024-11-23 02:54:22 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Tests: Disable a whitespace-setting test in Edge 14

Browse Source 
Working around this problem would require us to skip setting whitespace-only
values except when they're valid which would be very fragile. Another option
would be to set the value and see if it succeeded and then react to that.

We've tried something like that in the past to be able to overwrite !important
styles (see 24e5879) but it broke the CSS cascade (see
https://bugs.jquery.com/ticket/14836#comment:5) and was triggering
MutationObserver callbacks too often so it was reverted in PR gh-1532.

Ref gh-3204
Ref gh-1532
This commit is contained in:
Michał Gołębiowski 2016-09-12 18:36:48 +02:00
parent 52e24471c8
commit 3f5f5433d5
1 changed files with 10 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
test/unit/css.js
View File

@ -1322,7 +1322,16 @@ QUnit.test( "Keep the last style if the new one isn't recognized by the browser
el = jQuery( "<div></div>" ).css( "position", "absolute" ).css( "position", "fake value" );
assert.equal( el.css( "position" ), "absolute", "The old style is kept when setting an unrecognized value" );
el = jQuery( "<div></div>" ).css( "position", "absolute" ).css( "position", " " );
assert.equal( el.css( "position" ), "absolute", "The old style is kept when setting to a space" );
// Support: Edge 14
// 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.
if ( /edge\//i.test( navigator.userAgent ) ) {
assert.ok( true, "Skipped (Edge 14 handles whitespace-only values incorrectly)" );
} else {
assert.equal( el.css( "position" ), "absolute", "The old style is kept when setting to a space" );
}
} );
QUnit.test( "Reset the style if set to an empty string", function( assert ) {