Better discovery of DX CKE5 debug documentation

Usually I would be cautious to add console.log/warn/error, but since there is a console.error already, I don't think that should be a problem. Some other thoughts:

• Like you say, there is a risk in hard-coding documentation URLs. I don't know if there is an official policy for this (maybe we should discuss this with the documentation team?). Links to drupal.org in the current codebase mostly point to issues, not documentation. Those URLs tend to use the drupal.org/node/[nid] form instead of the aliased URL.
• If we decide to do this, it should also be mentioned on the documentation page itself that it is linked to from code, to avoid someone accidentally breaking the link or (re)moving the content.
• If the message recommends using an "unminified version of CKeditor", then as a user I expect to find these exact words on the documentation page. At the moment there is a disconnect between the (proposed) console message and the documentation, which speaks about "Build tools" and "Installing CKEditor 5 from source". It left me wondering if "installing from source" and "using an unminified version" are the same thing.
• The added message is not really a warning. IMHO console.log() or console.info() is more appropriate.

#2: love it! 🤩

Great feedback by @marcvangend, and already addressed by @Dom. — fantastic! I think there is zero chance of the link breaking because it's using the node ID, not the path alias.

This is unquestionably a DX improvement.

RTBC! (Only slightly clarified the handbook page 🤓)

+++ b/core/modules/ckeditor5/js/ckeditor5.es6.js
@@ -469,6 +469,7 @@
+          console.info('Debugging can be done with an unminified version of CKEditor by installing from the source file. Consult documentation at https://www.drupal.org/node/3258901')

The documentation page this references outdated The

yarn dll:build --dev

command was removed a while back. There is currently no command in core to build an unminified version.

@bnjmnm But then … wouldn't it be better to still commit this and then update the handbook page later? That would at least point users in a helpful direction? 😊

Also, are you sure? @Dom. actually used this earlier today. And it sure seems to check out 🤓

Based on what I can see, it seems to confirm @Dom.'s experience:

• https://github.com/ckeditor/ckeditor5/blob/v33.0.0/package.json#L171
• → https://github.com/ckeditor/ckeditor5/blob/master/scripts/dll/build-dlls...
• → https://github.com/ckeditor/ckeditor5/blob/master/scripts/dll/webpack.co...
• → https://github.com/ckeditor/ckeditor5/blob/master/scripts/dll/webpack.co...

wouldn't it be better to still commit this and then update the handbook page later? That would at least point users in a helpful direction?

If it pointed them in a helpful direction, yes. In this case they're directed to an outdated documentation page that has objectively incorrect information. It's bad enough this page is up to begin with. If someone is directed there intentionally, it gives them all the more reason to believe that something is wrong on their end, not the documentation. They could waste time trying to get a feature to work that doesn't actually exist.

Oh this is for installing an entire separate CKEditor, not an unminified version in core. That's my mistake.
At one when this was in contrib we were doing more direct interaction with dll and could create an unminified instance pretty easily. It kinda looked like that so I decided it was that 🤭. If this process works with the core plugins it sounds fine, especially if it benefits the dev days participation I've been pleased to see happening.

Needs reroll for 10.1

Rerolled for 10

Think this is a random failure.

Why the string split? it's not usual to do that in the JS

Thought I would be dinged for the 80 character limit but think I got that mixed up. Uploaded a patch with no breaks.

all good for me :)

Committed and pushed 58b45dcb6f to 10.1.x and 8bed000244 to 10.0.x. Thanks!

We could backport this to Drupal 9 if people want to.

Let's do that. Many people won't be jumping to 10, but will be making the small step to 9.5, and hence will be developing/testing CKEditor 5 in there, so … 🙏

Committed dfacc85 and pushed to 9.5.x. Thanks!