I've tried loading scripts in the footer and it seems to work well. Please note that the patch current has $preprocess set to FALSE because it would produce an error otherwise -- there's a bug in either the aggregator or in ckeditor.js.

Loading scripts in the footer should make pages load faster -- http://drupal.org/node/784626#comment-3837990 .

CommentFileSizeAuthor
footer.patch1.67 KBball.in.th

Comments

jcisio’s picture

jcisio commented

About ckeditor.utils.js, check #772396: Put ckeditor.utils.js in header for better aggregation.

About the editor itself, I'm not against :) But I prefer a basic loader (ckeditor_basic.js) for better experience.

ball.in.th’s picture

ball.in.th commented

I prefer ckeditor_basic.js too. They all can be moved to the footer. Since scripts in header should be minimal and scripts in footer are aggregated too, i think it's still best to be in the footer. There might not be that many modules to put scripts in the footer yet, but this would be a good start.

ball.in.th’s picture

ball.in.th commented

Update: aggregation have problems with both ckeditor.js & ckeditor_basic.js, not sure where the bug is though. So its best to set $preprocess to FALSE for now as in the original patch.

dczepierga’s picture

dczepierga commented
Status: Active » Closed (fixed)

Aggregation was fixed with Google Page Speed support, so now CKEditor module have full support for it.

Greetings