@ckeditor/ckeditor5-engine 45.2.1-alpha.9 → 46.0.0-alpha.0

package/src/model/writer.js

@@ -5,23 +5,23 @@
 /**
  * @module engine/model/writer
  */
-import AttributeOperation from './operation/attributeoperation.js';
-import DetachOperation from './operation/detachoperation.js';
-import InsertOperation from './operation/insertoperation.js';
-import MarkerOperation from './operation/markeroperation.js';
-import MergeOperation from './operation/mergeoperation.js';
-import MoveOperation from './operation/moveoperation.js';
-import RenameOperation from './operation/renameoperation.js';
-import RootAttributeOperation from './operation/rootattributeoperation.js';
-import RootOperation from './operation/rootoperation.js';
-import SplitOperation from './operation/splitoperation.js';
-import DocumentFragment from './documentfragment.js';
-import DocumentSelection from './documentselection.js';
-import Element from './element.js';
-import Position from './position.js';
-import Range from './range.js';
-import RootElement from './rootelement.js';
-import Text from './text.js';
+import { AttributeOperation } from './operation/attributeoperation.js';
+import { DetachOperation } from './operation/detachoperation.js';
+import { InsertOperation } from './operation/insertoperation.js';
+import { MarkerOperation } from './operation/markeroperation.js';
+import { MergeOperation } from './operation/mergeoperation.js';
+import { MoveOperation } from './operation/moveoperation.js';
+import { RenameOperation } from './operation/renameoperation.js';
+import { RootAttributeOperation } from './operation/rootattributeoperation.js';
+import { RootOperation } from './operation/rootoperation.js';
+import { SplitOperation } from './operation/splitoperation.js';
+import { ModelDocumentFragment } from './documentfragment.js';
+import { ModelDocumentSelection } from './documentselection.js';
+import { ModelElement } from './element.js';
+import { ModelPosition } from './position.js';
+import { ModelRange } from './range.js';
+import { ModelRootElement } from './rootelement.js';
+import { ModelText } from './text.js';
 import { CKEditorError, logWarning, toMap } from '@ckeditor/ckeditor5-utils';
 /**
  * The model can only be modified by using the writer. It should be used whenever you want to create a node, modify
@@ -39,14 +39,14 @@ import { CKEditorError, logWarning, toMap } from '@ckeditor/ckeditor5-utils';
  * Note that the writer should never be stored and used outside of the `change()` and
  * `enqueueChange()` blocks.
  *
- * Note that writer's methods do not check the {@link module:engine/model/schema~Schema}. It is possible
+ * Note that writer's methods do not check the {@link module:engine/model/schema~ModelSchema}. It is possible
  * to create incorrect model structures by using the writer. Read more about in
  * {@glink framework/deep-dive/schema#who-checks-the-schema "Who checks the schema?"}.
  *
  * @see module:engine/model/model~Model#change
  * @see module:engine/model/model~Model#enqueueChange
  */
