Skip to content
Permalink
Browse files

Fixes #1034 - Check for style.removeAttribute before calling it

Fixes issue in non IE browsers that happen to come down this path
  • Loading branch information...
gnarf committed Aug 6, 2012
1 parent beeab4d commit 07e50933c4293818c5b36d368368656844e4df88
Showing with 2 additions and 1 deletion.
  1. +2 −1 src/css.js
@@ -534,7 +534,8 @@ if ( !jQuery.support.opacity ) {
style.zoom = 1;

// if setting opacity to 1, and no other filters exist - attempt to remove filter attribute #6652
if ( value >= 1 && jQuery.trim( filter.replace( ralpha, "" ) ) === "" ) {
if ( value >= 1 && jQuery.trim( filter.replace( ralpha, "" ) ) === "" &&
style.removeAttribute ) {

// Setting style.filter to null, "" & " " still leave "filter:" in the cssText
// if "filter:" is present at all, clearType is disabled, we want to avoid this

2 comments on commit 07e5093

@gnarf

This comment has been minimized.

Copy link
Member Author

gnarf replied Aug 6, 2012

Correction, fixes #10394

@nepsdotin

This comment has been minimized.

Copy link

nepsdotin replied Mar 14, 2013

ok great

Please sign in to comment.
You can’t perform that action at this time.