Closed (works as designed)
Project:
Drupal core
Version:
9.5.x-dev
Component:
ckeditor5.module
Priority:
Normal
Category:
Plan
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
8 Apr 2021 at 17:25 UTC
Updated:
29 Aug 2022 at 10:48 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
gábor hojtsyComment #3
zrpnr@baddysonja suggested some of the CKEditor projects maintained by 1xINTERNET may be a good option for converting.
I have not evaluated any of these, but at a glance it seems like a few may not apply to CK5 and there are some which may be better handled by a different plugin. For example emoji could be special characters and codesnippet may be covered with either Code (included in the main project) or CodeBlock
Comment #4
gábor hojtsyRemoving the inappropriate tag. Drupal 10 should be enough.
Comment #5
wim leersGreat idea! 👍
But I'd suggest that instead of focusing on CKE4 modules contributed by a single organization, we should focus on the most widely used ones?
Comment #6
wim leersComment #7
wim leerseditor_advanced_link(#3232052: Drupal 10 & CKEditor 5 readiness) is ready! 🥳Comment #8
wim leerslinkitis almost ready, it's now fully working, see #3232190-14: CKEditor 5 readiness! 🚀Comment #9
luke.leberWith nearly 70,000 installs, the embed ecosystem as a whole is likely a non-trivial endeavor that could really stress test the existing API.
There are also integration complexities with things like Entity Browser that may catch edge cases.
Just an idea.
Comment #10
wim leers@Luke.Leber: Agreed! However, I'm far less concerned there. Rather than overriding existing functionality, it's completely independent functionality. CKEditor 5 is already integrated with Media Library and renders media embeds just fine. That is for Drupal core, not for https://www.drupal.org/project/entity_embed. But … that Drupal core functionality was actually a simplified port of https://www.drupal.org/project/entity_embed into Drupal core! It was hardened in contrib before going into Drupal core. Examples: #3064340: Make preview responses cacheable to accelerate previews, #2844822: The preview in CKEditor does not use the same Twig template as the one on the front end (default theme), #3064288: Only upcast `<drupal-entity>`, not any tag that has the appropriate `data-` attributes and many more.
So, while we definitely should have that ready, it's less likely it will find more edge cases.
Unfortunately, that module is de facto unmaintained 😱I am a co-maintainer for it simply because I had to help stabilize it before extracting the best parts of it for the media integration in Drupal core.
Any chance you're interested in becoming a co-maintainer, and start the CKEditor 5 port? 🤓😃 We'd definitely be able to give you any and all help you'd want or need! That'd allow us to continue working on stabilizing the CKEditor 5 module itself further for more edge cases.
Comment #11
luke.leberHey Wim, I don't think that I need to assume the mantle of co-maintainer to be able to contribute to entity embed. I probably won't have time til closer to Christmas, but will keep it on my back burner.
We have a unique use case in that we're running a customized embed formatter plugin (since long before core media embed support came to ckeditor4) and I'm eager to see if this can continue to be supported or if we'll have to figure something else out before cke4 support drops. It would be really nice if the upgrade path for folks like us that had to customize things before core media landed would be seamless.
I'll try to follow the same path as linkit and editor advanced link did and see what shakes out.
Comment #12
wim leersThat does sound interesting :)
That customized embed formatted plugin runs on the server side. The CKEditor code just provides a blank canvas for rendering. So I’m pretty sure you’ll be fine.
I think the key question here will be whether h you want the nicer UX without a Drupal dialog (more work), or the current UX. The current UX should be easy to port, and there already is an example in the form of the Media Library button.
Good luck and keep us posted!
Comment #13
wim leersComment #15
damienmckennaAdding the "ckeditor5" tag to make it easier to find other issues.
Comment #16
wim leersSemi-blocking CKEditor 5 issue: https://github.com/ckeditor/ckeditor5/issues/11128.
Comment #18
dercheffeHi,
I would vote for this Drupal module https://www.drupal.org/project/ckeditor_emojione too :)
There are many dependencies in the emoji module tho, compared to the emojione module.
Comment #19
WebbehCKEditor Font (#3239667: Drupal 10 & CKEditor 5 readiness) seems, at face-value, to be a (relatively?) simple port from 4->5, being a ckeditor core plugin that has a pretty light configuration. I'm also a co-maintainer, and would be happy to help on this in what ways I can.
Comment #20
wim leers#18: Why not rely on the OS-level emoji picker? Anyway, a quick search reveals https://ckeditor.com/docs/ckeditor5/latest/features/special-characters.h... — so supporting this should be trivial, since Drupal core already ships with the
SpecialCharacterspackage, it just doesn't load the emoji-specific plugin in that package yet:→ you'd need to add
SpecialCharactersEmojithere and configure it correctly. The contrib module should be simple to port (can you please create an issue in that issue queue? 🙏), but IMHO it should not be ported: content creators should start using the OS-level emoji picker. Every OS has shipped with that for years now.#19: 🥳 Great news! Commented over at #3239667-9: Drupal 10 & CKEditor 5 readiness 😊
Comment #21
dercheffeAgree completely with the "more native solution" in #20. In the example in the ckeditor5-docs is the emoji is part of the "special chars ckeditor5-bar. Then I would vote for a separate button like a "🙂". Best would be inside there also emoji categories, like common in several messengers.
Comment #22
wim leersI think you're contradicting yourself there? On the one hand you agree with using the native emoji picker, on the other you advocate for an even more custom UI? I'm confused now :D
Comment #23
dercheffeWim, in the example editor for the emoji's here the emoji's are inside the "special chars" (the omega sign). Then the user has to select the emoji's category, if he only wants emojis. IMO it's a bad UX (see the attached screenshot).
I just wanted to say it would be better for UX to separate the emoji's in an extra "button" (or however it's called 😅) in the editor's bar. The "native" technique behind the scenes is good as it it (so not necessary to implement 3rd party libraries like emojione for example)
Comment #24
wim leersWhen you say “native technique”, do you mean native to the CKEditor 5 Special Characters plug-in? Or do you mean native to the operating system? 🤓
Comment #25
wim leersLots of projects are now well on their way. And some are even looking into merging duplicate efforts: #3297106: ckeditor_font merging with ckeditor5_font, colorbutton! 🥳