Problem/Motivation

The CKEditor Read More module needs updates to work correctly with Drupal 11
and CKEditor 5.

The existing implementation primarily targets the legacy CKEditor integration
and does not provide a complete Drupal CKEditor 5 plugin configuration.

Proposed resolution

Update the module to support Drupal 10 and Drupal 11 with a CKEditor 5
integration while preserving the existing front-end Read More/Read Less
behavior.

Issue fork ckeditor_rm-3623201

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:

About issue forks

Comments

dency mary a created an issue. See original summary.

dency mary a opened merge request !1

dency mary a’s picture

dency mary a commented
Status: Active » Needs review

The Drupal 11 compatibility changes are ready for review.

Changes include:
- CKEditor 4 to CKEditor 5 migration.
- Drupal 11 compatibility updates.
- Updated libraries, CSS, JavaScript, and module configuration.
- Removed the legacy CKEditor 4 implementation.

The changes have been tested locally on Drupal 11.

merge request !1

shahul hameed’s picture

shahul hameed commented

Tested the MR locally on Drupal 11.4.6.

Testing performed:

* Enabled `ckeditor_rm` successfully on Drupal 11.
* Verified the CKEditor 5 plugin definition is discovered correctly.
* Added the Read more button to the CKEditor 5 toolbar.
* Created content using the Read more functionality.
* Verified the content saves and displays correctly.
* Verified Read more expands the hidden content.
* Verified Read less hides the content again.
* Edited and re-saved existing Read more content and confirmed the functionality still works.
* Reviewed the updated CSS/JS behavior.
* Ran `git diff --check` with no errors.

The proposed Drupal 11 / CKEditor 5 functionality is working as expected in my testing.

shahul hameed’s picture

shahul hameed commented
Status: Needs review » Fixed

The Drupal 11 compatibility changes have been reviewed and tested.

MR !1 has been merged as commit `8b6d79d5` on `8.x-1.x-dev`.

Thanks to everyone involved in the implementation and review.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

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