[patch][ccla]Editor: in mail demo, new message page yields page loading error (IE)

[Adam Peller]

In mail.html, hit "New Message", then close the tab (click the x), click OK to "are you sure you want to leave your changes?" then hit "New Message" again. 404 error.

Happens on IE6.

[bill]

Not happening for me, on IE6 or IE7 or FF. Still happening for you?

[Adam Peller]

works as of 2008-01-24

[bill]

Patch fixes problem on mail.html. Concerning the line:

ifr.src = 'javascript:void(0)';

Patch removes that line, but things seems to work even when that line is there. If it's removed then loading test_Editor.html gets warnings about "this page contains both secure and insecure content".

[Adam Peller]

I see javascript:"" used in popup.js. Is that the same thing?

I don't see the try/catch nor the setTimeout pattern used in popup.js, though. Why is it not an issue there?

also, the 'var e' is unnecessary.

[Douglas Hays]

When I leave the ifr.src='javascript:void(0)' statement in, I get an error on IE6:

[Douglas Hays]

screen shot of IE6 error

[Douglas Hays]

Using ifr.src=; also makes the IE6 error go away for me. I'm not getting the mixed content message with any combination.

[bill]

As per meeting today, Alex want to poke at this and see if he can find a solution w/out a timer, like in ​http://trac.dojotoolkit.org/browser/dojo/trunk/io/iframe.js#L101 (not sure how that's related but was mentioned in the meeting). Cougar is OK with the third patch above.

[Douglas Hays]

refreshed stale patch to fix merge problems

[Douglas Hays]

The OK and Cancel buttons are no longer showing on the New Message panel using IE6 with the 5617.3.patch. Pretty sure this used to work a few days ago.

[Douglas Hays]

The patch is still working. The missing Send/Cancel? buttons are from a separate regression covered by #6208.

[alex]

(In [13138]) clean patch for #5617 that avoids timer for all but IE 6. Fixes #5617. !strict