I installed latest CKEditor in my libraries folder (4.7.3.dee99e2) and updated WYSIWYG to latest version: 7.x-2.4. This version is however "not verified". Found this issue https://www.drupal.org/node/2851200 and it seems like the exact same thing. Here is a patch to allow version 4.7.3 to also be verified.

If this issue should be researched, written, posted and contributed in a different way, please be forgiving and tell me how to do better. I am willing to learn and improve how I go about posting issues and patches.

CommentFileSizeAuthor
#4 wysiwyg_support_ckeditor_4.8.0.230f715_4.patch588 byteslily.yan
wysiwyg-181465.patch588 bytesleanderl
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

leanderl created an issue. See original summary.

leanderl’s picture

leanderl CreditAttribution: leanderl commented
Issue summary: View changes
kpaxman’s picture

kpaxman CreditAttribution: kpaxman commented

Have you tested to confirm that it still works? I note that the previous ticket did say some testing had been done.

lily.yan’s picture

lily.yan CreditAttribution: lily.yan commented
FileSize
wysiwyg_support_ckeditor_4.8.0.230f715_4.patch588 bytes

I installed latest CKEditor in my libraries folder (4.8.0.230f715) and updated WYSIWYG to latest version: 7.x-2.x(233670fb).
Here is a patch to allow version 4.8.0 to also be verified. I did test it works.

I have created a new issue regarding 4.8.0.230f715 (https://www.drupal.org/project/wysiwyg/issues/2916245).

kpaxman’s picture

kpaxman CreditAttribution: kpaxman commented
Status: Active » Needs review

  • TwoD committed 61e1521 on 6.x-2.x 
    - #2916245, #2951295 by leanderl, lily.yan, jkdev, kpaxman, aitala:...
  • TwoD committed 86c1683 on 7.x-2.x
    - #2916245, #2951295 by leanderl, lily.yan, jkdev, kpaxman, aitala:...
TwoD’s picture

TwoD
Primary language Swedish
Location Sweden
CreditAttribution: TwoD as a volunteer commented
Status: Needs review » Fixed
Related issues: +#2951295: Support for CKEditor 4.8.0.230f715

Thank you for reporting, patching and testing this!
There will be a new release marking 4.9.2 as verified due to the recent security issue in some earlier versions.

Status: Fixed » Closed (fixed)

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

Liam Morland’s picture

Liam Morland
Primary language English
Location Ontario, CA 🇨🇦
CreditAttribution: Liam Morland at University of Waterloo commented
Related issues: -#2951295: Support for CKEditor 4.8.0.230f715