Ticket #4919 (closed Bug: fixed)
Jquery adapter sample file is incorrect
|Reported by:||wwalc||Owned by:||tobiasz.cudnik|
|Component:||General||Version:||SVN (CKEditor) - OLD|
There is a very specific way of creating samples for CKEditor. A short description is available here: http://docs.cksource.com/FCKeditor_3.x/Design_and_Architecture/CKReleaser#Samples
In short, html samples are just templates and sample-specific content should be placed inside of elements with the following IDs:
- headscript (optional): contains sample specific script to be loaded in the <head>.
- styles (optional): contains sample specific styles.
- html: contains the HTML data that effectively represents the sample.
- code: contains the sample code, used to illustrate the API usage in the sample.
Another rule is that all sample files should be loading sample.js:
In jqueryadapter.html the following files are loaded in the <head> section, which causes that sample processed by CKReleaser is not working:
Ideally, both files should be loaded in a script with id "headscript" and below there should be a jquery code $( function() ... } ); that is currently in a script that has no id attribute.
To test whether the template is correct, just launch _samples/sample.html?sample=jqueryadapter and see whether it works.
There is one more minor bug that can be corrected as well: http://www.fckeditor.net url shouldn't be used any more in the sample text.