search

javierdotnet / gwt-ckeditor

Automatically exported from code.google.com/p/gwt-ckeditor
0 stars 0 forks source link

CkEditor.js is not loading automatically when using the module #23

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
I have created a patch for you that will let GWT include the ckeditor.js into 
the page. This will remove the need to add it manually to the HTML page.

Index: src/com/axeiya/gwtckeditor/Gwtckeditor.gwt.xml
===================================================================
--- src/com/axeiya/gwtckeditor/Gwtckeditor.gwt.xml  (revision 63)
+++ src/com/axeiya/gwtckeditor/Gwtckeditor.gwt.xml  (working copy)
@@ -9,4 +9,5 @@
   <!-- Specify the app entry point class.                         -->

   <public path="public"/>
+  <script src="ckeditor/ckeditor.js"></script>
 </module>

Original issue reported on code.google.com by per.ab...@gmail.com on 15 Sep 2010 at 8:30

GoogleCodeExporter commented 9 years ago 
We will consider it, but keep in mind that this is not the only thing to do in 
the host page. The var CKEDITOR_BASEPATH has to be defined too.

Original comment by picard.d...@gmail.com on 22 Sep 2010 at 5:32

GoogleCodeExporter commented 9 years ago 
Have you tested it? I had no problems by adding the patch (actually jist adding 
the script line) and inheriting the module without setting the 
CKEDITOR_BASEPATH variable.

Running Linux 10.04, JDK 1.6, Eclipse Galileo, Firefox 3.6.10

Original comment by cirovlad...@gmail.com on 22 Sep 2010 at 9:05

GoogleCodeExporter commented 9 years ago 
http://docs.cksource.com/CKEditor_3.x/Developers_Guide/Specifying_the_Editor_Pat
h

Original comment by picard.d...@gmail.com on 23 Sep 2010 at 5:15

GoogleCodeExporter commented 9 years ago 
As defined at the Integration  page, it's enough to include the "ckeditor.js" 
file into your pages to have CKEditor working. There are some situations though 
when the original editor file name may not be used:

    * The file must be renamed, for any reason.
    * The original editor script is merged with other scripts used in the page, to reduce the number of files being downloaded. This single file will also have a name that's not "ckeditor.js". This is the most common case.
    * You have yet another file named "ckeditor.js", which is not in the editor installation directory. 

Ok, although I think that would be for uncommon situations (i'm glad ckeditor 
developers have thought of them). In any case, if any of us (developers) are in 
this situation we'd still be able to set the variable in the host html page.

Original comment by cirovlad...@gmail.com on 23 Sep 2010 at 1:32

GoogleCodeExporter commented 9 years ago 
Done. Thank you for the patch.

Original comment by picard.d...@gmail.com on 29 Jan 2011 at 7:20