CKEditor could not be loaded. You can edit the HTML directly instead.

[jamesmacwhite]

Describe the bug

When using 5.1.0 for the new rich text option for a cell, the following error is raised:

CKEditor could not be loaded. You can edit the HTML directly instead.

In Console the following error is raised:

RichTextCellDialog.ts:206 TypeError: Failed to resolve module specifier '@craftcms/ckeditor'
    at RichTextCellDialog.ts:37:38
    at preload-helper-uBIymjUX.js:1:1203
    at async Promise.all (index 0)
    at async n (RichTextCellDialog.ts:44:39)
    at async r.createEditor (RichTextCellDialog.ts:159:41)

The CKEditor plugin is installed.

This was tested using a fresh table which is within a matrix block. I don't know if that has any relevance to the issue.

I have resaved the table field within the matrix block, the only changes are under the settings key in project config for the field.

Steps to reproduce

1. Update to 5.1.0
2. Create table in a matrix block with the new rich text option
3. The modal shows CKEditor could not be loaded. You can edit the HTML directly instead.

Craft CMS version

5.11.3

Plugin version

5.1.0

Multi-site?

Yes

Additional context

No response

[engram-design]

Can you confirm what CKEditor plugin version you're on? I imagine the latest?

[jamesmacwhite]

Ahhhhh, yeah that will be it. I'm on 4.x currently because of the mess with 5.x and the changing of per-field options no long being possible via the previous config method, I've deferred upgrading, but I guess this will force me to!

[jamesmacwhite]

Just to confirm, upgrading to 5.x resolves the error, but I guess, tablemaker might want to require CKEditor 5.x from 5.1.0 and above?

I guess it is tricky though, because rich text is an option not a requirement.

[engram-design]

Oh gotcha, yeah we can take a look at 4.x I just assumed by now people had gotten through the controversy! But failing support for both versions (which I kinda don’t want to do as they are different editor versions) we can completely make this better notification wise.

[jamesmacwhite]

No worries, I have avoided it, but I have just done the upgrade on dev now.

I think the main issue is that the configs get migrated, but now they are all independent by default until you implement an additional measure to link them back to the original shared config approach, which can be done, but just requires more work than before in some cases.

I'll just need to look at the transition with the previous markdown approach I used, currently multi-line type with markdown to basically implement rich text before this.

[engram-design]

Just looked into 4.x support anyway, and yeah it's a full, separate bit of functionality, so likely not going to do that. Regardless, updated the handling for the next release so at least it doesn't throw that error.

To get this early, run composer require verbb/tablemaker:"dev-craft-5 as 5.1.0".

Sorry to cause you to forcibly update CKEditor!

[jamesmacwhite]

Thanks! No worries, not really forced to be fair, just having to bite the bullet, but we needed to do it. The update notification was bothering me ha ha. The config stuff isn't that bad, but it just requires a bit more work/alternatives. I left a lot of other Craft CMS users to test that one and figure out the issues.

That change will certainly help for any other CKEditor 4.x hold outs though, to be fair it was handled to a point already, just not obvious about being specific 4.x related.

I have my own migration needed for the markdown workaround we've been doing in cells, but the native rich text support is awesome!

Non devs hate markdown and probably even some devs do too lol.