Update from 4.14.0 to 4.14.1.

CKEditor 4.14.1 has been released: https://ckeditor.com/blog/CKEditor-4.14.1-with-resizing-and-toolbar-impr...

I encountered this Bug in Drupal 8.9, and the new release fixes it: https://github.com/ckeditor/ckeditor4/issues/3926

Release notes snippet

CKEditor has been updated from from 4.14.0 to 4.14.1, which includes numerous bugfixes and improvements.

Comments

maacl created an issue. See original summary.

nkoporec’s picture

Tested the patch and it works(tested on latest 8.9), it fixes the issue with CKEditor drag, where a JS error would should up:
Issue

MykolaVeryha’s picture

StatusFileSize
new590 bytes
MykolaVeryha’s picture

We need to add this patch too
It's to clear cache

MykolaVeryha’s picture

It works on core 8.8.8

maacl’s picture

StatusFileSize
new1.9 MB
new595 bytes

Thanks! Updated the patch.

samiullah’s picture

Applied the patch on clean install of 9.1x
Composer was updated to 4.14.1
This can be moved to RTBC

samiullah’s picture

Status: Needs review » Reviewed & tested by the community
alexpott’s picture

Title: Update CKEditor to version 4.14.1 » [backport] Update CKEditor to version 4.14.1
Version: 9.1.x-dev » 8.9.x-dev

I built ckeditor locally and the only difference was a compile timestamp. As this is a bugfix this can go back to 9.0.x. Also I think the same logic applies to 8.9.x - and we should consider committing this there.

  • alexpott committed db47cc8 on 9.1.x
    Issue #3155110 by maacl, Mykola Veryha: Update CKEditor to version 4.14....

  • alexpott committed 4fbe447 on 9.0.x
    Issue #3155110 by maacl, Mykola Veryha: Update CKEditor to version 4.14....
wim leers’s picture

we should consider committing this there.

+1

alexpott’s picture

Title: [backport] Update CKEditor to version 4.14.1 » Update CKEditor to version 4.14.1
Status: Reviewed & tested by the community » Fixed

Cherry-picked to 8.9.x as front end framework manager @lauriii +1'd too.

  • alexpott committed ae04062 on 8.9.x
    Issue #3155110 by maacl, Mykola Veryha: Update CKEditor to version 4.14....

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

xjm’s picture

Ouch -- this should have been in the 8.9.3 and 9.0.3 release notes. Please always tag any dependency updates that get backported for the release notes.

catch’s picture

Title: Update CKEditor to version 4.14.1 » [backport] Update CKEditor to version 4.14.1
Version: 8.9.x-dev » 8.8.x-dev
Status: Closed (fixed) » Needs work

Also we should probably backport this to 8.8.x

catch’s picture

Status: Needs work » Fixed

Or not - 4.14.0 was a security release, 4.14.1 is not, so we're OK leaving 8.8.x on 4.14.0 for now.

MykolaVeryha’s picture

Could you write a reason why not? we have a bug with 4.14.0. You can see it above
Why we should not fix the bug? It's not okay to leave bugs.

catch’s picture

8.8.x only receives security fixes. The bug has been fixed in 9.0.x and 8.9.x and all Drupal 8 sites should be aiming to update to those versions ASAP.

See https://www.drupal.org/core/release-cycle-overview

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.