Linking in CKEditor 5: URLs with top-level domain but without protocol should get a protocol added automatically

#17 - exactly

"Let's overengineering" is the tagline of Drupal.

Definitively outdated since 📌 [11.x] Remove EditorLinkDialog, EditorImageDialog and EditorMediaDialog in Drupal 11 Fixed .

But … also still an issue with CKEditor 5. I just tried it and was able to reproduce the same problem.

Fortunately, they have infrastructure in place that makes it easy to achieve this: https://ckeditor.com/docs/ckeditor5/latest/features/link.html#adding-def...

This is so absurdly low-hanging fruit that I immediately created a merge request and screencast for it. It's 2 lines of code. IMHO it doesn't even need tests, because the CKEditor 5 team obviously is testing this already.

That's great, thanks for the MR.
Surprised ckeditor doesn't have a default value in the first place, and agree that it would be silly to add test coverage for a 3rd party library.

Anyone know if there's a similar issue open for the link widget already?
That's another super annoying UX / validation constraint when protocol is missing.

nod_ → closed merge request !6969

Fixing a 7 years old issue is nice, especially with a diff that's 4 lines

Committed e84fc60 and pushed to 11.x. Thanks!

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