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 )
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
Description: | modified (diff) |
---|---|
Status: | new → confirmed |
I don't see a 404, but two images are loaded twice: one with the timestamp and the second one without it.
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.