Problem/Motivation

Per the following on the contrib ckeditor module for CKEditor 4, it is no longer supported in Drupal and should no longer be used or maintained.

To ease the transition to CKEditor 5 (in Drupal 9.5 and Drupal 10.0), it is possible to use the 1.0.x branch of this project until contributed and custom modules are ready. Once CKEditor 4 no longer receives security updates (the end of 2023), this project will be marked unsupported.

All implementation for CKEditor 4 and related logic/configuration/testing should be removed from the module.

Steps to reproduce

  • Go to patternkit JSON Editor settings
  • CKEditor 4 should no longer be an available option (regardless whether the drupal/ckeditor module is available

Proposed resolution

  • Remove all configuration options to use CKEditor 4 or related toolbars
  • Update existing configuration to remove these settings if used
  • Remove related logic to attach CKEditor 4

User interface changes

  • CKEditor 4 is no longer an option for usage in Patternkit content

API changes

None

Data model changes

None

CommentFileSizeAuthor
#5 DAT-5290_Retest.odt506.78 KBminsharm

Issue fork patternkit-3416535

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

slucero created an issue. See original summary.

slucero’s picture

Issue summary: View changes
Status: Active » Needs review
slucero’s picture

minsharm’s picture

StatusFileSize
new506.78 KB

Retested the issue and CKEditor 4 is not longer available for Patternkit configuration

Steps to test :

Install both Ckeditor 4 and ckeditor 5
Go to patternkit JSON Editor settings.

Results : CKEditor 4 is no longer an available option (regardless whether the drupal/ckeditor module is available).

Attaching screenshots.

  • slucero committed adb1ac0a on 9.1.x
    Issue #3416535 by slucero, minsharm: Remove Support for CKEditor 4
    
slucero’s picture

Status: Needs review » Fixed

Merged for inclusion in the Beta 9 release.

Status: Fixed » Closed (fixed)

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