Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
Add buttons for CKEditor5
Comment | File | Size | Author |
---|---|---|---|
#7 | 3293130-upgrade-path-7.patch | 6.98 KB | Wim Leers |
Comments
Comment #2
cegri CreditAttribution: cegri as a volunteer commentedIs this feature planned / scheduled?
Comment #3
urix CreditAttribution: urix commentedI'd love to see the integration too.
IMCE is a great tool for image management.
Comment #4
Wim LeersAs https://www.drupal.org/project/usage/imce shows, IMCE usage is dwindling significantly. People are migrating to Drupal core's Media Library. 80% of the user base (all versions of the IMCE module except
8.x-2.x
) is on legacy Drupal versions: 6, 7 and 8. Only 20% (60K sites today) are on Drupal 9.This shows that the Drupal ecosystem is moving on to Media Library.
With the https://www.drupal.org/project/media_migration module, doing that is easy (for D7 → D9).
I think our time would be far better spent getting the entire Drupal ecosystem on Media Library rather than working on CKEditor 5 support for IMCE.
Comment #5
Wim LeersThis will be critical once Drupal 10 is out, which is … soon.
Comment #6
ufku CreditAttribution: ufku commentedReleased 3.0.0 which is compatible with Drupal 10 and has ckeditor5 integration.
Comment #7
Wim LeersAwesome work in https://git.drupalcode.org/project/imce/-/commit/f7dcab09281eb6e30095179...! 👏
I do see in https://git.drupalcode.org/project/imce/-/tree/3.x/src/Plugin that the CKEditor 4 and 5 plugins are both present in the same version — that's great! That enables sites to easily transition.
What would make things even better though, is if there was an automated update path. For that, you'd need to add a
CKEditor4To5Upgrade
plugin, which would be really simple in this caseImceImage
(CKE4) →imce_image
(CKE5)ImceLink
(CKE4) →imce_link
(CKE5)Ideally you'd also add test coverage to prove that the upgrade path works.
Patch attached that does all of that.
Comment #9
ufku CreditAttribution: ufku commentedCommitted, thanks.