Posted by ball.in.th on January 3, 2011 at 12:25pm
3 followers
Jump to:
| Project: | CKEditor - WYSIWYG HTML editor |
| Version: | 6.x-1.2 |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
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 .
| Attachment | Size |
|---|---|
| footer.patch | 1.67 KB |
Comments
#1
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.
#2
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.
#3
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.
#4
Aggregation was fixed with Google Page Speed support, so now CKEditor module have full support for it.
Greetings