]> source.dussan.org Git - jquery.git/commitdiff
Fixed #1438 where a filter could be set in IE but not have opacity in it. The JS...
authorDavid Serduke <davidserduke@gmail.com>
Tue, 4 Dec 2007 18:15:27 +0000 (18:15 +0000)
committerDavid Serduke <davidserduke@gmail.com>
Tue, 4 Dec 2007 18:15:27 +0000 (18:15 +0000)
src/core.js
test/unit/core.js

index bd6d929d5a4277dc96690efbb19a1b6a75de56b6..5a842d5e7ce94719799db96e928fc10851dd878b 100644 (file)
@@ -1064,7 +1064,7 @@ jQuery.extend({
                                                (parseFloat( value ).toString() == "NaN" ? "" : "alpha(opacity=" + value * 100 + ")");
                                }
        
-                               return elem.filter 
+                               return elem.filter && elem.filter.indexOf("opacity=") >= 0 ?
                                        (parseFloat( elem.filter.match(/opacity=([^)]*)/)[1] ) / 100).toString() :
                                        "";
                        }
index a1ebb1ba428d111a38e93a5e7576c98f7d4c00ea..f40f3b02ec8c58ddd302bdd891048091b49f6127 100644 (file)
@@ -380,7 +380,7 @@ test("css(String|Hash)", function() {
 });
 
 test("css(String, Object)", function() {
-       expect(18);
+       expect(19);
        ok( $('#foo').is(':visible'), 'Modifying CSS display: Assert element is visible');
        $('#foo').css('display', 'none');
        ok( !$('#foo').is(':visible'), 'Modified CSS display: Assert element is hidden');
@@ -404,6 +404,11 @@ test("css(String, Object)", function() {
        });
        $('#foo').css('opacity', '');
        ok( $('#foo').css('opacity') == '1', "Assert opacity is 1 when set to an empty String" );
+       // for #1438, IE throws JS error when filter exists but doesn't have opacity in it
+       if (jQuery.browser.msie) {
+               $('#foo').css("filter", "progid:DXImageTransform.Microsoft.Chroma(color='red');");
+       }
+       equals( $('#foo').css('opacity'), '1', "Assert opacity is 1 when a different filter is set in IE, #1438" );
 });
 
 test("jQuery.css(elem, 'height') doesn't clear radio buttons (bug #1095)", function () {