-export default class Writer {
+export class ModelWriter {
     /**
      * Instance of the model on which this writer operates.
      */
@@ -68,7 +68,7 @@ export default class Writer {
         this.batch = batch;
     }
     /**
-     * Creates a new {@link module:engine/model/text~Text text node}.
+     * Creates a new {@link module:engine/model/text~ModelText text node}.
      *
      * ```ts
      * writer.createText( 'foo' );
@@ -77,13 +77,13 @@ export default class Writer {
      *
      * @param data Text data.
      * @param attributes Text attributes.
-     * @returns {module:engine/model/text~Text} Created text node.
+     * @returns {module:engine/model/text~ModelText} Created text node.
      */
     createText(data, attributes) {
-        return new Text(data, attributes);
+        return new ModelText(data, attributes);
     }
     /**
-     * Creates a new {@link module:engine/model/element~Element element}.
+     * Creates a new {@link module:engine/model/element~ModelElement element}.
      *
      * ```ts
      * writer.createElement( 'paragraph' );
@@ -95,15 +95,15 @@ export default class Writer {
      * @returns Created element.
      */
     createElement(name, attributes) {
-        return new Element(name, attributes);
+        return new ModelElement(name, attributes);
     }
     /**
-     * Creates a new {@link module:engine/model/documentfragment~DocumentFragment document fragment}.
+     * Creates a new {@link module:engine/model/documentfragment~ModelDocumentFragment document fragment}.
      *
      * @returns Created document fragment.
      */
     createDocumentFragment() {
-        return new DocumentFragment();
+        return new ModelDocumentFragment();
     }
     /**
      * Creates a copy of the element and returns it. Created element has the same name and attributes as the original element.
@@ -152,27 +152,27 @@ export default class Writer {
      * Note that you cannot re-insert a node from a document to a different document or a document fragment. In this case,
      * `model-writer-insert-forbidden-move` is thrown.
      *
-     * If you want to move {@link module:engine/model/range~Range range} instead of an
-     * {@link module:engine/model/item~Item item} use {@link module:engine/model/writer~Writer#move `Writer#move()`}.
+     * If you want to move {@link module:engine/model/range~ModelRange range} instead of an
+     * {@link module:engine/model/item~ModelItem item} use {@link module:engine/model/writer~ModelWriter#move `Writer#move()`}.
      *
      * **Note:** For a paste-like content insertion mechanism see
      * {@link module:engine/model/model~Model#insertContent `model.insertContent()`}.
      *
      * @param item Item or document fragment to insert.
-     * @param offset Offset or one of the flags. Used only when second parameter is a {@link module:engine/model/item~Item model item}.
+     * @param offset Offset or one of the flags. Used only when second parameter is a {@link module:engine/model/item~ModelItem model item}.
      */
     insert(item, itemOrPosition, offset = 0) {
         this._assertWriterUsedCorrectly();
-        if (item instanceof Text && item.data == '') {
+        if (item instanceof ModelText && item.data == '') {
             return;
         }
-        const position = Position._createAt(itemOrPosition, offset);
+        const position = ModelPosition._createAt(itemOrPosition, offset);
         // If item has a parent already.
         if (item.parent) {
             // We need to check if item is going to be inserted within the same document.
             if (isSameTree(item.root, position.root)) {
                 // If it's we just need to move it.
-                this.move(Range._createOn(item), position);
+                this.move(ModelRange._createOn(item), position);
                 return;
             }
             // If it isn't the same root.
@@ -194,21 +194,21 @@ export default class Writer {
             }
         }
         const version = position.root.document ? position.root.document.version : null;
-        const children = item instanceof DocumentFragment ?
+        const children = item instanceof ModelDocumentFragment ?
             item._removeChildren(0, item.childCount) :
             item;
         const insert = new InsertOperation(position, children, version);
-        if (item instanceof Text) {
+        if (item instanceof ModelText) {
             insert.shouldReceiveAttributes = true;
         }
         this.batch.addOperation(insert);
         this.model.applyOperation(insert);
-        // When element is a DocumentFragment we need to move its markers to Document#markers.
-        if (item instanceof DocumentFragment) {
+        // When element is a ModelDocumentFragment we need to move its markers to Document#markers.
+        if (item instanceof ModelDocumentFragment) {
             for (const [markerName, markerRange] of item.markers) {
-                // We need to migrate marker range from DocumentFragment to Document.
-                const rangeRootPosition = Position._createAt(markerRange.root, 0);
-                const range = new Range(markerRange.start._getCombined(rangeRootPosition, position), markerRange.end._getCombined(rangeRootPosition, position));
+                // We need to migrate marker range from ModelDocumentFragment to Document.
+                const rangeRootPosition = ModelPosition._createAt(markerRange.root, 0);
+                const range = new ModelRange(markerRange.start._getCombined(rangeRootPosition, position), markerRange.end._getCombined(rangeRootPosition, position));
                 const options = { range, usingOperation: true, affectsData: true };
                 if (this.model.markers.has(markerName)) {
                     this.updateMarker(markerName, options);
@@ -223,7 +223,7 @@ export default class Writer {
     itemOrPosition, // Too complicated when not using `any`.
     offset // Too complicated when not using `any`.
     ) {
-        if (attributes instanceof DocumentFragment || attributes instanceof Element || attributes instanceof Position) {
+        if (attributes instanceof ModelDocumentFragment || attributes instanceof ModelElement || attributes instanceof ModelPosition) {
             this.insert(this.createText(text), attributes, itemOrPosition);
         }
         else {
@@ -234,7 +234,7 @@ export default class Writer {
     itemOrPositionOrOffset, // Too complicated when not using `any`.
     offset // Too complicated when not using `any`.
     ) {
-        if (attributes instanceof DocumentFragment || attributes instanceof Element || attributes instanceof Position) {
+        if (attributes instanceof ModelDocumentFragment || attributes instanceof ModelElement || attributes instanceof ModelPosition) {
             this.insert(this.createElement(name), attributes, itemOrPositionOrOffset);
         }
         else {
@@ -251,8 +251,8 @@ export default class Writer {
      *
      * Note that if the item already has parent it will be removed from the previous parent.
      *
-     * If you want to move {@link module:engine/model/range~Range range} instead of an
-     * {@link module:engine/model/item~Item item} use {@link module:engine/model/writer~Writer#move `Writer#move()`}.
+     * If you want to move {@link module:engine/model/range~ModelRange range} instead of an
+     * {@link module:engine/model/item~ModelItem item} use {@link module:engine/model/writer~ModelWriter#move `Writer#move()`}.
      *
      * @param item Item or document fragment to insert.
      */
@@ -260,7 +260,7 @@ export default class Writer {
         this.insert(item, parent, 'end');
     }
     appendText(text, attributes, parent) {
-        if (attributes instanceof DocumentFragment || attributes instanceof Element) {
+        if (attributes instanceof ModelDocumentFragment || attributes instanceof ModelElement) {
             this.insert(this.createText(text), attributes, 'end');
         }
         else {
@@ -268,7 +268,7 @@ export default class Writer {
         }
     }
     appendElement(name, attributes, parent) {
-        if (attributes instanceof DocumentFragment || attributes instanceof Element) {
+        if (attributes instanceof ModelDocumentFragment || attributes instanceof ModelElement) {
             this.insert(this.createElement(name), attributes, 'end');
         }
         else {
@@ -276,8 +276,8 @@ export default class Writer {
         }
     }
     /**
-     * Sets value of the attribute with given key on a {@link module:engine/model/item~Item model item}
-     * or on a {@link module:engine/model/range~Range range}.
+     * Sets value of the attribute with given key on a {@link module:engine/model/item~ModelItem model item}
+     * or on a {@link module:engine/model/range~ModelRange range}.
      *
      * @param key Attribute key.
      * @param value Attribute new value.
@@ -285,7 +285,7 @@ export default class Writer {
      */
     setAttribute(key, value, itemOrRange) {
         this._assertWriterUsedCorrectly();
-        if (itemOrRange instanceof Range) {
+        if (itemOrRange instanceof ModelRange) {
             const ranges = itemOrRange.getMinimalFlatRanges();
             for (const range of ranges) {
                 setAttributeOnRange(this, key, value, range);
@@ -296,8 +296,8 @@ export default class Writer {
         }
     }
     /**
-     * Sets values of attributes on a {@link module:engine/model/item~Item model item}
-     * or on a {@link module:engine/model/range~Range range}.
+     * Sets values of attributes on a {@link module:engine/model/item~ModelItem model item}
+     * or on a {@link module:engine/model/range~ModelRange range}.
      *
      * ```ts
      * writer.setAttributes( {
@@ -315,15 +315,15 @@ export default class Writer {
         }
     }
     /**
-     * Removes an attribute with given key from a {@link module:engine/model/item~Item model item}
-     * or from a {@link module:engine/model/range~Range range}.
+     * Removes an attribute with given key from a {@link module:engine/model/item~ModelItem model item}
+     * or from a {@link module:engine/model/range~ModelRange range}.
      *
      * @param key Attribute key.
      * @param itemOrRange Model item or range from which the attribute will be removed.
      */
     removeAttribute(key, itemOrRange) {
         this._assertWriterUsedCorrectly();
-        if (itemOrRange instanceof Range) {
+        if (itemOrRange instanceof ModelRange) {
             const ranges = itemOrRange.getMinimalFlatRanges();
             for (const range of ranges) {
                 setAttributeOnRange(this, key, null, range);
@@ -345,7 +345,7 @@ export default class Writer {
                 this.removeAttribute(attribute, item);
             }
         };
-        if (!(itemOrRange instanceof Range)) {
+        if (!(itemOrRange instanceof ModelRange)) {
             removeAttributesFromItem(itemOrRange);
         }
         else {
@@ -376,16 +376,16 @@ export default class Writer {
      * These parameters work the same way as {@link #createPositionAt `writer.createPositionAt()`}.
      *
      * Note that items can be moved only within the same tree. It means that you can move items within the same root
-     * (element or document fragment) or between {@link module:engine/model/document~Document#roots documents roots},
+     * (element or document fragment) or between {@link module:engine/model/document~ModelDocument#roots documents roots},
      * but you cannot move items from document fragment to the document or from one detached element to another. Use
-     * {@link module:engine/model/writer~Writer#insert} in such cases.
+     * {@link module:engine/model/writer~ModelWriter#insert} in such cases.
      *
      * @param range Source range.
-     * @param offset Offset or one of the flags. Used only when second parameter is a {@link module:engine/model/item~Item model item}.
+     * @param offset Offset or one of the flags. Used only when second parameter is a {@link module:engine/model/item~ModelItem model item}.
      */
     move(range, itemOrPosition, offset) {
         this._assertWriterUsedCorrectly();
-        if (!(range instanceof Range)) {
+        if (!(range instanceof ModelRange)) {
             /**
              * Invalid range to move.
              *
@@ -401,7 +401,7 @@ export default class Writer {
              */
             throw new CKEditorError('writer-move-range-not-flat', this);
         }
-        const position = Position._createAt(itemOrPosition, offset);
+        const position = ModelPosition._createAt(itemOrPosition, offset);
         // Do not move anything if the move target is same as moved range start.
         if (position.isEqual(range.start)) {
             return;
@@ -411,7 +411,7 @@ export default class Writer {
         if (!isSameTree(range.root, position.root)) {
             /**
              * Range is going to be moved within not the same document. Please use
-             * {@link module:engine/model/writer~Writer#insert insert} instead.
+             * {@link module:engine/model/writer~ModelWriter#insert insert} instead.
              *
              * @error writer-move-different-document
              */
@@ -423,13 +423,13 @@ export default class Writer {
         this.model.applyOperation(operation);
     }
     /**
-     * Removes given model {@link module:engine/model/item~Item item} or {@link module:engine/model/range~Range range}.
+     * Removes given model {@link module:engine/model/item~ModelItem item} or {@link module:engine/model/range~ModelRange range}.
      *
      * @param itemOrRange Model item or range to remove.
      */
     remove(itemOrRange) {
         this._assertWriterUsedCorrectly();
-        const rangeToRemove = itemOrRange instanceof Range ? itemOrRange : Range._createOn(itemOrRange);
+        const rangeToRemove = itemOrRange instanceof ModelRange ? itemOrRange : ModelRange._createOn(itemOrRange);
         const ranges = rangeToRemove.getMinimalFlatRanges().reverse();
         for (const flat of ranges) {
             // If part of the marker is removed, create additional marker operation for undo purposes.
@@ -451,7 +451,7 @@ export default class Writer {
         const nodeAfter = position.nodeAfter;
         // If part of the marker is removed, create additional marker operation for undo purposes.
         this._addOperationForAffectedMarkers('merge', position);
-        if (!(nodeBefore instanceof Element)) {
+        if (!(nodeBefore instanceof ModelElement)) {
             /**
              * Node before merge position must be an element.
              *
@@ -459,7 +459,7 @@ export default class Writer {
              */
             throw new CKEditorError('writer-merge-no-element-before', this);
         }
-        if (!(nodeAfter instanceof Element)) {
+        if (!(nodeAfter instanceof ModelElement)) {
             /**
              * Node after merge position must be an element.
              *
@@ -478,8 +478,8 @@ export default class Writer {
      * Shortcut for {@link module:engine/model/model~Model#createPositionFromPath `Model#createPositionFromPath()`}.
      *
      * @param root Root of the position.
-     * @param path Position path. See {@link module:engine/model/position~Position#path}.
-     * @param stickiness Position stickiness. See {@link module:engine/model/position~PositionStickiness}.
+     * @param path Position path. See {@link module:engine/model/position~ModelPosition#path}.
+     * @param stickiness Position stickiness. See {@link module:engine/model/position~ModelPositionStickiness}.
      */
     createPositionFromPath(root, path, stickiness) {
         return this.model.createPositionFromPath(root, path, stickiness);
@@ -487,7 +487,7 @@ export default class Writer {
     /**
      * Shortcut for {@link module:engine/model/model~Model#createPositionAt `Model#createPositionAt()`}.
      *
-     * @param offset Offset or one of the flags. Used only when first parameter is a {@link module:engine/model/item~Item model item}.
+     * @param offset Offset or one of the flags. Used only when first parameter is a {@link module:engine/model/item~ModelItem model item}.
      */
     createPositionAt(itemOrPosition, offset) {
         return this.model.createPositionAt(itemOrPosition, offset);
@@ -544,7 +544,7 @@ export default class Writer {
     _mergeDetached(position) {
         const nodeBefore = position.nodeBefore;
         const nodeAfter = position.nodeAfter;
-        this.move(Range._createIn(nodeAfter), Position._createAt(nodeBefore, 'end'));
+        this.move(ModelRange._createIn(nodeAfter), ModelPosition._createAt(nodeBefore, 'end'));
         this.remove(nodeAfter);
     }
     /**
@@ -553,10 +553,10 @@ export default class Writer {
      * @param position Position between merged elements.
      */
     _merge(position) {
-        const targetPosition = Position._createAt(position.nodeBefore, 'end');
-        const sourcePosition = Position._createAt(position.nodeAfter, 0);
+        const targetPosition = ModelPosition._createAt(position.nodeBefore, 'end');
+        const sourcePosition = ModelPosition._createAt(position.nodeAfter, 0);
         const graveyard = position.root.document.graveyard;
-        const graveyardPosition = new Position(graveyard, [0]);
+        const graveyardPosition = new ModelPosition(graveyard, [0]);
         const version = position.root.document.version;
         const merge = new MergeOperation(sourcePosition, position.nodeAfter.maxOffset, targetPosition, graveyardPosition, version);
         this.batch.addOperation(merge);
@@ -570,7 +570,7 @@ export default class Writer {
      */
     rename(element, newName) {
         this._assertWriterUsedCorrectly();
-        if (!(element instanceof Element)) {
+        if (!(element instanceof ModelElement)) {
             /**
              * Trying to rename an object which is not an instance of Element.
              *
@@ -579,7 +579,7 @@ export default class Writer {
             throw new CKEditorError('writer-rename-not-element-instance', this);
         }
         const version = element.root.document ? element.root.document.version : null;
-        const renameOperation = new RenameOperation(Position._createBefore(element), element.name, newName, version);
+        const renameOperation = new RenameOperation(ModelPosition._createBefore(element), element.name, newName, version);
         this.batch.addOperation(renameOperation);
         this.model.applyOperation(renameOperation);
     }
@@ -621,7 +621,7 @@ export default class Writer {
         }
         // We need to cache elements that will be created as a result of the first split because
         // we need to create a range from the end of the first split element to the beginning of the
-        // first copy element. This should be handled by LiveRange but it doesn't work on detached nodes.
+        // first copy element. This should be handled by ModelLiveRange but it doesn't work on detached nodes.
         let firstSplitElement;
         let firstCopyElement;
         do {
@@ -641,13 +641,13 @@ export default class Writer {
         } while (splitElement !== limitElement);
         return {
             position,
-            range: new Range(Position._createAt(firstSplitElement, 'end'), Position._createAt(firstCopyElement, 0))
+            range: new ModelRange(ModelPosition._createAt(firstSplitElement, 'end'), ModelPosition._createAt(firstCopyElement, 0))
         };
     }
     /**
      * Wraps the given range with the given element or with a new element (if a string was passed).
      *
-     * **Note:** range to wrap should be a "flat range" (see {@link module:engine/model/range~Range#isFlat `Range#isFlat`}).
+     * **Note:** range to wrap should be a "flat range" (see {@link module:engine/model/range~ModelRange#isFlat `Range#isFlat`}).
      * If not, an error will be thrown.
      *
      * @param range Range to wrap.
@@ -663,7 +663,7 @@ export default class Writer {
              */
             throw new CKEditorError('writer-wrap-range-not-flat', this);
         }
-        const element = elementOrString instanceof Element ? elementOrString : new Element(elementOrString);
+        const element = elementOrString instanceof ModelElement ? elementOrString : new ModelElement(elementOrString);
         if (element.childCount > 0) {
             /**
              * Element to wrap with is not empty.
@@ -682,8 +682,8 @@ export default class Writer {
         }
         this.insert(element, range.start);
         // Shift the range-to-wrap because we just inserted an element before that range.
-        const shiftedRange = new Range(range.start.getShiftedBy(1), range.end.getShiftedBy(1));
-        this.move(shiftedRange, Position._createAt(element, 0));
+        const shiftedRange = new ModelRange(range.start.getShiftedBy(1), range.end.getShiftedBy(1));
+        this.move(shiftedRange, ModelPosition._createAt(element, 0));
     }
     /**
      * Unwraps children of the given element – all its children are moved before it and then the element is removed.
@@ -701,7 +701,7 @@ export default class Writer {
              */
             throw new CKEditorError('writer-unwrap-element-no-parent', this);
         }
-        this.move(Range._createIn(element), this.createPositionAfter(element));
+        this.move(ModelRange._createIn(element), this.createPositionAfter(element));
         this.remove(element);
     }
     /**
@@ -717,8 +717,8 @@ export default class Writer {
      * The `options.affectsData` parameter, which defaults to `false`, allows you to define if a marker affects the data. It should be
      * `true` when the marker change changes the data returned by the
      * {@link module:core/editor/editor~Editor#getData `editor.getData()`} method.
-     * When set to `true` it fires the {@link module:engine/model/document~Document#event:change:data `change:data`} event.
-     * When set to `false` it fires the {@link module:engine/model/document~Document#event:change `change`} event.
+     * When set to `true` it fires the {@link module:engine/model/document~ModelDocument#event:change:data `change:data`} event.
+     * When set to `false` it fires the {@link module:engine/model/document~ModelDocument#event:change `change`} event.
      *
      * Create marker directly base on marker's name:
      *
@@ -791,7 +791,8 @@ export default class Writer {
      * As the first parameter you can set marker name or instance. If none of them is provided, new marker, with a unique
      * name is created and returned.
      *
-     * **Note**: If you want to change the {@link module:engine/view/element~Element view element} of the marker while its data in the model
+     * **Note**: If you want to change the {@link module:engine/view/element~ViewElement view element}
+     * of the marker while its data in the model
      * remains the same, use the dedicated {@link module:engine/controller/editingcontroller~EditingController#reconvertMarker} method.
      *
      * The `options.usingOperation` parameter lets you change if the marker should be managed by operations or not. See
@@ -801,8 +802,8 @@ export default class Writer {
      * The `options.affectsData` parameter, which defaults to `false`, allows you to define if a marker affects the data. It should be
      * `true` when the marker change changes the data returned by
      * the {@link module:core/editor/editor~Editor#getData `editor.getData()`} method.
-     * When set to `true` it fires the {@link module:engine/model/document~Document#event:change:data `change:data`} event.
-     * When set to `false` it fires the {@link module:engine/model/document~Document#event:change `change`} event.
+     * When set to `true` it fires the {@link module:engine/model/document~ModelDocument#event:change:data `change:data`} event.
+     * When set to `false` it fires the {@link module:engine/model/document~ModelDocument#event:change `change`} event.
      *
      * Update marker directly base on marker's name:
      *
@@ -1006,13 +1007,13 @@ export default class Writer {
         this.model.document.selection._setTo(...args);
     }
     /**
-     * Moves {@link module:engine/model/documentselection~DocumentSelection#focus} to the specified location.
+     * Moves {@link module:engine/model/documentselection~ModelDocumentSelection#focus} to the specified location.
      *
      * The location can be specified in the same form as
      * {@link #createPositionAt `writer.createPositionAt()`} parameters.
      *
      * @param itemOrPosition
-     * @param offset Offset or one of the flags. Used only when first parameter is a {@link module:engine/model/item~Item model item}.
+     * @param offset Offset or one of the flags. Used only when first parameter is a {@link module:engine/model/item~ModelItem model item}.
      */
     setSelectionFocus(itemOrPosition, offset) {
         this._assertWriterUsedCorrectly();
@@ -1058,7 +1059,7 @@ export default class Writer {
         }
     }
     /**
-     * Temporarily changes the {@link module:engine/model/documentselection~DocumentSelection#isGravityOverridden gravity}
+     * Temporarily changes the {@link module:engine/model/documentselection~ModelDocumentSelection#isGravityOverridden gravity}
      * of the selection from left to right.
      *
      * The gravity defines from which direction the selection inherits its attributes. If it's the default left gravity,
@@ -1083,13 +1084,13 @@ export default class Writer {
         return this.model.document.selection._overrideGravity();
     }
     /**
-     * Restores {@link ~Writer#overrideSelectionGravity} gravity to default.
+     * Restores {@link ~ModelWriter#overrideSelectionGravity} gravity to default.
      *
      * Restoring the gravity is only possible using the unique identifier returned by
-     * {@link ~Writer#overrideSelectionGravity}. Note that the gravity remains overridden as long as won't be restored
+     * {@link ~ModelWriter#overrideSelectionGravity}. Note that the gravity remains overridden as long as won't be restored
      * the same number of times it was overridden.
      *
-     * @param uid The unique id returned by {@link ~Writer#overrideSelectionGravity}.
+     * @param uid The unique id returned by {@link ~ModelWriter#overrideSelectionGravity}.
      */
     restoreSelectionGravity(uid) {
         this.model.document.selection._restoreGravity(uid);
@@ -1102,7 +1103,7 @@ export default class Writer {
         const selection = this.model.document.selection;
         // Store attribute in parent element if the selection is collapsed in an empty node.
         if (selection.isCollapsed && selection.anchor.parent.isEmpty) {
-            const storeKey = DocumentSelection._getStoreAttributeKey(key);
+            const storeKey = ModelDocumentSelection._getStoreAttributeKey(key);
             this.setAttribute(storeKey, value, selection.anchor.parent);
         }
         selection._setAttribute(key, value);
@@ -1114,7 +1115,7 @@ export default class Writer {
         const selection = this.model.document.selection;
         // Remove stored attribute from parent element if the selection is collapsed in an empty node.
         if (selection.isCollapsed && selection.anchor.parent.isEmpty) {
-            const storeKey = DocumentSelection._getStoreAttributeKey(key);
+            const storeKey = ModelDocumentSelection._getStoreAttributeKey(key);
             this.removeAttribute(storeKey, selection.anchor.parent);
         }
         selection._removeAttribute(key);
@@ -1228,11 +1229,11 @@ function setAttributeOnRange(writer, key, value, range) {
     }
     // Because position in the loop is not the iterator position (see let position comment), the last position in
     // the while loop will be last but one position in the range. We need to check the last position manually.
-    if (position instanceof Position && position != lastSplitPosition && valueBefore != value) {
+    if (position instanceof ModelPosition && position != lastSplitPosition && valueBefore != value) {
         addOperation();
     }
     function addOperation() {
-        const range = new Range(lastSplitPosition, position);
+        const range = new ModelRange(lastSplitPosition, position);
         const version = range.root.document ? doc.version : null;
         const operation = new AttributeOperation(range, key, valueBefore, value, version);
         writer.batch.addOperation(operation);
@@ -1255,7 +1256,7 @@ function setAttributeOnItem(writer, key, value, item) {
             operation = new RootAttributeOperation(item, key, previousValue, value, version);
         }
         else {
-            range = new Range(Position._createBefore(item), writer.createPositionAfter(item));
+            range = new ModelRange(ModelPosition._createBefore(item), writer.createPositionAfter(item));
             const version = range.root.document ? doc.version : null;
             operation = new AttributeOperation(range, key, previousValue, value, version);
         }
@@ -1286,7 +1287,7 @@ function applyRemoveOperation(position, howMany, batch, model) {
     let operation;
     if (position.root.document) {
         const doc = model.document;
-        const graveyardPosition = new Position(doc.graveyard, [0]);
+        const graveyardPosition = new ModelPosition(doc.graveyard, [0]);
         operation = new MoveOperation(position, howMany, graveyardPosition, doc.version);
     }
     else {
@@ -1310,7 +1311,7 @@ function isSameTree(rootA, rootB) {
         return true;
     }
     // If both roots are documents root it is operation within the document what we still treat as the same tree.
-    if (rootA instanceof RootElement && rootB instanceof RootElement) {
+    if (rootA instanceof ModelRootElement && rootB instanceof ModelRootElement) {
         return true;
     }
     return false;