Closed (works as designed)
Project:
CKEditor 4 - WYSIWYG HTML editor
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
28 Jan 2013 at 04:14 UTC
Updated:
2 Jun 2014 at 15:56 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
jmoughon commentedI turned off the JS stuff too.
Comment #2
iamcarrico commentedOkay, I fixed the code in the dev branch so that when the "push js in footer" setting is off--- it will work. Still need to figure out why it is not working with our processing... but with my experiences with WYSIWYGs... its just that way.
Comment #3
jmoughon commentedO Joy, it works (with move to footer disabled)!!! Thanks a bunch Ian. Let me know how I can help with maintenance or anything else. May require a little knowledge dump or something?
Leaving open for complete fix...
Comment #4
caponey commentedHaving same issue.
This only happens with CKeditor module, and not with Wysiwyg module + CKeditor library.
Comment #5
iamcarrico commentedI am moving this to the Magic module's issue queue, as this functionality is being pulled from Aurora and put there for better open source collaboration.
On a related note: I would bet a good deal of money that we just need to check with the CKEditor module on what needs to have 'force header' in their javascript. If somebody has the time to check their code to see what needs to be in the head vs. the footer, that may be the issue.
Comment #6
iamcarrico commentedSo I have figured out (what I think) is the issue. It is just that when we push all js (except libraries) to the footer, the order of the CKeditor files gets all sorts of messed up. Ironically, because they are putting them in the footer in the first place. So 1) well done CKEditor guys, but 2) Can we change the weight? I am attaching a patch that seems to fix it for me, but y'all would know better what this is doing.
To give a touch of background, the magic module takes code that was in Aurora to enhance the CSS/JS handling and puts it in a contrib module. The error is that as we force all js in the footer, CKEditor breaks.
Comment #7
jmoughon commented