Fix disconnected editor not destroyed (#73)

* Fix editor disconnected not destroyed

* changelog

* eslint

* add tests

* Update src/test/ts/browser/LoadTest.ts

Co-authored-by: tiny-ben-tran <[email protected]>

* Update src/test/ts/browser/LoadTest.ts

Co-authored-by: tiny-ben-tran <[email protected]>

---------

Co-authored-by: a.dmitryuk <[email protected]>
Co-authored-by: tiny-ben-tran <[email protected]>

Author: 3 people

CHANGELOG.md

@@ -8,6 +8,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Fixed
 - Editor initialization failure error handling. #INT-3366
+- Disconnected Editor not destroyed.
 
 ## 2.3.1 - 2025-08-11
 

src/main/ts/component/Editor.ts

@@ -338,6 +338,9 @@ class TinyMceEditor extends HTMLElement {
   disconnectedCallback(): void {
     this._mutationObserver.disconnect();
     this._updateForm();
+    if (this._editor) {
+      this._getTinymce().remove(this._editor);
+    }
   }
 
   get value(): string | null {

src/test/ts/browser/LoadTest.ts

@@ -33,13 +33,18 @@ UnitTest.asynctest('LoadTest', (success, failure) => {
     }),
     Step.sync(() => makeTinymceElement({
       'setup': 'customElementTinymceSetup',
-      'on-init': 'customElementTinymceInit'
+      'on-init': 'customElementTinymceInit',
+      'id': 'example_id'
     }, '<p>Hello world</p>')),
     Waiter.sTryUntilPredicate('Waiting for editor setup', () => seenSetup),
     Waiter.sTryUntilPredicate('Waiting for editor init', () => seenInit),
     Step.sync(() => {
       Assertions.assertHtmlStructure('', '<p>Hello world</p>', editorInstance.getContent() as string);
+      Assertions.assertEq('An editor instance is registered', true, Global.tinymce.get('example_id') !== null);
     }),
-    Step.sync(() => removeTinymceElement())
+    Step.sync(() => removeTinymceElement()),
+    Step.sync(() => {
+      Assertions.assertEq('The editor instance is removed', true, Global.tinymce.get('example_id') === null);
+    })
   ], success, failure);
 });