Update from 4.6.2 to CKEditor 4.7.1: http://ckeditor.com/blog/CKEditor-4.7-released + http://ckeditor.com/blog/CKEditor-4.7.1-released
4.7.0 + 4.7.1 bring:
- Paste from Excel
Advanced table selection— see #4- improved conformance with Content Security Policy:
CKEditor 4.7 brings improved compliance with CSP with the removal of dynamic evaluation calls in the template system. Thanks to this, the editor can now be used with
unsafe-evalContent Security Policy. - improved Paste from Word
- improved general pasting
- Mexican Spanish localization
See also:
#1858210: [meta] Content editing experience follow-ups — in-place editing and WYSIWYG
#1950098: Update CKEditor library to 4.1
#2036253: Update CKEditor library to 4.2
#2039163: Update CKEditor library to 4.4
#2271051: Update CKEditor library to 4.4.4
#2345961: Update CKEditor library to 4.4.5
#2384581: Security: Update CKEditor library to 4.4.6
#2415111: Update CKEditor library to 4.4.7
#2521820: Update CKEditor library to 4.5.3
#2321583: Update CKEditor library to 4.5.5
#2663566: Update CKEditor library to 4.5.7
#2698587: Update CKEditor library to 4.5.8
#2724225: Update CKEditor library to 4.5.9
#2765751: Update CKEditor library to 4.5.10
#2797427: Update CKEditor library to 4.5.11
#2828494: Update CKEditor library to 4.6
#2848215: Update CKEditor library to 4.6.2
| Comment | File | Size | Author |
|---|---|---|---|
| #7 | ckeditor_4.7.1-2893566-7.patch | 3.11 MB | wim leers |
Comments
Comment #2
wim leersComment #3
wim leersWhen building CKEditor 4.7.1, I noticed that it grew from 563 KB for 4.6.2 to 576 KB for 4.7.1. I asked the CKEditor team for an explanation.
Comment #4
wim leersThe new actually is a new optional plugin: https://ckeditor.com/addon/tableselection. It's 31 KB, which is quite a lot. So, for now, not adding this to our CKEditor build. It'll make CKEditor slower for everyone, even though only a minority uses the "Table" tool in CKEditor. Besides, these updates are meant to be just updates, not adding more plugins.
IS updated.
Comment #5
wim leerswas one of the highlights of CKEditor 4.7. What does this mean exactly?
Source: http://ckeditor.com/blog/CKEditor-4.7-released
Therefore this closes #2789139: [upstream] CSP requires 'unsafe-inline' because of CKEditor 4 and helps #2513356: Add a default Content-Security-Policy and clickjacking defence to core.
Comment #6
wim leersThe new Mexican Spanish localization (
es-mx) is one that Drupal does not have.See https://localize.drupal.org/.
The infrastructure for making CKEditor use the same language as Drupal core was added in #2050097: Map CKEditor languages to Drupal languages. And even then, there were some localizations that CKEditor has that Drupal doesn't, for example
en-au.This then follows the example set there: we still add this language, so that sites that add
es-mxthemselves, then they'd get that translation in CKEditor too.Comment #7
wim leersOnly two noteworthy things in this patch:
ignorelines tobuild-config.js, which have been added upstream (to preventbender.ci.jsand.travis.ymlfrom making it into Drupal core)es-mxtranslationcore.libraries.ymlI did manual testing, it still works fine.
Comment #8
grandmaglassesropemanComment #9
grandmaglassesropemanThis doesn't appear to be a problem with this version, however
ckeditor.jsis making a request forcore/modules/ckeditor/tests/modules/ckeditor_test.csswhich does not exist. Beyond that, my manual testing didn't present any issues.Comment #10
wim leers#9: is it possible you had the
ckeditor_testmodule installed during your testing? The purpose of that module is testinghook_ckeditor_css_alter(), which is indeed adding that file.Comment #11
grandmaglassesropeman@Wim Leers
Yep. Awesome. 🎉
Comment #12
webchickAsked for some more details on what @drpal's manual testing entailed, and he responded: "Clicked everything on the node/edit screen (ckeditor). Added/removed everything in the ckeditor setup. I tried to make sure all of the buttons worked/behaved as expected."
Sounds great. The sooner this is in front of people, the sooner we'll find any outstanding issues that weren't caught by this manual testing, so let's get this in.
Committed and pushed to 8.4.x. Thanks!
Comment #15
gábor hojtsyRemoving release notes tag as this was superseded by #2904142: Update CKEditor library to 4.7.2.