Use delayIfDetached config for editor set to true by default

[sculpt0r]

Are you reporting a feature request or a bug?

Feature

Base on our investigation on ckeditor/ckeditor4#4390 issue and fix we provide for it: ckeditor/ckeditor4#4463. We should use the delayIfDetached config set to true by default in integrations, just to prevent any issue. Also, we need to adjust editor creation: to properly mantain editor data (ckeditor/ckeditor4#4390 (comment))

ckeditor4-angular/src/ckeditor/ckeditor.component.ts

Lines 302 to 304 in a6be961

	const instance: CKEditor4.Editor = this.type === CKEditor4.EditorType.INLINE
	? CKEDITOR.inline( element, this.config )
	: CKEDITOR.replace( element, this.config );

The above line assumes we get editor instance, but that doesn't have to be true with delayed creation.

Expected result

What is the expected result of the above steps?

Actual result

What is the actual result of the above steps?

Other details

• Browser: -
• OS: -
• Integration version: -
• CKEditor version: -
• Installed CKEditor plugins: -

[Dumluregn]

I suppose this should be done after the release of CKEditor with ckeditor/ckeditor4#4463, so when 4.17.0 is out? And we should mark it as blocked for now (here and in the other integration repos)?

[sculpt0r]

Actually, it can be done without ckeditor/ckeditor4#4463. It will help prevent errors until reattach happen. So very partially, but still ;)

And indeed, only after 4.17.0 release.

So I mark all of them as blocked until the next major release.

[sculpt0r]

Since https://github.com/ckeditor/ckeditor4/releases/tag/4.17.0 it is no longer blocked.

[CKEditorBot]

Closed in #222