@ckeditor/ckeditor5-engine 39.0.2 → 40.1.0

package/LICENSE.md

@@ -2,7 +2,7 @@ Software License Agreement
 ==========================
 
 **CKEditor&nbsp;5 editing engine** – https://github.com/ckeditor/ckeditor5-engine <br>
-Copyright (c) 2003-2023, [CKSource Holding sp. z o.o.](https://cksource.com) All rights reserved.
+Copyright (c) 2003–2023, [CKSource Holding sp. z o.o.](https://cksource.com) All rights reserved.
 
 Licensed under the terms of [GNU General Public License Version 2 or later](http://www.gnu.org/licenses/gpl.html).
 
@@ -13,9 +13,9 @@ Where not otherwise indicated, all CKEditor content is authored by CKSource engi
 
 The following libraries are included in CKEditor under the [MIT license](https://opensource.org/licenses/MIT):
 
-* lodash - Copyright (c) JS Foundation and other contributors https://js.foundation/. Based on Underscore.js, copyright Jeremy Ashkenas, DocumentCloud and Investigative Reporters & Editors http://underscorejs.org/.
+* Lodash - Copyright (c) JS Foundation and other contributors https://js.foundation/. Based on Underscore.js, copyright Jeremy Ashkenas, DocumentCloud and Investigative Reporters & Editors http://underscorejs.org/.
 
 Trademarks
 ----------
 
-**CKEditor** is a trademark of [CKSource Holding sp. z o.o.](https://cksource.com) All other brand and product names are trademarks, registered trademarks or service marks of their respective holders.
+**CKEditor** is a trademark of [CKSource Holding sp. z o.o.](https://cksource.com) All other brand and product names are trademarks, registered trademarks, or service marks of their respective holders.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ckeditor/ckeditor5-engine",
-  "version": "39.0.2",
+  "version": "40.1.0",
   "description": "The editing engine of CKEditor 5 – the best browser-based rich text editor.",
   "keywords": [
     "wysiwyg",
@@ -23,7 +23,7 @@
   ],
   "main": "src/index.js",
   "dependencies": {
-    "@ckeditor/ckeditor5-utils": "39.0.2",
+    "@ckeditor/ckeditor5-utils": "40.1.0",
     "lodash-es": "4.17.21"
   },
   "author": "CKSource (http://cksource.com/)",

package/src/controller/datacontroller.d.ts

@@ -45,7 +45,7 @@ export default class DataController extends DataController_base {
     readonly model: Model;
     /**
      * Mapper used for the conversion. It has no permanent bindings, because these are created while getting data and
-     * ae cleared directly after the data are converted. However, the mapper is defined as a class property, because
+     * are cleared directly after the data are converted. However, the mapper is defined as a class property, because
      * it needs to be passed to the `DowncastDispatcher` as a conversion API.
      */
     readonly mapper: Mapper;

package/src/conversion/downcastdispatcher.js

@@ -258,7 +258,7 @@ export default class DowncastDispatcher extends EmitterMixin() {
     _convertInsert(range, conversionApi, options = {}) {
         if (!options.doNotAddConsumables) {
             // Collect a list of things that can be consumed, consisting of nodes and their attributes.
-            this._addConsumablesForInsert(conversionApi.consumable, Array.from(range));
+            this._addConsumablesForInsert(conversionApi.consumable, range);
         }
         // Fire a separate insert event for each node and text fragment contained in the range.
         for (const data of Array.from(range.getWalker({ shallow: true })).map(walkerValueToEventData)) {

package/src/conversion/downcasthelpers.js

@@ -1810,16 +1810,11 @@ function normalizeModelElementConfig(model) {
     if (typeof model == 'string') {
         model = { name: model };
     }
-    // List of attributes that should trigger reconversion.
-    if (!model.attributes) {
-        model.attributes = [];
-    }
-    else if (!Array.isArray(model.attributes)) {
-        model.attributes = [model.attributes];
-    }
-    // Whether a children insertion/deletion should trigger reconversion.
-    model.children = !!model.children;
-    return model;
+    return {
+        name: model.name,
+        attributes: model.attributes ? toArray(model.attributes) : [],
+        children: !!model.children
+    };
 }
 /**
  * Takes `config.view`, and if it is an {@link module:engine/view/elementdefinition~ElementDefinition}, converts it

package/src/conversion/upcastdispatcher.js

@@ -187,7 +187,7 @@ export default class UpcastDispatcher extends EmitterMixin() {
         const documentFragment = writer.createDocumentFragment();
         // When there is a conversion result.
         if (modelRange) {
-            // Remove all empty elements that were create while splitting.
+            // Remove all empty elements that were created while splitting.
             this._removeEmptyElements();
             // Move all items that were converted in context tree to the document fragment.
             for (const item of Array.from(this._modelCursor.parent.getChildren())) {

package/src/conversion/viewconsumable.js

@@ -430,11 +430,15 @@ export class ViewElementConsumables {
                  *
                  * What you have done is trying to use:
                  *
-                 *		consumables.add( { attributes: [ 'class', 'style' ] } );
+                 * ```ts
+                 * consumables.add( { attributes: [ 'class', 'style' ] } );
+                 * ```
                  *
                  * While each class and style should be registered separately:
                  *
-                 *		consumables.add( { classes: 'some-class', styles: 'font-weight' } );
+                 * ```ts
+                 * consumables.add( { classes: 'some-class', styles: 'font-weight' } );
+                 * ```
                  *
                  * @error viewconsumable-invalid-attribute
                  */

package/src/dataprocessor/basichtmlwriter.js

@@ -2,6 +2,7 @@
  * @license Copyright (c) 2003-2023, CKSource Holding sp. z o.o. All rights reserved.
  * For licensing, see LICENSE.md or https://ckeditor.com/legal/ckeditor-oss-license
  */
+import { global } from '@ckeditor/ckeditor5-utils';
 /**
  * Basic HTML writer. It uses the native `innerHTML` property for basic conversion
  * from a document fragment to an HTML string.
@@ -11,7 +12,7 @@ export default class BasicHtmlWriter {
      * Returns an HTML string created from the document fragment.
      */
     getHtml(fragment) {
-        const doc = document.implementation.createHTMLDocument('');
+        const doc = global.document.implementation.createHTMLDocument('');
         const container = doc.createElement('div');
         container.appendChild(fragment);
         return container.innerHTML;

package/src/index.d.ts

@@ -30,7 +30,7 @@ export { default as RootAttributeOperation } from './model/operation/rootattribu
 export { default as RootOperation } from './model/operation/rootoperation';
 export { default as NoOperation } from './model/operation/nooperation';
 export { transformSets } from './model/operation/transform';
-export { default as DocumentSelection, type DocumentSelectionChangeRangeEvent } from './model/documentselection';
+export { default as DocumentSelection, type DocumentSelectionChangeRangeEvent, type DocumentSelectionChangeMarkerEvent } from './model/documentselection';
 export { default as Range } from './model/range';
 export { default as LiveRange, type LiveRangeChangeRangeEvent } from './model/liverange';
 export { default as LivePosition } from './model/liveposition';
@@ -47,7 +47,7 @@ export type { Marker } from './model/markercollection';
 export type { default as Batch } from './model/batch';
 export type { default as Differ, DiffItem, DiffItemAttribute, DiffItemInsert, DiffItemRemove } from './model/differ';
 export type { default as Item } from './model/item';
-export type { default as Node } from './model/node';
+export type { default as Node, NodeAttributes } from './model/node';
 export type { default as RootElement } from './model/rootelement';
 export type { default as Schema, SchemaAttributeCheckCallback, SchemaChildCheckCallback, AttributeProperties, SchemaItemDefinition } from './model/schema';
 export type { default as Selection, Selectable } from './model/selection';
@@ -74,7 +74,7 @@ export { default as ViewRawElement } from './view/rawelement';
 export { default as ViewUIElement } from './view/uielement';
 export { default as ViewDocumentFragment } from './view/documentfragment';
 export { default as ViewTreeWalker, type TreeWalkerValue as ViewTreeWalkerValue } from './view/treewalker';
-export type { default as ViewElementDefinition } from './view/elementdefinition';
+export type { default as ViewElementDefinition, ElementObjectDefinition } from './view/elementdefinition';
 export type { default as ViewDocumentSelection } from './view/documentselection';
 export { default as AttributeElement } from './view/attributeelement';
 export type { default as ViewItem } from './view/item';

package/src/model/batch.d.ts

@@ -17,7 +17,7 @@ import type Operation from './operation/operation';
  * @see module:engine/model/model~Model#enqueueChange
  * @see module:engine/model/model~Model#change
  */
-export default class Batch {
+export default class Batch implements BatchType {
     /**
      * An array of operations that compose this batch.
      */

package/src/model/document.js

@@ -192,8 +192,7 @@ export default class Document extends EmitterMixin() {
      * @param includeDetached Specified whether detached roots should be returned as well.
      */
     getRoots(includeDetached = false) {
-        return Array.from(this.roots)
-            .filter(root => root != this.graveyard && (includeDetached || root.isAttached()) && root._isLoaded);
+        return this.roots.filter(root => root != this.graveyard && (includeDetached || root.isAttached()) && root._isLoaded);
     }
     /**
      * Used to register a post-fixer callback. A post-fixer mechanism guarantees that the features

package/src/model/markercollection.d.ts

@@ -107,10 +107,10 @@ export default class MarkerCollection extends MarkerCollection_base implements I
      * Iterates over all markers that starts with given `prefix`.
      *
      * ```ts
-     * const markerFooA = markersCollection.set( 'foo:a', rangeFooA );
-     * const markerFooB = markersCollection.set( 'foo:b', rangeFooB );
-     * const markerBarA = markersCollection.set( 'bar:a', rangeBarA );
-     * const markerFooBarA = markersCollection.set( 'foobar:a', rangeFooBarA );
+     * const markerFooA = markersCollection._set( 'foo:a', rangeFooA );
+     * const markerFooB = markersCollection._set( 'foo:b', rangeFooB );
+     * const markerBarA = markersCollection._set( 'bar:a', rangeBarA );
+     * const markerFooBarA = markersCollection._set( 'foobar:a', rangeFooBarA );
      * Array.from( markersCollection.getMarkersGroup( 'foo' ) ); // [ markerFooA, markerFooB ]
      * Array.from( markersCollection.getMarkersGroup( 'a' ) ); // []
      * ```

package/src/model/markercollection.js

@@ -190,10 +190,10 @@ export default class MarkerCollection extends EmitterMixin() {
      * Iterates over all markers that starts with given `prefix`.
      *
      * ```ts
-     * const markerFooA = markersCollection.set( 'foo:a', rangeFooA );
-     * const markerFooB = markersCollection.set( 'foo:b', rangeFooB );
-     * const markerBarA = markersCollection.set( 'bar:a', rangeBarA );
-     * const markerFooBarA = markersCollection.set( 'foobar:a', rangeFooBarA );
+     * const markerFooA = markersCollection._set( 'foo:a', rangeFooA );
+     * const markerFooB = markersCollection._set( 'foo:b', rangeFooB );
+     * const markerBarA = markersCollection._set( 'bar:a', rangeBarA );
+     * const markerFooBarA = markersCollection._set( 'foobar:a', rangeFooBarA );
      * Array.from( markersCollection.getMarkersGroup( 'foo' ) ); // [ markerFooA, markerFooB ]
      * Array.from( markersCollection.getMarkersGroup( 'a' ) ); // []
      * ```

package/src/model/operation/operation.d.ts

@@ -2,12 +2,12 @@
  * @license Copyright (c) 2003-2023, CKSource Holding sp. z o.o. All rights reserved.
  * For licensing, see LICENSE.md or https://ckeditor.com/legal/ckeditor-oss-license
  */
-import type Batch from '../batch';
-import type Document from '../document';
-import type { Selectable } from '../selection';
 /**
  * @module engine/model/operation/operation
  */
+import type Batch from '../batch';
+import type Document from '../document';
+import type { Selectable } from '../selection';
 /**
  * Abstract base operation class.
  */

package/src/model/operation/operation.js

@@ -2,9 +2,6 @@
  * @license Copyright (c) 2003-2023, CKSource Holding sp. z o.o. All rights reserved.
  * For licensing, see LICENSE.md or https://ckeditor.com/legal/ckeditor-oss-license
  */
-/**
- * @module engine/model/operation/operation
- */
 /**
  * Abstract base operation class.
  */

package/src/model/operation/transform.js

@@ -2,6 +2,9 @@
  * @license Copyright (c) 2003-2023, CKSource Holding sp. z o.o. All rights reserved.
  * For licensing, see LICENSE.md or https://ckeditor.com/legal/ckeditor-oss-license
  */
+/**
+ * @module engine/model/operation/transform
+ */
 import InsertOperation from './insertoperation';
 import AttributeOperation from './attributeoperation';
 import RenameOperation from './renameoperation';
@@ -16,9 +19,6 @@ import Range from '../range';
 import Position from '../position';
 import { compareArrays } from '@ckeditor/ckeditor5-utils';
 const transformations = new Map();
-/**
- * @module engine/model/operation/transform
- */
 /**
  * Sets a transformation function to be be used to transform instances of class `OperationA` by instances of class `OperationB`.
  *

package/src/model/operation/utils.js

@@ -138,7 +138,11 @@ export function _normalizeNodes(nodes) {
                 convert(node);
             }
         }
-        // Skip unrecognized type.
+        else {
+            // Skip unrecognized type.
+            // eslint-disable-next-line @typescript-eslint/no-unused-vars
+            const unreachable = nodes;
+        }
     }
     convert(nodes);
     // Merge text nodes.

package/src/view/document.d.ts

@@ -128,6 +128,8 @@ export default class Document extends Document_base {
      * 		// Let other post-fixers know that something changed.
      * 		return true;
      * 	}
+     *
+     * 	return false;
      * } );
      * ```
      *
@@ -166,8 +168,6 @@ export type ViewDocumentPostFixer = (writer: DowncastWriter) => boolean;
  * * `children` - for child list changes,
  * * `attributes` - for element attributes changes,
  * * `text` - for text nodes changes.
- *
- * @typedef {String} module:engine/view/document~ChangeType
  */
 export type ChangeType = 'children' | 'attributes' | 'text';
 /**

package/src/view/document.js

@@ -79,6 +79,8 @@ export default class Document extends BubblingEmitterMixin(ObservableMixin()) {
      * 		// Let other post-fixers know that something changed.
      * 		return true;
      * 	}
+     *
+     * 	return false;
      * } );
      * ```
      *
@@ -98,7 +100,7 @@ export default class Document extends BubblingEmitterMixin(ObservableMixin()) {
      * Destroys this instance. Makes sure that all observers are destroyed and listeners removed.
      */
     destroy() {
-        this.roots.map(root => root.destroy());
+        this.roots.forEach(root => root.destroy());
         this.stopListening();
     }
     /**

package/src/view/documentfragment.d.ts

@@ -65,6 +65,10 @@ export default class DocumentFragment extends DocumentFragment_base implements I
      * Artificial element name. Returns `undefined`. Added for compatibility reasons.
      */
     get name(): undefined;
+    /**
+     * Artificial element getFillerOffset. Returns `undefined`. Added for compatibility reasons.
+     */
+    get getFillerOffset(): undefined;
     /**
      * Returns the custom property value for the given key.
      */

package/src/view/documentfragment.js

@@ -78,6 +78,12 @@ export default class DocumentFragment extends EmitterMixin(TypeCheckable) {
     get name() {
         return undefined;
     }
+    /**
+     * Artificial element getFillerOffset. Returns `undefined`. Added for compatibility reasons.
+     */
+    get getFillerOffset() {
+        return undefined;
+    }
     /**
      * Returns the custom property value for the given key.
      */

package/src/view/domconverter.d.ts

@@ -101,6 +101,10 @@ export default class DomConverter {
      * Matcher for inline object view elements. This is an extension of a simple {@link #inlineObjectElements} array of element names.
      */
     private readonly _inlineObjectElementMatcher;
+    /**
+     * Set of elements with temporary custom properties that require clearing after render.
+     */
+    private readonly _elementsWithTemporaryCustomProperties;
     /**
      * Creates a DOM converter.
      *
@@ -167,20 +171,22 @@ export default class DomConverter {
      * @param html Textual representation of the HTML that will be set on `domElement`.
      */
     setContentOf(domElement: DomElement, html: string): void;
-    /**
-     * Converts the view to the DOM. For all text nodes, not bound elements and document fragments new items will
-     * be created. For bound elements and document fragments the method will return corresponding items.
-     *
-     * @param viewNode View node or document fragment to transform.
-     * @param options Conversion options.
-     * @param options.bind Determines whether new elements will be bound.
-     * @param options.withChildren If `false`, node's and document fragment's children will not be converted.
-     * @returns Converted node or DocumentFragment.
-     */
-    viewToDom(viewNode: ViewNode | ViewDocumentFragment, options?: {
+    viewToDom(viewNode: ViewText, options?: {
+        bind?: boolean;
+        withChildren?: boolean;
+    }): DomText;
+    viewToDom(viewNode: ViewElement, options?: {
         bind?: boolean;
         withChildren?: boolean;
-    }): DomNode | DomDocumentFragment;
+    }): DomElement;
+    viewToDom(viewNode: ViewNode, options?: {
+        bind?: boolean;
+        withChildren?: boolean;
+    }): DomNode;
+    viewToDom(viewNode: ViewDocumentFragment, options?: {
+        bind?: boolean;
+        withChildren?: boolean;
+    }): DomDocumentFragment;
     /**
      * Sets the attribute on a DOM element.
      *
@@ -212,7 +218,7 @@ export default class DomConverter {
      * @param options See {@link module:engine/view/domconverter~DomConverter#viewToDom} options parameter.
      * @returns DOM nodes.
      */
-    viewChildrenToDom(viewElement: ViewElement, options?: {
+    viewChildrenToDom(viewElement: ViewElement | ViewDocumentFragment, options?: {
         bind?: boolean;
         withChildren?: boolean;
     }): IterableIterator<Node>;
@@ -477,6 +483,12 @@ export default class DomConverter {
      * @param pattern Pattern matching a view element which should be treated as an inline object.
      */
     registerInlineObjectMatcher(pattern: MatcherPattern): void;
+    /**
+     * Clear temporary custom properties.
+     *
+     * @internal
+     */
+    _clearTemporaryCustomProperties(): void;
     /**
      * Returns the block {@link module:engine/view/filler filler} node based on the current {@link #blockFillerMode} setting.
      */

package/src/view/domconverter.js

@@ -5,7 +5,7 @@
 /**
  * @module engine/view/domconverter
  */
-/* globals Node, NodeFilter, DOMParser, Text */
+/* globals Node, NodeFilter, DOMParser */
 import ViewText from './text';
 import ViewElement from './element';
 import ViewUIElement from './uielement';
@@ -16,7 +16,7 @@ import ViewDocumentFragment from './documentfragment';
 import ViewTreeWalker from './treewalker';
 import { default as Matcher } from './matcher';
 import { BR_FILLER, INLINE_FILLER_LENGTH, NBSP_FILLER, MARKED_NBSP_FILLER, getDataWithoutFiller, isInlineFiller, startsWithFiller } from './filler';
-import { global, logWarning, indexOf, getAncestors, isText, isComment, isValidAttributeName, first } from '@ckeditor/ckeditor5-utils';
+import { global, logWarning, indexOf, getAncestors, isText, isComment, isValidAttributeName, first, env } from '@ckeditor/ckeditor5-utils';
 const BR_FILLER_REF = BR_FILLER(global.document); // eslint-disable-line new-cap
 const NBSP_FILLER_REF = NBSP_FILLER(global.document); // eslint-disable-line new-cap
 const MARKED_NBSP_FILLER_REF = MARKED_NBSP_FILLER(global.document); // eslint-disable-line new-cap
@@ -70,6 +70,10 @@ export default class DomConverter {
          * Matcher for inline object view elements. This is an extension of a simple {@link #inlineObjectElements} array of element names.
          */
         this._inlineObjectElementMatcher = new Matcher();
+        /**
+         * Set of elements with temporary custom properties that require clearing after render.
+         */
+        this._elementsWithTemporaryCustomProperties = new Set();
         this.document = document;
         this.renderingMode = renderingMode;
         this.blockFillerMode = blockFillerMode || (renderingMode === 'editing' ? 'br' : 'nbsp');
@@ -230,57 +234,65 @@ export default class DomConverter {
             return this._domDocument.createTextNode(textData);
         }
         else {
-            if (this.mapViewToDom(viewNode)) {
-                return this.mapViewToDom(viewNode);
+            const viewElementOrFragment = viewNode;
+            if (this.mapViewToDom(viewElementOrFragment)) {
+                // Do not reuse element that is marked to not reuse (for example an IMG element
+                // so it can immediately display a placeholder background instead of waiting for the new src to load).
+                if (viewElementOrFragment.getCustomProperty('editingPipeline:doNotReuseOnce')) {
+                    this._elementsWithTemporaryCustomProperties.add(viewElementOrFragment);
+                }
+                else {
+                    return this.mapViewToDom(viewElementOrFragment);
+                }
             }
             let domElement;
-            if (viewNode.is('documentFragment')) {
+            if (viewElementOrFragment.is('documentFragment')) {
                 // Create DOM document fragment.
                 domElement = this._domDocument.createDocumentFragment();
                 if (options.bind) {
-                    this.bindDocumentFragments(domElement, viewNode);
+                    this.bindDocumentFragments(domElement, viewElementOrFragment);
                 }
             }
-            else if (viewNode.is('uiElement')) {
-                if (viewNode.name === '$comment') {
-                    domElement = this._domDocument.createComment(viewNode.getCustomProperty('$rawContent'));
+            else if (viewElementOrFragment.is('uiElement')) {
+                if (viewElementOrFragment.name === '$comment') {
+                    domElement = this._domDocument.createComment(viewElementOrFragment.getCustomProperty('$rawContent'));
                 }
                 else {
                     // UIElement has its own render() method (see #799).
-                    domElement = viewNode.render(this._domDocument, this);
+                    domElement = viewElementOrFragment.render(this._domDocument, this);
                 }
                 if (options.bind) {
-                    this.bindElements(domElement, viewNode);
+                    this.bindElements(domElement, viewElementOrFragment);
                 }
                 return domElement;
             }
             else {
                 // Create DOM element.
-                if (this._shouldRenameElement(viewNode.name)) {
-                    _logUnsafeElement(viewNode.name);
-                    domElement = this._createReplacementDomElement(viewNode.name);
+                if (this._shouldRenameElement(viewElementOrFragment.name)) {
+                    _logUnsafeElement(viewElementOrFragment.name);
+                    domElement = this._createReplacementDomElement(viewElementOrFragment.name);
                 }
-                else if (viewNode.hasAttribute('xmlns')) {
-                    domElement = this._domDocument.createElementNS(viewNode.getAttribute('xmlns'), viewNode.name);
+                else if (viewElementOrFragment.hasAttribute('xmlns')) {
+                    domElement = this._domDocument.createElementNS(viewElementOrFragment.getAttribute('xmlns'), viewElementOrFragment.name);
                 }
                 else {
-                    domElement = this._domDocument.createElement(viewNode.name);
+                    domElement = this._domDocument.createElement(viewElementOrFragment.name);
                 }
                 // RawElement take care of their children in RawElement#render() method which can be customized
                 // (see https://github.com/ckeditor/ckeditor5/issues/4469).
-                if (viewNode.is('rawElement')) {
-                    viewNode.render(domElement, this);
+                if (viewElementOrFragment.is('rawElement')) {
+                    viewElementOrFragment.render(domElement, this);
                 }
                 if (options.bind) {
-                    this.bindElements(domElement, viewNode);
+                    this.bindElements(domElement, viewElementOrFragment);
                 }
                 // Copy element's attributes.
-                for (const key of viewNode.getAttributeKeys()) {
-                    this.setDomElementAttribute(domElement, key, viewNode.getAttribute(key), viewNode);
+                for (const key of viewElementOrFragment.getAttributeKeys()) {
+                    this.setDomElementAttribute(domElement, key, viewElementOrFragment.getAttribute(key), viewElementOrFragment);
                 }
             }
             if (options.withChildren !== false) {
-                for (const child of this.viewChildrenToDom(viewNode, options)) {
+                for (const child of this.viewChildrenToDom(viewElementOrFragment, options)) {
                     domElement.appendChild(child);
                 }
             }
@@ -526,6 +538,10 @@ export default class DomConverter {
      * @returns View selection.
      */
     domSelectionToView(domSelection) {
+        // See: https://github.com/ckeditor/ckeditor5/issues/9635.
+        if (isGeckoRestrictedDomSelection(domSelection)) {
+            return new ViewSelection([]);
+        }
         // DOM selection might be placed in fake selection container.
         // If container contains fake selection - return corresponding view selection.
         if (domSelection.rangeCount === 1) {
@@ -921,6 +937,17 @@ export default class DomConverter {
     registerInlineObjectMatcher(pattern) {
         this._inlineObjectElementMatcher.add(pattern);
     }
+    /**
+     * Clear temporary custom properties.
+     *
+     * @internal
+     */
+    _clearTemporaryCustomProperties() {
+        for (const element of this._elementsWithTemporaryCustomProperties) {
+            element._removeCustomProperty('editingPipeline:doNotReuseOnce');
+        }
+        this._elementsWithTemporaryCustomProperties.clear();
+    }
     /**
      * Returns the block {@link module:engine/view/filler filler} node based on the current {@link #blockFillerMode} setting.
      */
@@ -1379,6 +1406,27 @@ function _logUnsafeElement(elementName) {
         logWarning('domconverter-unsafe-style-element-detected');
     }
 }
+/**
+ * In certain cases, Firefox mysteriously assigns so called "restricted objects" to native DOM Range properties.
+ * Any attempt at accessing restricted object's properties causes errors.
+ * See: https://github.com/ckeditor/ckeditor5/issues/9635.
+ */
+function isGeckoRestrictedDomSelection(domSelection) {
+    if (!env.isGecko) {
+        return false;
+    }
+    if (!domSelection.rangeCount) {
+        return false;
+    }
+    const container = domSelection.getRangeAt(0).startContainer;
+    try {
+        Object.prototype.toString.call(container);
+    }
+    catch (error) {
+        return true;
+    }
+    return false;
+}
 /**
  * While rendering the editor content, the {@link module:engine/view/domconverter~DomConverter} detected a `<script>` element that may
  * disrupt the editing experience. To avoid this, the `<script>` element was replaced with `<span data-ck-unsafe-element="script"></span>`.

package/src/view/downcastwriter.d.ts

@@ -877,7 +877,7 @@ export default class DowncastWriter {
      * @internal
      * @param slotFactory The slot factory.
      */
-    _registerSlotFactory(slotFactory: (writer: DowncastWriter, modeOrFilter: string | SlotFilter) => Element): void;
+    _registerSlotFactory(slotFactory: (writer: DowncastWriter, modeOrFilter: 'children' | SlotFilter) => Element): void;
     /**
      * Clears the registered slot factory.
      *

package/src/view/element.d.ts

@@ -383,7 +383,7 @@ export default class Element extends Node {
      * @internal
      * @fires change
      */
-    _removeClass(className: string | Array<string>): void;
+    _removeClass(className: ArrayOrItem<string>): void;
     /**
      * Adds style to the element.
      *

package/src/view/renderer.js

@@ -230,6 +230,7 @@ export default class Renderer extends ObservableMixin() {
         // Otherwise, FF may throw an error (https://github.com/ckeditor/ckeditor5/issues/721).
         this._updateFocus();
         this._updateSelection();
+        this.domConverter._clearTemporaryCustomProperties();
         this.markedTexts.clear();
         this.markedAttributes.clear();
         this.markedChildren.clear();
@@ -258,7 +259,7 @@ export default class Renderer extends ObservableMixin() {
         // This would produce incorrect element mappings.
         //
         // Converting live list to an array to make the list static.
-        const actualDomChildren = Array.from(this.domConverter.mapViewToDom(viewElement).childNodes);
+        const actualDomChildren = Array.from(domElement.childNodes);
         const expectedDomChildren = Array.from(this.domConverter.viewChildrenToDom(viewElement, { withChildren: false }));
         const diff = this._diffNodeLists(actualDomChildren, expectedDomChildren);
         const actions = this._findUpdateActions(diff, actualDomChildren, expectedDomChildren, areSimilarElements);
@@ -272,7 +273,7 @@ export default class Renderer extends ObservableMixin() {
                     // UIElement and RawElement are special cases. Their children are not stored in a view (#799)
                     // so we cannot use them with replacing flow (since they use view children during rendering
                     // which will always result in rendering empty elements).
-                    if (viewChild && !(viewChild.is('uiElement') || viewChild.is('rawElement'))) {
+                    if (viewChild && !viewChild.is('uiElement') && !viewChild.is('rawElement')) {
                         this._updateElementMappings(viewChild, actualDomChildren[deleteIndex]);
                     }
                     remove(expectedDomChildren[insertIndex]);

package/src/view/stylesmap.d.ts

@@ -593,7 +593,7 @@ export declare class StylesProcessor {
     private _mapStyleNames;
 }
 /**
- * A CSS style property descriptor that contains tuplet of two strings:
+ * A CSS style property descriptor that contains tuple of two strings:
  *
  * - first string describes property name
  * - second string describes property value

package/src/view/stylesmap.js

@@ -24,8 +24,7 @@ export default class StylesMap {
      */
     get isEmpty() {
         const entries = Object.entries(this._styles);
-        const from = Array.from(entries);
-        return !from.length;
+        return !entries.length;
     }
     /**
      * Number of styles defined.
@@ -45,7 +44,7 @@ export default class StylesMap {
      */
     setTo(inlineStyle) {
         this.clear();
-        const parsedStyles = Array.from(parseInlineStyles(inlineStyle).entries());
+        const parsedStyles = parseInlineStyles(inlineStyle);
         for (const [key, value] of parsedStyles) {
             this._styleProcessor.toNormalizedForm(key, value, this._styles);
         }
@@ -325,7 +324,7 @@ export default class StylesMap {
         if (!parentObject) {
             return;
         }
-        const isParentEmpty = !Array.from(Object.keys(parentObject)).length;
+        const isParentEmpty = !Object.keys(parentObject).length;
         if (isParentEmpty) {
             this.remove(parentPath);
         }
@@ -484,7 +483,7 @@ export class StylesProcessor {
             ...expandedStyleNames,
             ...Object.keys(styles)
         ]);
-        return Array.from(styleNamesKeysSet.values());
+        return Array.from(styleNamesKeysSet);
     }
     /**
      * Returns related style names.

package/src/view/uielement.js

@@ -84,7 +84,8 @@ export default class UIElement extends Element {
      *
      * @param domConverter Instance of the DomConverter used to optimize the output.
      */
-    render(domDocument, domConverter) {
+    render(domDocument, domConverter // eslint-disable-line @typescript-eslint/no-unused-vars
+    ) {
         // Provide basic, default output.
         return this.toDomElement(domDocument);
     }

package/src/view/view.d.ts

@@ -15,7 +15,7 @@ import Selection, { type PlaceOrOffset, type Selectable, type SelectionOptions }
 import type { default as Observer, ObserverConstructor } from './observer/observer';
 import type { StylesProcessor } from './stylesmap';
 import type Element from './element';
-import type Node from './node';
+import type { default as Node } from './node';
 import type Item from './item';
 import KeyObserver from './observer/keyobserver';
 import FakeSelectionObserver from './observer/fakeselectionobserver';