Opened 13 years ago
Closed 13 years ago
#5617 closed defect (fixed)
[patch][ccla]Editor: in mail demo, new message page yields page loading error (IE)
Reported by: | Adam Peller | Owned by: | alex |
---|---|---|---|
Priority: | high | Milestone: | 1.1 |
Component: | Dijit | Version: | 1.1b1 |
Keywords: | Cc: | liucougar, bill | |
Blocked By: | Blocking: |
Description (last modified by )
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.
Attachments (2)
Change History (17)
comment:1 Changed 13 years ago by
Owner: | set to Adam Peller |
---|
comment:2 Changed 13 years ago by
Resolution: | → worksforme |
---|---|
Status: | new → closed |
works as of 2008-01-24
comment:3 Changed 13 years ago by
Description: | modified (diff) |
---|---|
Resolution: | worksforme |
Status: | closed → reopened |
Version: | 1.0 → 1.1b1 |
comment:4 Changed 13 years ago by
Priority: | normal → high |
---|
comment:5 Changed 13 years ago by
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".
comment:6 Changed 13 years ago by
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.
comment:7 Changed 13 years ago by
Cc: | liucougar alex added |
---|
comment:8 Changed 13 years ago by
comment:9 Changed 13 years ago by
Using ifr.src=; also makes the IE6 error go away for me. I'm not getting the mixed content message with any combination.
comment:11 Changed 13 years ago by
Summary: | IE: mail demo, new message yields page loading error → Editor: in mail demo, new message page yields page loading error (IE) |
---|
comment:12 Changed 13 years ago by
Owner: | changed from Adam Peller to alex |
---|---|
Status: | reopened → new |
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.
comment:13 Changed 13 years ago by
Cc: | bill added; alex removed |
---|
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.
comment:14 Changed 13 years ago by
The patch is still working. The missing Send/Cancel? buttons are from a separate regression covered by #6208.
comment:15 Changed 13 years ago by
Summary: | Editor: in mail demo, new message page yields page loading error (IE) → [patch][ccla]Editor: in mail demo, new message page yields page loading error (IE) |
---|
comment:16 Changed 13 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
Not happening for me, on IE6 or IE7 or FF. Still happening for you?