Opened 16 years ago
Closed 16 years ago
#3672 closed Bug (fixed)
[IE] Dialogs flicker when displaying
Reported by: | Damian | Owned by: | Tobiasz Cudnik |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 3.0 |
Component: | General | Version: | |
Keywords: | IBM Confirmed Review+ | Cc: |
Description
When dialogs are opened for the first time, they load and display correctly. On subsequent invocations of a dialog, the whole window is blanked out while the dialog displays. The worst case seems to be on IE6; the delay can last more than a second, while the whole window remains blank.
The effect seems more pronounced when the editor is embedded within a larger application, but is still noticeable in the samples.
Attachments (2)
Change History (11)
comment:1 Changed 16 years ago by
Keywords: | Confirmed added |
---|
comment:2 Changed 16 years ago by
Owner: | set to Tobiasz Cudnik |
---|---|
Status: | new → assigned |
comment:3 Changed 16 years ago by
Keywords: | Pending added |
---|
comment:4 Changed 16 years ago by
I have tried on today's nightly build (http://nightly.ckeditor.com/3668/_samples/ajax.html). The effect is still noticeable.
To reproduce, open the nightly sample in IE6 and once the editor has been created, click on the Smiley dialog. Notice that it loads properly the first time and that the editor does not disappear. Close the dialog and open it again, this time when the dialog opens the whole page goes blank for a fraction of a second. Sometimes this "fraction of a second" can go beyond a second.
Tested on Win XP, IE 6.
Changed 16 years ago by
Attachment: | 3672.patch added |
---|
comment:5 Changed 16 years ago by
Keywords: | Review? added; Pending removed |
---|
I was able to reproduce this issue once again when i've switched examples back to strict mode.
Attached patch caches cover element (which caused problems, mostly in IE) so it's much faster when opening dialog second time. Gains time also when opening other dialogs after first one and on all browsers.
comment:6 Changed 16 years ago by
Keywords: | Review- added; Review? removed |
---|
The coverElement variable is actually a div, so all calls to coverElement.getFrameDocument() are invalid. The iframe is the first child inside the coverElement.
Changed 16 years ago by
Attachment: | 3672_2.patch added |
---|
comment:7 Changed 16 years ago by
Keywords: | Review? added; Review- removed |
---|
You right Martin, uploaded a correct patch.
comment:8 Changed 16 years ago by
Keywords: | Review+ added; Review? removed |
---|
Please try the newest trunk, in which there is a new skin and many editor's layout structures have been changed. I cant reproduce white screen on newest trunk, while i can on older, before merging of #3503.