Changes between Initial Version and Version 1 of Ticket #8561


Ignore:
Timestamp:
Feb 3, 2009, 6:04:08 PM (11 years ago)
Author:
bill
Comment:

Hmm it's strange that you are seeing that but we don't see it in our tests. I looked at the code though and the code does seem strange, particularly the push functions (which caches iframes for future use):

this.push = function(iframe){
	iframe.style.display="";
	...

Seems like that should be setting display to none.

Do you have a test case? Or does it reproduce for you for any of the tests in dijit/tests?

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #8561

    • Property Owner set to bill
    • Property Status changed from new to assigned
    • Property Milestone changed from tbd to 1.3
  • Ticket #8561 – Description

    initial v1  
    44
    55The offending code is in dijit/_base/popup.js:245
    6 
     6{{{
    77  var burl = dojo.config["dojoBlankHtmlUrl"] ||
    88    (dojo.moduleUrl("dojo", "resources/blank.html")+"") ||
     
    1212     + "z-index: -1; filter:Alpha(Opacity=\"0\");'>";
    1313  iframe = dojo.doc.createElement(html);
    14 
     14}}}
    1515(Reformatted slightly for readability). This is only used on IE, phiggins told me on IRC it was needed to prevent bleed-through of SELECT.
    1616