Opened 3 years ago

Closed 3 years ago

#13497 closed Bug (invalid)

multiple contents.css loaded for multiple instance (v4.5.1)

Reported by: Carlos Cuesta Iglesias Owned by:
Priority: Normal Milestone:
Component: Performance Version:
Keywords: Cc:

Description

Hello:

It is already documented and closed but the error persists in this release. Steps to reproduce the error:

1) Run: <!DOCTYPE html> <html>

<head>

<title>Prueba CKEditor</title> <meta charset="UTF-8"> <meta name="viewport" content="width=device-width, initial-scale=1.0">

<script type="text/javascript" src="../XXXXXX/jquery-2.1.3.min.js"></script> <script src="../XXXXXX/ckeditor.js"></script>

<script type="text/javascript">

$(function () {

var arrayTxtArea = ["txta1", "txta2", "txta3", "txta4"]; var total = arrayTxtArea.length; for (var i = 0; i < total; i++) {

CKEDITOR.replace(arrayTxtArea[i]);

}

});

</script>

</head> <body>

<textarea id="txta1" name="txta1"></textarea> <textarea id="txta2" name="txta2"></textarea> <textarea id="txta3" name="txta3"></textarea> <textarea id="txta4" name="txta4"></textarea>

</body>

</html>

2) See debug output:

http://localhost/XXXXX/ckeditor/contents.css [HTTP/1.1 200 OK 15ms] http://localhost/XXXXX/ckeditor/contents.css [HTTP/1.1 200 OK 8ms] http://localhost/XXXXX/ckeditor/contents.css [HTTP/1.1 200 OK 8ms] http://localhost/XXXXX/ckeditor/contents.css [HTTP/1.1 200 OK 8ms]

Thank You,

Carlos Universidad de Caldas - Colombia

Attachments (2)

TheBug.png (26.0 KB) - added by Carlos Cuesta Iglesias 3 years ago.
Result debug
TheBug.html.txt (1.0 KB) - added by Carlos Cuesta Iglesias 3 years ago.
The source code

Download all attachments as: .zip

Change History (5)

comment:1 Changed 3 years ago by Piotrek Koszuliński

Component: Core : StylesPerformance
Keywords: Contents.css removed
Status: newpending
Version: 4.5.1

Don't you have cache disabled when the console is open? For realistic results you would need to check with the dev tools closed. There's no reason why this file would be requested multiple times. It should be cached the first time it's loaded and then reused.

Changed 3 years ago by Carlos Cuesta Iglesias

Attachment: TheBug.png added

Result debug

Changed 3 years ago by Carlos Cuesta Iglesias

Attachment: TheBug.html.txt added

The source code

comment:2 in reply to:  1 Changed 3 years ago by Carlos Cuesta Iglesias

Hi:

This only takes a few minutes. Please try to see what happens with the code sent.

Thank you very much!

comment:3 Changed 3 years ago by Piotrek Koszuliński

Resolution: invalid
Status: pendingclosed

I tried. And as I said - it's loaded once. I've checked on many browsers.

p@m ~> tail -1000 /var/log/apache2/access_log | grep contents.css
::1 - - [03/Jul/2015:14:45:06 +0200] "GET /ckeditor-releases/contents.css?t=F62A HTTP/1.1" 200 1967
::1 - - [03/Jul/2015:14:45:07 +0200] "GET /ckeditor-releases/contents.css?t=F62A HTTP/1.1" 200 1967
::1 - - [03/Jul/2015:14:45:30 +0200] "GET /ckeditor-releases/contents.css?t=F62B HTTP/1.1" 200 1967
::1 - - [03/Jul/2015:14:47:00 +0200] "GET /ckeditor-releases/contents.css?t=F62B HTTP/1.1" 200 1967
::1 - - [06/Jul/2015:09:19:47 +0200] "GET /ckeditor-dev/contents.css HTTP/1.1" 304 -
::1 - - [06/Jul/2015:10:05:58 +0200] "GET /ckeditor-dev/contents.css HTTP/1.1" 200 1835
::1 - - [06/Jul/2015:10:08:58 +0200] "GET /ckeditor-dev/contents.css HTTP/1.1" 200 1835
127.0.0.1 - - [06/Jul/2015:10:11:19 +0200] "GET /ckeditor-dev/contents.css HTTP/1.1" 304 -

You're reading the results from your dev tools incorrectly.

Note: See TracTickets for help on using tickets.
© 2003 – 2017 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy