From: Colin Snover <colin@alpha.zetafleet.com>
Date: Sat, 9 Oct 2010 14:42:01 +0000 (-0400)
Subject: Prevent IE from throwing errors when setting RGBA values. Fixes #5509.
X-Git-Tag: 1.4.3rc1~23
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=2ca36598954759c5b5dce569a39c52b981ed4ab2;p=jquery.git

Prevent IE from throwing errors when setting RGBA values. Fixes #5509.
---

diff --git a/src/css.js b/src/css.js
index 4bf818e5a..99cb73581 100644
--- a/src/css.js
+++ b/src/css.js
@@ -88,7 +88,11 @@ 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 )) !== undefined ) {
-				style[ name ] = value;
+				// Wrapped to prevent IE from throwing errors when 'invalid' values are provided
+				// Fixes bug #5509
+				try {
+					style[ name ] = value;
+				} catch(e) {}
 			}
 
 		} else {
diff --git a/test/unit/css.js b/test/unit/css.js
index 468f76386..26dd7047d 100644
--- a/test/unit/css.js
+++ b/test/unit/css.js
@@ -64,7 +64,7 @@ test("css(String|Hash)", function() {
 });
 
 test("css(String, Object)", function() {
-	expect(21);
+	expect(22);
 
 	ok( jQuery('#nothiddendiv').is(':visible'), 'Modifying CSS display: Assert element is visible');
 	jQuery('#nothiddendiv').css("display", 'none');
@@ -104,6 +104,16 @@ test("css(String, Object)", function() {
 
 	equals( ret, div, "Make sure setting undefined returns the original set." );
 	equals( div.css("display"), display, "Make sure that the display wasn't changed." );
+
+	// Test for Bug #5509
+	var success = true;
+	try {
+		jQuery('#foo').css("backgroundColor", "rgba(0, 0, 0, 0.1)");
+	}
+	catch (e) {
+		success = false;
+	}
+	ok( success, "Setting RGBA values does not throw Error" );
 });
 
 if(jQuery.browser.msie) {