Generally a script should be placed just before </body> for better performance. But in case of JavaScript aggregation, put in footer generates one more http requests.

So I think CKEditor should detect if aggregation is enabled and place ckeditor.utils.js in header (in the "theme" scope, however).

CommentFileSizeAuthor
#3 772396-ckeditor-header.patch912 bytesjcisio
#2 772396-ckeditor-header.patch483 bytesjcisio
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

wwalc’s picture

wwalc CreditAttribution: wwalc commented

Sounds good, any chance for a patch? :)

jcisio’s picture

jcisio CreditAttribution: jcisio commented
Status: Active » Needs review
FileSize
772396-ckeditor-header.patch483 bytes

Just checkout and create patch. Not tested, but should work.

jcisio’s picture

jcisio CreditAttribution: jcisio commented
FileSize
772396-ckeditor-header.patch912 bytes

Wrong cvs diff syntax. Use this one.

bryancasler’s picture

bryancasler CreditAttribution: bryancasler commented

#3 worked for me, did exactly what it said it would

jcisio’s picture

jcisio CreditAttribution: jcisio commented
Status: Needs review » Reviewed & tested by the community

Change the status thus :)

wwalc’s picture

wwalc CreditAttribution: wwalc commented
Status: Reviewed & tested by the community » Fixed

Patch committed into CVS, thanks jcisio!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.