Opened 8 years ago

Closed 8 years ago

#16912 closed task (wontfix)

BackgroundIframe: remove filter:opacity

Reported by: bill Owned by: bill
Priority: undecided Milestone: 1.9
Component: Dijit Version: 1.8.3
Keywords: Cc:
Blocked By: Blocking:

Description

In [3153] I added a filter:Alpha(Opacity=\"0\") to the BackgroundIframe. Not sure why I added that; maybe to have semi-transparent dropdowns. But ISTM not to be used today, and according to ​http://code.google.com/p/google-web-toolkit/wiki/UnderstandingMemoryLeaks it can use a lot of memory.

Change History (6)

comment:1 Changed 8 years ago by bill

Resolution: → fixed
Status: new → closed

In [31003]:

Remove the opacity filters from BackgroundIframe. I think I put them in a long time ago to support semi-transparent dropdowns, but they aren't being used now, and on IE they are dangerous in terms of memory usage. Fixes #16912 !strict.

comment:2 Changed 8 years ago by bill

In [31016]:

Turns out transparency is needed on IE for the DialogUnderlay to be transparent. Making that work again. Refs #16912 !strict.

comment:3 Changed 8 years ago by bill

Milestone: tbd → 1.9

comment:4 Changed 8 years ago by bill

Resolution: fixed
Status: closed → reopened

Turns out that transparency is also needed for irregularly shaped popups, i.e. for tooltips (and TooltipDialogs) so that you can see the part of the screen near the tooltip's connector (triangle pointer thing).

Could make the code conditional but it's probably not worth it; I'm going to just rollback the changes.

comment:5 Changed 8 years ago by bill

In [31252]:

make BackgroundIframe transparent again (in all cases), so that tooltips and TooltipDialogs appear correctly on IE, refs #16912

comment:6 Changed 8 years ago by bill

Resolution: → wontfix
Status: reopened → closed
Note: See TracTickets for help on using tickets.