Opened 14 years ago

Last modified 14 years ago

#6187 closed Bug

IE6 skin images 404 — at Initial Version

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

Description

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 (0)

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