[Rt-commit] rt branch, 4.2/ckeditor-contents-css, created. rt-4.2.3-87-g0a3bade

? sunnavy sunnavy at bestpractical.com
Mon Apr 28 11:40:25 EDT 2014


The branch, 4.2/ckeditor-contents-css has been created
        at  0a3badea2d49615f47d971e9d94ab9b11a87afe0 (commit)

- Log -----------------------------------------------------------------
commit 0a3badea2d49615f47d971e9d94ab9b11a87afe0
Author: sunnavy <sunnavy at bestpractical.com>
Date:   Thu Apr 24 23:28:45 2014 +0800

    explicitly set contentsCss in ckeditor config
    
    ckeditor figures basePath from the location of ckeditor.js specified in
    <script>'s src by default, but if we are in production mode ($DevelMode is
    false), since ckeditor.js is also appended into /NoAuth/js/squished-...js,
    ckeditor fails to find the correct basePath.
    
    though we also set basePath before calling ckeditor, it's too late for
    contentsCss url, which is a bug of ckeditor and fixed in 4.3.2
    
    see also #29780 and http://dev.ckeditor.com/ticket/11350

diff --git a/share/static/RichText/config.js b/share/static/RichText/config.js
index 8317bfc..35ca412 100644
--- a/share/static/RichText/config.js
+++ b/share/static/RichText/config.js
@@ -44,4 +44,5 @@ config.font_names =
     'Times New Roman/Times New Roman, Times, serif;' +
     'Trebuchet MS/Trebuchet MS, Helvetica, sans-serif;' +
     'Verdana/Verdana, Geneva, sans-serif';
+config.contentsCss = CKEDITOR.basePath + 'contents.css'; // won't need this after ckeditor 4.3.2
 };

-----------------------------------------------------------------------


More information about the rt-commit mailing list