Bug Tracker

Opened 10 years ago

Last modified 9 years ago

#6652 closed enhancement

Remove filter:alpha(opacity=100) after animation — at Version 23

Reported by: Avorin Owned by:
Priority: blocker Milestone: 1.6.3
Component: css Version: 1.4.2
Keywords: Internet, Explorer, opacity, 1.7-discuss Cc: alexander.trefz@…
Blocked by: #6841 Blocking:

Description (last modified by john)

Internet Explorer renders some strange things(font, some backgrounds etc.) when filter: alpha(opacity=100) is applied(i.e. if opacity is animated to 1).

I have written a fix for that:

insert at Line 4603

if( style.filter === "alpha(opacity=100)" ) {

style.filter = "";

}

that removes the filter if the value is 100.

Change History (21)

comment:1 Changed 10 years ago by dmethvin

See #3238 and #3502 for background.

comment:2 Changed 10 years ago by snover

#7400 is a duplicate of this ticket.

comment:3 Changed 10 years ago by SlexAxton

Milestone: 1.4.31.4.5
Priority: low
Status: newopen

comment:4 Changed 10 years ago by snover

#3230 is a duplicate of this ticket.

comment:5 Changed 10 years ago by snover

Blocked by: 6841 added

comment:8 Changed 10 years ago by snover

#4779 is a duplicate of this ticket.

comment:9 Changed 10 years ago by snover

#3901 is a duplicate of this ticket.

comment:10 Changed 10 years ago by snover

#5676 is a duplicate of this ticket.

comment:11 Changed 10 years ago by snover

As touched upon in #6841, it is important that the filter only be removed if there is no existing filter in any stylesheets that will cascade and cause the opacity to reset back to where it was before.

comment:12 Changed 10 years ago by jitter

#7932 is a duplicate of this ticket.

comment:13 Changed 9 years ago by john

Let's consider this for 1.7.

comment:14 Changed 9 years ago by Steven <jquery.good@…>

Note that in order to solve some bugs, the filter attribute should be removed completely, instead of set to an empty value.

Here is the jsFiddle containing the complete "plugin" that I include on all my sites to fix a clear type issue in IE7: http://jsfiddle.net/gWFx4/4/ - it removes the filter field if it is either empty or only has alpha(opacity=100)

comment:15 Changed 9 years ago by scottgonzalez

comment:16 Changed 9 years ago by john

Keywords: 1.7-discuss added

Nominating ticket for 1.7 discussion.

comment:17 Changed 9 years ago by Rick Waldron

Description: modified (diff)

+0, Scott proposed a better approach

comment:18 Changed 9 years ago by Rick Waldron

Description: modified (diff)

My comments above may be mistaken, apparently _this_ was on the proposals list but scott also re-proposed it

comment:19 Changed 9 years ago by jaubourg

Description: modified (diff)

-1, No matter what you do in the end, you'll have this strange rendering during the animation anyway, so I see little value in this.

comment:20 Changed 9 years ago by timmywil

+1, If we can figure out a way to do this without presenting bugs, I think it is needed.

comment:21 Changed 9 years ago by paul.irish

Description: modified (diff)

+1, look at all the dupes of this one. people hit this guy often, let's help 'em out

comment:22 Changed 9 years ago by dmethvin

+1, The nay is that stylesheet-imposed filters will then take effect but I agree with paul_irish that this is too common to ignore.

comment:23 Changed 9 years ago by john

Description: modified (diff)

+0, This will allow the stylesheet filters to override the element (potentially making it hidden or partially visible) which seems unintended. While I agree that it looks bad in its end state and should look better, I'm not sure if the solution is really that good.

Note: See TracTickets for help on using tickets.