IE6 Image Dialog positioning

[Shaun]

This seems to be intermittent -- I have not figured out how to reproduce it 100% of the time.

When opening the editor in IE6, and clicking the "Image" icon in the toolbar, the dialog box that pops up is positioned half-way off the top of the screen.

This was experienced on multiple computers running IE6. 1 PC was running IE6 under Microsoft Virtual PC. 2 PCs were running IE6 natively. All of them experienced this bug and I think it happened under a cache-free editor-load.

I'm using a clean install with the default replace-textarea-by-ID implementation.

[Shaun]

I also wanted to add that in my implementation, you had to scroll down the page a ways in order to see the editor.

[Garry Yao]

A sample page for reproducing would be great.

[Shaun]

It appears that this happens only under a non-cached load.

To test it:

1. Open IE6 or IE7 and delete your temporary files.
2. Navigate to ​http://www.simshaun.com/ckeditor/_samples/replacebyclass.html
3. *Bug* Scroll down to the editor and click the Image icon in the toolbar. For me, this places the dialog way up in the page.
4. Refresh the page, scroll down to the editor and click the Image icon again. This time, the dialog is correctly positioned. This is why I am coming to the conclusion it only occurs when the CKEditor files are not cached.

Also, I'm noticing that IE6 seems to take a lot longer than other browsers to load the editor when files are not cached. It appears that IE6 is re-downloading the sprites for every icon in the toolbar rather than just downloading it once.

[Shaun]

The bug still exists in 3.0.2.

After some more testing I've confirmed it only occurs under a page load where the editor must be fully loaded.

I've still got to do some more testing, but doing this seems to fix it:

CKEditor 3.0.2/_source/plugins/dialog/plugin.js, Line 687 (in the show() function):

100, this );

Change to

CKEDITOR.env.ie6Compat ? 1000 : 100, this);

[Shaun]

Forgot to mention, I believe it only occurs when it takes the browser a second or two to load the dialog.

The change I made may need to be applied to IE7 as well, but I've not finished testing yet.

[Jakub Ś]

This is no longer reproducible on current trunk.