Changes between Initial Version and Version 1 of Ticket #6187


Ignore:
Timestamp:
Aug 22, 2010, 6:04:55 PM (14 years ago)
Author:
Alfonso Martínez de Lizarrondo
Comment:

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.

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #6187

    • Property Status changed from new to confirmed

  • Ticket #6187 – Description

    initial v1  
    22
    33Error log:
     4{{{
    45"GET /js/ckeditor/config.js?t=A7HG4HT HTTP/1.1" 200 1289
    56"GET /js/ckeditor/skins/kama/images/sprites_ie6.png?t=A7HG4HT HTTP/1.1" 304
    … …  
    1011"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
    1112"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
     13}}}
    1214
    1315Suggested fix:
    1416I 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?:
     17{{{
    1518        if ( CKEDITOR.env.ie && CKEDITOR.env.version < 7 )
    1619        {
    … …  
    1922                preload.push( 'icons.png', 'images/sprites_ie6.png', 'images/dialog_sides.gif' );
    2023        }
     24}}}
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy