library-mirrors/jquery
1
0
Fork 0
mirror of https://github.com/jquery/jquery.git synced 2025-01-10 18:24:24 +00:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

Css: Revert 24e587929f

Browse Source 
The workaround to be able to change !important styles broke the browser
keeping the old CSS value if the new one was rejected. Patching it would
involve a significant perf hit (~33%) so the initial patch needs to be
reverted instead.

Tests by m_gol & gibson042.

Fixes #14836
Closes gh-1532
This commit is contained in:
Michał Gołębiowski 2014-03-10 00:59:14 +01:00
parent b19d4d3450
commit 2c180ef938
2 changed files with 20 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/css.js
View File

@ -286,9 +286,6 @@ jQuery.extend({
// If a hook was provided, use that value, otherwise just set the specified value
if ( !hooks || !("set" in hooks) || (value = hooks.set( elem, value, extra )) !== undefined ) {
// Support: Chrome, Safari
// Setting style to blank string required to delete "style: x !important;"
style[ name ] = "";
style[ name ] = value;
}

25
test/unit/css.js
View File

@ -204,7 +204,7 @@ test( "css() explicit and relative values", 29, function() {
});
test("css(String, Object)", function() {
expect( 19 );
expect( 20 );
var j, div, display, ret, success;
jQuery("#nothiddendiv").css("top", "-1em");
@ -239,7 +239,6 @@ test("css(String, Object)", function() {
equal( ret, div, "Make sure setting undefined returns the original set." );
equal( div.css("display"), display, "Make sure that the display wasn't changed." );
// Test for Bug #5509
success = true;
try {
jQuery( "#foo" ).css( "backgroundColor", "rgba(0, 0, 0, 0.1)" );
@ -247,7 +246,11 @@ test("css(String, Object)", function() {
catch (e) {
success = false;
}
ok( success, "Setting RGBA values does not throw Error" );
ok( success, "Setting RGBA values does not throw Error (#5509)" );
jQuery( "#foo" ).css( "font", "7px/21px sans-serif" );
strictEqual( jQuery( "#foo" ).css( "line-height" ), "21px",
"Set font shorthand property (#14759)" );
});
test( "css(Array)", function() {
@ -923,10 +926,20 @@ test( ":visible/:hidden selectors", function() {
t( "Is Hidden", "#form input:hidden", ["hidden1","hidden2"] );
});
test( "Override !important when changing styles (#14394)", function() {
test( "Keep the last style if the new one isn't recognized by the browser (#14836)", function() {
expect( 2 );
var el;
el = jQuery( "<div></div>" ).css( "color", "black" ).css( "color", "fake value" );
equal( el.css( "color" ), "black", "The old style is kept when setting an unrecognized value" );
el = jQuery( "<div></div>" ).css( "color", "black" ).css( "color", " " );
equal( el.css( "color" ), "black", "The old style is kept when setting to a space" );
});
test( "Reset the style if set to an empty string", function() {
expect( 1 );
var el = jQuery( "<div style='display: block !important;'></div>" ).css( "display", "none" );
equal( el.css( "display" ), "none", "New style replaced !important" );
var el = jQuery( "<div></div>" ).css( "color", "black" ).css( "color", "" );
equal( el.css( "color" ), "", "The style can be reset by setting to an empty string" );
});
asyncTest( "Clearing a Cloned Element's Style Shouldn't Clear the Original Element's Style (#8908)", 24, function() {