Opened 14 years ago

Last modified 14 years ago

#6187 closed Bug

IE6 skin images 404 — at Version 1

Reported by: Niko Viitala Owned by:
Priority: Normal Milestone: CKEditor 3.4.2
Component: UI : Skins Version:
Keywords: Cc:

Description (last modified by Alfonso Martínez de Lizarrondo)

IE6 (SP2) tries to load skin images two times. In the other time, it tries to search images in wrong path "/js/ckeditor/skins/kama/http://URL/js/ckeditor/skins/kama/icons.png" etc, so those images are redirect to 404. The same seems to happen in skin v2 too. IE > 6 and other browsers doesn't have the bug.

Error log:

"GET /js/ckeditor/config.js?t=A7HG4HT HTTP/1.1" 200 1289
"GET /js/ckeditor/skins/kama/images/sprites_ie6.png?t=A7HG4HT HTTP/1.1" 304
"GET /js/ckeditor/skins/kama/icons.png?t=A7HG4HT HTTP/1.1" 304 304
"GET /js/ckeditor/skins/kama/images/dialog_sides.gif?t=A7HG4HT HTTP/1.1" 304
"GET /js/ckeditor/skins/kama/https://www.xxx.fi/js/ckeditor/skins/kama/icons.png?t=A7HG4HT&t=A7HG4HT HTTP/1.1" 404
"GET /js/ckeditor/skins/kama/https://www.xxx.fi/js/ckeditor/skins/kama/images/sprites_ie6.png?t=A7HG4HT&t=A7HG4HT HTTP/1.1" 404
"GET /js/ckeditor/skins/kama/https://www.xxx.fi/js/ckeditor/skins/kama/images/dialog_sides.gif?t=A7HG4HT&t=A7HG4HT HTTP/1.1" 404
"GET /js/ckeditor/skins/kama/https://www.xxx.fi/js/ckeditor/skins/kama/https://www.tyokyvyntuki.fi/js/ckeditor/skins/kama/icons.png?t=A7HG4HT&t=A7HG4HT&t=A7HG4HT HTTP/1.1" 404

Suggested fix: I deleted these lines (in compressed ckeditor.js or in skin.js) and it's working fine again. Maybe a better approach would be to fix preload function?:

        if ( CKEDITOR.env.ie && CKEDITOR.env.version < 7 )
        {
                // For IE6, we need to preload some images, otherwhise they will be
                // downloaded several times (CSS background bug).
                preload.push( 'icons.png', 'images/sprites_ie6.png', 'images/dialog_sides.gif' );
        }

Change History (1)

comment:1 Changed 14 years ago by Alfonso Martínez de Lizarrondo

Description: modified (diff)
Status: newconfirmed

I don't see a 404, but two images are loaded twice: one with the timestamp and the second one without it.

http://nightly.ckeditor.com/5829/_samples/replacebyclass.html
http://nightly.ckeditor.com/5829/ckeditor.js
http://nightly.ckeditor.com/5829/_samples/sample.js
http://nightly.ckeditor.com/5829/config.js?t=A7M74HW
http://nightly.ckeditor.com/5829/skins/kama/icons.png?t=A7M74HW
http://nightly.ckeditor.com/5829/skins/kama/images/sprites_ie6.png?t=A7M74HW
http://nightly.ckeditor.com/5829/skins/kama/images/dialog_sides.gif?t=A7M74HW
http://nightly.ckeditor.com/5829/skins/kama/editor.css?t=A7M74HW
http://nightly.ckeditor.com/5829/lang/es.js?t=A7M74HW
http://nightly.ckeditor.com/5829/skins/kama/icons.png
http://nightly.ckeditor.com/5829/skins/kama/images/sprites_ie6.png
http://nightly.ckeditor.com/5829/contents.css
http://nightly.ckeditor.com/5829/plugins/styles/styles/default.js?t=A7M74HW&t=A7M74HW

So we should certainly look at the reason of this problem.
I guess that the ones without the timestamp are loaded due to the stylesheet references.
And maybe the patch to avoid zoom problems in IE can expand this issue to all IE versions.

Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy