Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TopUp IFrame is not shown until clicked on in IE6 #2

Open
bright4 opened this issue Oct 22, 2009 · 2 comments
Open

TopUp IFrame is not shown until clicked on in IE6 #2

bright4 opened this issue Oct 22, 2009 · 2 comments

Comments

@bright4
Copy link

bright4 commented Oct 22, 2009

This is most probably some IE6 render quirk, but on some pages with redirection, the iframe goes hidden after the redirection kicks in (at least, that's what I think it does).

The fix that works is to change the following line from the 'ie6fix' CSS:

.tu_transparent { filter: alpha(opacity = 0); }

into:

.tu_transparent { visibility: hidden; }

Perhaps changing all other (also non-IE) zero-opacity settings into this visibility:hidden is a performance improvement?

Thanks,
Chris (your colleauge)

@bright4
Copy link
Author

bright4 commented Oct 22, 2009

The above comment was made too soon... (but still may be an improvement, no? ;-) Also, it should be about the 'iefix' and not the 'ie6fix' CSS.

The real issue was with the following piece from the 'iefix' CSS:

#tu_overlay { top: expression((ignoreMe = document.documentElement.scrollTop ? document.documentElement.scrollTop : document.body.scrollTop) + "px"); left: expression((ignoreMe2 = document.documentElement.scrollLeft ? document.documentElement.scrollLeft : document.body.scrollLeft) + "px"); position: absolute; }

Removing this resolves the issue. In my case, I moved it to the 'ie7fix' CSS.

@archan937
Copy link
Owner

Hi Chris,

I will take a look at the possible performance improvement. As for the dodgy iframe rendering in IE6, do you have a test page online so I can test it in IE6 and IE7 myself? Either way, thank you for contributing little fixes for TopUp.

Greets,
Paul.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants