Browse Source

Merge branch 'docs-fixes'

Maciek 9 years ago
parent
commit
f0eb0c29f4
29 changed files with 54 additions and 99 deletions
  1. 2 2
      packages/ckeditor5-engine/src/conversion/buildmodelconverter.js
  2. 1 1
      packages/ckeditor5-engine/src/conversion/buildviewconverter.js
  3. 5 17
      packages/ckeditor5-engine/src/conversion/model-selection-to-view-converters.js
  4. 6 28
      packages/ckeditor5-engine/src/conversion/model-to-view-converters.js
  5. 1 1
      packages/ckeditor5-engine/src/conversion/modelconsumable.js
  6. 1 6
      packages/ckeditor5-engine/src/conversion/view-selection-to-model-converters.js
  7. 1 9
      packages/ckeditor5-engine/src/conversion/view-to-model-converters.js
  8. 2 2
      packages/ckeditor5-engine/src/conversion/viewconversiondispatcher.js
  9. 3 3
      packages/ckeditor5-engine/src/model/delta/splitdelta.js
  10. 3 3
      packages/ckeditor5-engine/src/model/delta/transform.js
  11. 1 1
      packages/ckeditor5-engine/src/model/delta/wrapdelta.js
  12. 2 1
      packages/ckeditor5-engine/src/model/document.js
  13. 1 1
      packages/ckeditor5-engine/src/model/documentfragment.js
  14. 1 1
      packages/ckeditor5-engine/src/model/node.js
  15. 1 1
      packages/ckeditor5-engine/src/model/nodelist.js
  16. 1 1
      packages/ckeditor5-engine/src/model/operation/operation.js
  17. 1 1
      packages/ckeditor5-engine/src/model/operation/removeoperation.js
  18. 2 2
      packages/ckeditor5-engine/src/model/operation/transform.js
  19. 1 1
      packages/ckeditor5-engine/src/model/position.js
  20. 2 2
      packages/ckeditor5-engine/src/view/attributeelement.js
  21. 2 2
      packages/ckeditor5-engine/src/view/containerelement.js
  22. 3 2
      packages/ckeditor5-engine/src/view/document.js
  23. 1 1
      packages/ckeditor5-engine/src/view/editableelement.js
  24. 2 2
      packages/ckeditor5-engine/src/view/element.js
  25. 2 3
      packages/ckeditor5-engine/src/view/observer/domeventobserver.js
  26. 1 1
      packages/ckeditor5-engine/src/view/observer/fakeselectionobserver.js
  27. 1 1
      packages/ckeditor5-engine/src/view/observer/keyobserver.js
  28. 3 2
      packages/ckeditor5-engine/src/view/observer/mutationobserver.js
  29. 1 1
      packages/ckeditor5-engine/src/view/rooteditableelement.js

+ 2 - 2
packages/ckeditor5-engine/src/conversion/buildmodelconverter.js

@@ -27,7 +27,7 @@ import ViewContainerElement from '../view/containerelement.js';
  * {@link module:engine/conversion/buildmodelconverter~buildModelConverter}.
  *
  * If you need more complex converters, see {@link module:engine/conversion/modelconversiondispatcher~ModelConversionDispatcher},
- * {@link module:engine/conversion/modeltoview~modelToView}, {@link module:engine/conversion/modelconsumable~ModelConsumable},
+ * {@link module:engine/conversion/model-to-view-converters}, {@link module:engine/conversion/modelconsumable~ModelConsumable},
  * {@link module:engine/conversion/mapper~Mapper}.
  *
  * Using this API it is possible to create three kinds of converters:
@@ -166,7 +166,7 @@ class ModelConverterBuilder {
 	 * `string`, view element instance which will be cloned and used, or creator function which returns view element that
 	 * will be used. Keep in mind that when you view element instance or creator function, it has to be/return a
 	 * proper type of view element: {@link module:engine/view/containerelement~ContainerElement ViewContainerElement} if you convert
-	 * from element or {@link module:engine/view/attributeelement~ViewAttributeElement ViewAttributeElement} if you convert from attribute.
+	 * from element or {@link module:engine/view/attributeelement~AttributeElement ViewAttributeElement} if you convert from attribute.
 	 *
 	 *		buildModelConverter().for( dispatcher ).fromElement( 'paragraph' ).toElement( 'p' );
 	 *

+ 1 - 1
packages/ckeditor5-engine/src/conversion/buildviewconverter.js

@@ -23,7 +23,7 @@ import isIterable from '../../utils/isiterable.js';
  * {@link module:engine/conversion/buildviewconverter~buildViewConverter}.
  *
  * If you need more complex converters, see {@link module:engine/conversion/viewconversiondispatcher~ViewConversionDispatcher},
- * {@link module:engine/conversion/view-to-model-converters~viewToModel}, {@link module:engine/conversion/viewconsumable~ViewConsumable}.
+ * {@link module:engine/conversion/view-to-model-converters}, {@link module:engine/conversion/viewconsumable~ViewConsumable}.
  *
  * Using this API it is possible to create various kind of converters:
  *

+ 5 - 17
packages/ckeditor5-engine/src/conversion/model-selection-to-view-converters.js

@@ -3,10 +3,6 @@
  * For licensing, see LICENSE.md.
  */
 
-/**
- * @module engine/conversion/model-selection-to-view-converters
- */
-
 import ViewElement from '../view/element.js';
 import ViewRange from '../view/range.js';
 import viewWriter from '../view/writer.js';
@@ -16,7 +12,7 @@ import viewWriter from '../view/writer.js';
  * {@link module:engine/view/selection~Selection view selection} converters for
  * {@link module:engine/conversion/modelconversiondispatcher~ModelConversionDispatcher}.
  *
- * @namespace module:engine/conversion/model-selection-to-view-converters~modelSelectionToView
+ * @module engine/conversion/model-selection-to-view-converters
  */
 
 /**
@@ -26,8 +22,6 @@ import viewWriter from '../view/writer.js';
  *
  *		modelDispatcher.on( 'selection', convertRangeSelection() );
  *
- * @external module:engine/conversion/model-selection-to-view-converters~modelSelectionToView
- * @function module:engine/conversion/model-selection-to-view-converters~modelSelectionToView.convertRangeSelection
  * @returns {Function} Selection converter.
  */
 export function convertRangeSelection() {
@@ -65,16 +59,14 @@ export function convertRangeSelection() {
  *		-> <p><strong>f</strong>^<strong>oo</strong>bar</p>
  *
  * By breaking attribute elements like `<strong>`, selection is in correct element. See also complementary
- * {@link module:engine/conversion/model-selection-to-view-converters~modelSelectionToView.convertSelectionAttribute attribute converter}
+ * {@link module:engine/conversion/model-selection-to-view-converters~convertSelectionAttribute attribute converter}
  * for selection attributes,
  * which wraps collapsed selection into view elements. Those converters together ensure, that selection ends up in
  * appropriate attribute elements.
  *
- * See also {@link module:engine/conversion/model-selection-to-view-converters~modelSelectionToView.clearAttributes} which does a clean-up
+ * See also {@link module:engine/conversion/model-selection-to-view-converters~clearAttributes} which does a clean-up
  * by merging attributes.
  *
- * @external module:engine/conversion/model-selection-to-view-converters~modelSelectionToView
- * @function module:engine/conversion/model-selection-to-view-converters~modelSelectionToView.convertCollapsedSelection
  * @returns {Function} Selection converter.
  */
 export function convertCollapsedSelection() {
@@ -122,7 +114,7 @@ export function convertCollapsedSelection() {
  *		modelDispatcher.on( 'selectionAttribute:style', convertSelectionAttribute( styleCreator ) );
  *
  * **Note:** You can use the same `elementCreator` function for this converter factory
- * and {@link module:engine/conversion/modeltoview~modelToView.wrap}
+ * and {@link module:engine/conversion/model-to-view-converters~wrap}
  * model to view converter, as long as the `elementCreator` function uses only the first parameter (attribute value).
  *
  *		modelDispatcher.on( 'selection', convertCollapsedSelection() );
@@ -147,8 +139,6 @@ export function convertCollapsedSelection() {
  * but then it got wrapped-back by `convertSelectionAttribute()` converter. In second example, notice how `<strong>` element
  * is broken to prevent putting selection in it, since selection has no `bold` attribute.
  *
- * @external module:engine/conversion/model-selection-to-view-converters~modelSelectionToView
- * @function module:engine/conversion/model-selection-to-view-converters~modelSelectionToView.convertCollapsedSelection
  * @param {module:engine/view/attributeelement~AttributeElement|Function} elementCreator View element,
  * or function returning a view element, which will be used for wrapping.
  * @returns {Function} Selection converter.
@@ -197,11 +187,9 @@ export function convertSelectionAttribute( elementCreator ) {
  *
  *		modelDispatcher.on( 'selection', clearAttributes() );
  *
- * See {@link module:engine/conversion/model-selection-to-view-converters~modelSelectionToView.convertCollapsedSelection}
+ * See {@link module:engine/conversion/model-selection-to-view-converters~convertCollapsedSelection}
  * which do the opposite by breaking attributes in the selection position.
  *
- * @external module:engine/conversion/model-selection-to-view-converters~modelSelectionToView
- * @function module:engine/conversion/model-selection-to-view-converters~modelSelectionToView.clearAttributes
  * @returns {Function} Selection converter.
  */
 export function clearAttributes() {

+ 6 - 28
packages/ckeditor5-engine/src/conversion/model-to-view-converters.js

@@ -3,10 +3,6 @@
  * For licensing, see LICENSE.md.
  */
 
-/**
- * @module engine/conversion/model-to-view-converters
- */
-
 import ModelRange from '../model/range.js';
 import ModelPosition from '../model/position.js';
 import ModelElement from '../model/element.js';
@@ -22,7 +18,7 @@ import viewWriter from '../view/writer.js';
  * Contains {@link module:engine/model/model model} to {@link module:engine/view/view view} converters for
  * {@link module:engine/conversion/modelconversiondispatcher~ModelConversionDispatcher}.
  *
- * @namespace module:engine/conversion/modeltoview~modelToView
+ * @module engine/conversion/model-to-view-converters
  */
 
 /**
@@ -49,8 +45,6 @@ import viewWriter from '../view/writer.js';
  *			}
  *		) );
  *
- * @external module:engine/conversion/modeltoview~modelToView
- * @function module:engine/conversion/modeltoview~modelToView.insertElement
  * @param {module:engine/view/element~Element|Function} elementCreator View element, or function returning a view element, which
  * will be inserted.
  * @returns {Function} Insert element event converter.
@@ -79,8 +73,6 @@ export function insertElement( elementCreator ) {
  *
  *		modelDispatcher.on( 'insert:$text', insertText() );
  *
- * @external module:engine/conversion/modeltoview~modelToView
- * @function module:engine/conversion/modeltoview~modelToView.insertText
  * @returns {Function} Insert text event converter.
  */
 export function insertText() {
@@ -121,13 +113,11 @@ export function insertText() {
  *			return { key, value };
  *		} ) );
  *
- * @external module:engine/conversion/modeltoview~modelToView
- * @function module:engine/conversion/modeltoview~modelToView.setAttribute
  * @param {Function} [attributeCreator] Function returning an object with two properties: `key` and `value`, which
  * represents attribute key and attribute value to be set on a {@link module:engine/view/element~Element view element}.
  * The function is passed all the parameters of the
  * {@link module:engine/conversion/modelconversiondispatcher~ModelConversionDispatcher#event:addAttribute}
- * or {@link module:engine/conversion/modelconversiondispatcher~ModelConversionDispatcher.changeAttribute} event.
+ * or {@link module:engine/conversion/modelconversiondispatcher~ModelConversionDispatcher#event:changeAttribute} event.
  * @returns {Function} Set/change attribute converter.
  */
 export function setAttribute( attributeCreator ) {
@@ -151,7 +141,7 @@ export function setAttribute( attributeCreator ) {
  *
  * **Note:** Provided attribute creator should always return the same `key` for given attribute from the model.
  *
- * **Note:** You can use the same attribute creator as in {@link module:engine/conversion/modeltoview~modelToView.setAttribute}.
+ * **Note:** You can use the same attribute creator as in {@link module:engine/conversion/model-to-view-converters~setAttribute}.
  *
  * The converter automatically consumes corresponding value from consumables list and stops the event (see
  * {@link module:engine/conversion/modelconversiondispatcher~ModelConversionDispatcher}).
@@ -170,8 +160,6 @@ export function setAttribute( attributeCreator ) {
  *			return { key, value };
  *		} ) );
  *
- * @external module:engine/conversion/modeltoview~modelToView
- * @function module:engine/conversion/modeltoview~modelToView.removeAttribute
  * @param {Function} [attributeCreator] Function returning an object with two properties: `key` and `value`, which
  * represents attribute key and attribute value to be removed from {@link module:engine/view/element~Element view element}.
  * The function is passed all the parameters of the
@@ -206,7 +194,7 @@ export function removeAttribute( attributeCreator ) {
  *
  * The wrapping node depends on passed parameter. If {@link module:engine/view/element~Element} was passed, it will be cloned and
  * the copy will become the wrapping element. If `Function` is provided, it is passed all the parameters of the
- * {@link module:engine/conversion/modelconversiondispatcher~ModelConversionDispatcher#event:setAttribute setAttribute event}.
+ * {@link module:engine/conversion/modelconversiondispatcher~ModelConversionDispatcher#event:addAttribute addAttribute event}.
  * It's expected that the function returns a {@link module:engine/view/element~Element}.
  * The result of the function will be the wrapping element.
  * When provided `Function` does not return element, then will be no conversion.
@@ -216,8 +204,6 @@ export function removeAttribute( attributeCreator ) {
  *
  *		modelDispatcher.on( 'addAttribute:bold', wrap( new ViewElement( 'strong' ) ) );
  *
- * @external module:engine/conversion/modeltoview~modelToView
- * @function module:engine/conversion/modeltoview~modelToView.wrap
  * @param {module:engine/view/element~Element|Function} elementCreator View element, or function returning a view element, which will
  * be used for wrapping.
  * @returns {Function} Set/change attribute converter.
@@ -256,7 +242,7 @@ export function wrap( elementCreator ) {
  * The view element type that will be unwrapped depends on passed parameter.
  * If {@link module:engine/view/element~Element} was passed, it will be used to look for similar element in the view for unwrapping.
  * If `Function` is provided, it is passed all the parameters of the
- * {@link module:engine/conversion/modelconversiondispatcher~ModelConversionDispatcher#event:setAttribute setAttribute event}.
+ * {@link module:engine/conversion/modelconversiondispatcher~ModelConversionDispatcher#event:addAttribute addAttribute event}.
  * It's expected that the function returns a {@link module:engine/view/element~Element}.
  * The result of the function will be used to look for similar element in the view for unwrapping.
  *
@@ -265,9 +251,7 @@ export function wrap( elementCreator ) {
  *
  *		modelDispatcher.on( 'removeAttribute:bold', unwrap( new ViewElement( 'strong' ) ) );
  *
- * @see module:engine/conversion/modeltoview~modelToView.wrap
- * @external module:engine/conversion/modeltoview~modelToView
- * @function module:engine/conversion/modeltoview~modelToView.unwrap
+ * @see module:engine/conversion/model-to-view-converters~wrap
  * @param {module:engine/view/element~Element|Function} elementCreator View element, or function returning a view element, which will
  * be used for unwrapping.
  * @returns {Function} Remove attribute converter.
@@ -292,8 +276,6 @@ export function unwrap( elementCreator ) {
  *
  *		modelDispatcher.on( 'move', move() );
  *
- * @external module:engine/conversion/modeltoview~modelToView
- * @function module:engine/conversion/modeltoview~modelToView.move
  * @returns {Function} Move event converter.
  */
 export function move() {
@@ -324,8 +306,6 @@ export function move() {
  *
  *		modelDispatcher.on( 'remove', remove() );
  *
- * @external module:engine/conversion/modeltoview~modelToView
- * @function module:engine/conversion/modeltoview~modelToView.remove
  * @returns {Function} Remove event converter.
  */
 export function remove() {
@@ -377,8 +357,6 @@ function findViewTextRange( start, size ) {
  *
  * This converter re-uses converters added for `insert`, `move` and `remove` change types.
  *
- * @external module:engine/conversion/modeltoview~modelToView
- * @function module:engine/conversion/modeltoview~modelToView.rename
  * @fires module:engine/conversion/modelconversiondispatcher~ModelConversionDispatcher#event:insert
  * @fires module:engine/conversion/modelconversiondispatcher~ModelConversionDispatcher#event:move
  * @fires module:engine/conversion/modelconversiondispatcher~ModelConversionDispatcher#event:remove

+ 1 - 1
packages/ckeditor5-engine/src/conversion/modelconsumable.js

@@ -29,7 +29,7 @@ import TextProxy from '../model/textproxy.js';
  * @link module:engine/conversion/modelconsumable~ModelConsumable#add add method} directly.
  * However, it is important to understand how consumable values can be
  * {@link module:engine/conversion/modelconsumable~ModelConsumable#consume consumed}.
- * See {@link module:engine/conversion/modeltoview~modelToView default model to view converters} for more information.
+ * See {@link module:engine/conversion/model-selection-to-view-converters default model to view converters} for more information.
  *
  * Keep in mind, that one conversion event may have multiple callbacks (converters) attached to it. Each of those is
  * able to convert one or more parts of the model. However, when one of those callbacks actually converts

+ 1 - 6
packages/ckeditor5-engine/src/conversion/view-selection-to-model-converters.js

@@ -4,15 +4,10 @@
  */
 
 /**
- * @module engine/conversion/view-selection-to-model-converters
- */
-
-/**
  * Contains {@link module:engine/view/selection~Selection view selection}
  * to {@link module:engine/model/selection~Selection model selection} conversion
+ * @module engine/conversion/view-selection-to-model-converters
  * helper.
- *
- * @namespace viewSelectionToModel
  */
 
 import ModelSelection from '../model/selection.js';

+ 1 - 9
packages/ckeditor5-engine/src/conversion/view-to-model-converters.js

@@ -3,10 +3,6 @@
  * For licensing, see LICENSE.md.
  */
 
-/**
- * @module engine/conversion/view-to-model-converters
- */
-
 import ModelDocumentFragment from '../model/documentfragment.js';
 import ModelText from '../model/text.js';
 import { normalizeNodes } from '../model/writer.js';
@@ -15,7 +11,7 @@ import { normalizeNodes } from '../model/writer.js';
  * Contains {@link module:engine/view/view view} to {@link module:engine/model/model model} converters for
  * {@link module:engine/conversion/viewconversiondispatcher~ViewConversionDispatcher}.
  *
- * @namespace viewToModel
+ * @module engine/conversion/view-to-model-converters
  */
 
 /**
@@ -29,8 +25,6 @@ import { normalizeNodes } from '../model/writer.js';
  * When a view element is being converted to the model but it does not have converter specified, that view element
  * will be converted to {@link module:engine/model/documentfragment~DocumentFragment model document fragment} and returned.
  *
- * @external module:engine/conversion/view-to-model-converters~viewToModel
- * @function module:engine/conversion/view-to-model-converters~viewToModel.convertToModelFragment
  * @returns {Function} Universal converter for view {@link module:engine/view/documentfragment~DocumentFragment fragments} and
  * {@link module:engine/view/element~Element elements} that returns
  * {@link module:engine/model/documentfragment~DocumentFragment model fragment} with children of converted view item.
@@ -49,8 +43,6 @@ export function convertToModelFragment() {
 /**
  * Function factory, creates a converter that converts {@link module:engine/view/text~Text} to {@link module:engine/model/text~Text}.
  *
- * @external module:engine/conversion/view-to-model-converters~viewToModel
- * @function module:engine/conversion/view-to-model-converters~viewToModel.convertText
  * @returns {Function} {@link module:engine/view/text~Text View text} converter.
  */
 export function convertText() {

+ 2 - 2
packages/ckeditor5-engine/src/conversion/viewconversiondispatcher.js

@@ -89,10 +89,10 @@ import extend from '../../utils/lib/lodash/extend.js';
  *		// is going to be appended directly to a '$root' element, use that in `context`.
  *		viewDispatcher.convert( viewDocumentFragment, { context: [ '$root' ] } );
  *
- * Before each conversion process, `ViewConversionDispatcher` fires {@link ~ViewConversionDispatcher#viewCleanup}
+ * Before each conversion process, `ViewConversionDispatcher` fires {@link ~ViewConversionDispatcher#event:viewCleanup}
  * event which can be used to prepare tree view for conversion.
  *
- * @mixes utils.EmitterMixin
+ * @mixes module:utils/emittermixin~EmitterMixin
  * @fires viewCleanup
  * @fires element
  * @fires text

+ 3 - 3
packages/ckeditor5-engine/src/model/delta/splitdelta.js

@@ -53,12 +53,12 @@ export default class SplitDelta extends Delta {
 	 * as `SplitDelta` has to create a new node. If `SplitDelta` was created through
 	 * {@link module:engine/model/delta/delta~Delta#getReversed reversing}
 	 * a {@link module:engine/model/delta/mergedelta~MergeDelta merge delta},
-	 * this will be a {@link module:engine/model/opertation/reinsertoperation~ReinsertOperation reinsert operation},
+	 * this will be a {@link module:engine/model/operation/reinsertoperation~ReinsertOperation reinsert operation},
 	 * as we will want to re-insert the exact element that was removed by that merge delta.
 	 *
 	 * @protected
-	 * @type {module:engine/model/opertation/insertoperation~InsertOperation|
-	 * module:engine/model/opertation/reinsertoperation~ReinsertOperation|null}
+	 * @type {module:engine/model/operation/insertoperation~InsertOperation|
+	 * module:engine/model/operation/reinsertoperation~ReinsertOperation|null}
 	 */
 	get _cloneOperation() {
 		return this.operations[ 0 ] || null;

+ 3 - 3
packages/ckeditor5-engine/src/model/delta/transform.js

@@ -26,14 +26,14 @@ const specialCases = new Map();
  * we need to handle transformations in special cases in a custom way.
  *
  * The function itself looks whether two given delta types have a special case function registered. If so, the deltas are
- * transformed using that function. If not, {@link module:engine/model/delta/delta~Delta.defaultTransform default transformation algorithm}
+ * transformed using that function. If not, {@link module:engine/model/delta/transform~defaultTransform default transformation algorithm}
  * is used.
  *
  * @param {module:engine/model/delta/delta~Delta} a Delta that will be transformed.
  * @param {module:engine/model/delta/delta~Delta} b Delta to transform by.
  * @param {Boolean} isAMoreImportantThanB Flag indicating whether the delta which will be transformed (`a`) should be treated
  * as more important when resolving conflicts. Note that this flag is used only if provided deltas have same
- * {@link module:engine/model/delta/delta~Delta.priorities priority}. If deltas have different priorities, their importance is resolved
+ * {@link module:engine/model/delta/delta~Delta._priority priority}. If deltas have different priorities, their importance is resolved
  * automatically and overwrites this flag.
  * @returns {Array.<module:engine/model/delta/delta~Delta>} Result of the transformation.
  */
@@ -67,7 +67,7 @@ function updateBaseVersion( baseVersion, deltas ) {
  * @param {module:engine/model/delta/delta~Delta} b Delta to transform by.
  * @param {Boolean} isAMoreImportantThanB Flag indicating whether the delta which will be transformed (`a`) should be treated
  * as more important when resolving conflicts. Note that this flag is used only if provided deltas have same
- * {@link module:engine/model/delta/delta~Delta.priorities priority}. If deltas have different priorities, their importance is resolved
+ * {@link module:engine/model/delta/delta~Delta._priority priority}. If deltas have different priorities, their importance is resolved
  * automatically and overwrites this flag.
  * @returns {Array.<module:engine/model/delta/delta~Delta>} Result of the transformation, that is an array with single delta instance.
  */

+ 1 - 1
packages/ckeditor5-engine/src/model/delta/wrapdelta.js

@@ -50,7 +50,7 @@ export default class WrapDelta extends Delta {
 	 * Operation that inserts wrapping element or `null` if there are no operations in the delta.
 	 *
 	 * @protected
-	 * @type {module:engine/model/operation/insertoperation~InsertOperation|module:engine/model/opertation/reinsertoperation~ReinsertOperation}
+	 * @type {module:engine/model/operation/insertoperation~InsertOperation|module:engine/model/operation/reinsertoperation~ReinsertOperation}
 	 */
 	get _insertOperation() {
 		return this.operations[ 0 ] || null;

+ 2 - 1
packages/ckeditor5-engine/src/model/document.js

@@ -38,6 +38,7 @@ const graveyardName = '$graveyard';
  *		doc.batch().insert( position, nodes ).split( otherPosition );
  *
  * @see ~Document#batch
+ * @mixes module:utils/emittermixin~EmitterMixin
  */
 export default class Document {
 	/**
@@ -123,7 +124,7 @@ export default class Document {
 	 * Graveyard tree root. Document always have a graveyard root, which stores removed nodes.
 	 *
 	 * @readonly
-	 * @type {module:engine/model/rootelement~RootElement}
+	 * @member {module:engine/model/rootelement~RootElement}
 	 */
 	get graveyard() {
 		return this.getRoot( graveyardName );

+ 1 - 1
packages/ckeditor5-engine/src/model/documentfragment.js

@@ -150,7 +150,7 @@ export default class DocumentFragment {
 	 * Converts offset "position" to index "position".
 	 *
 	 * Returns index of a node that occupies given offset. If given offset is too low, returns `0`. If given offset is
-	 * too high, returns {@link module:engine/model/documentfragment~DocumentFragment#getChildCount index after last child}.
+	 * too high, returns index after last child}.
 	 *
 	 *		const textNode = new Text( 'foo' );
 	 *		const pElement = new Element( 'p' );

+ 1 - 1
packages/ckeditor5-engine/src/model/node.js

@@ -19,7 +19,7 @@ import CKEditorError from '../../utils/ckeditorerror.js';
  * However, it is **very important** that nodes already attached to model tree should be only changed through
  * {@link module:engine/model/document~Document#batch Batch API}.
  *
- * Changes done by `Node` methods, like {@link module:engine/model/node~Node#insertChildren insertChildren} or
+ * Changes done by `Node` methods, like {@link module:engine/model/element~Element#insertChildren insertChildren} or
  * {@link module:engine/model/node~Node#setAttribute setAttribute}
  * do not generate {@link module:engine/model/operation/operation~Operation operations}
  * which are essential for correct editor work if you modify nodes in {@link module:engine/model/document~Document document} root.

+ 1 - 1
packages/ckeditor5-engine/src/model/nodelist.js

@@ -131,7 +131,7 @@ export default class NodeList {
 	 * Converts offset in node list to index.
 	 *
 	 * Returns index of a node that occupies given offset. Throws {@link module:utils/ckeditorerror~CKEditorError CKEditorError}
-	 * `model-nodelist-offset-out-of-bounds` if given offset is less than `0` or more than {@link #getMaxOffset}.
+	 * `model-nodelist-offset-out-of-bounds` if given offset is less than `0` or more than {@link #maxOffset}.
 	 *
 	 * @param {Number} offset Offset to look for.
 	 * @returns {Number} Index of a node that occupies given offset.

+ 1 - 1
packages/ckeditor5-engine/src/model/operation/operation.js

@@ -26,7 +26,7 @@ export default class Operation {
 		 * {@link module:engine/model/document~Document#version document version} the
 		 * {@link module:utils/ckeditorerror~CKEditorError model-document-applyOperation-wrong-version} error is thrown.
 		 *
-		 * @member {Number} #baseVersion
+		 * @member {Number}
 		 */
 		this.baseVersion = baseVersion;
 

+ 1 - 1
packages/ckeditor5-engine/src/model/operation/removeoperation.js

@@ -94,7 +94,7 @@ export default class RemoveOperation extends MoveOperation {
 
 	/**
 	 * @inheritDoc
-	 * @returns {module:engine/model/opertation/reinsertoperation~ReinsertOperation}
+	 * @returns {module:engine/model/operation/reinsertoperation~ReinsertOperation}
 	 */
 	getReversed() {
 		return new ReinsertOperation( this.targetPosition, this.howMany, this.sourcePosition, this.baseVersion + 1 );

+ 2 - 2
packages/ckeditor5-engine/src/model/operation/transform.js

@@ -31,10 +31,10 @@ import compareArrays from '../../../utils/comparearrays.js';
  * connected with transformed operation, those changes will be reflected in the parameters of the returned operation(s).
  *
  * Whenever the {@link module:engine/model/document~Document document}
- * has different {@link module:engine/model/document~Document#baseVersion}
+ * has different {@link module:engine/model/document~Document#version}
  * than the operation you want to {@link module:engine/model/document~Document#applyOperation apply}, you need to transform that
  * operation by all operations which were already applied to the {@link module:engine/model/document~Document document} and have greater
- * {@link module:engine/model/document~Document#baseVersion} than the operation being applied. Transform them in the same order as those
+ * {@link module:engine/model/document~Document#version} than the operation being applied. Transform them in the same order as those
  * operations which were applied. This way all modifications done to the Tree Data Model will be reflected
  * in the operation parameters and the operation will "operate" on "up-to-date" version of the Tree Data Model.
  * This is mostly the case with Operational Transformations but it might be needed in particular features as well.

+ 1 - 1
packages/ckeditor5-engine/src/model/position.js

@@ -149,7 +149,7 @@ export default class Position {
 
 	/**
 	 * Position {@link module:engine/model/position~Position#offset offset} converted to an index in position's parent node. It is
-	 * equal to the {@link module:engine/model/node~Node#getIndex index} of a node after this position. If position is placed
+	 * equal to the {@link module:engine/model/node~Node#index index} of a node after this position. If position is placed
 	 * in text node, position index is equal to the index of that text node.
 	 *
 	 * @readonly

+ 2 - 2
packages/ckeditor5-engine/src/view/attributeelement.js

@@ -36,12 +36,12 @@ export default class AttributeElement extends Element {
 		 * {@link module:engine/view/element~Element#isSimilar similar}. Setting different priorities on similar
  		 * nodes may prevent merging, e.g. two `<abbr>` nodes next each other shouldn't be merged.
 		 *
-		 * @member {Number} module:engine/view/attributeelement~AttributeElement#priority
+		 * @member {Number}
 		 */
 		this.priority = DEFAULT_PRIORITY;
 
 		/**
-		 * Returns block {@link module:engine/view/filler~Filler filler} offset or `null` if block filler is not needed.
+		 * Returns block {@link module:engine/view/filler filler} offset or `null` if block filler is not needed.
 		 *
 		 * @method #getFillerOffset
 		 * @returns {Number|null} Block filler offset or `null` if block filler is not needed.

+ 2 - 2
packages/ckeditor5-engine/src/view/containerelement.js

@@ -54,7 +54,7 @@ export default class ContainerElement extends Element {
 		super( name, attrs, children );
 
 		/**
-		 * Returns block {@link module:engine/view/filler~Filler filler} offset or `null` if block filler is not needed.
+		 * Returns block {@link module:engine/view/filler filler} offset or `null` if block filler is not needed.
 		 *
 		 * @method #getFillerOffset
 		 * @returns {Number|null} Block filler offset or `null` if block filler is not needed.
@@ -63,7 +63,7 @@ export default class ContainerElement extends Element {
 	}
 }
 
-// Returns block {@link module:engine/view/filler~Filler filler} offset or `null` if block filler is not needed.
+// Returns block {@link module:engine/view/filler filler} offset or `null` if block filler is not needed.
 //
 // @returns {Number|null} Block filler offset or `null` if block filler is not needed.
 function getFillerOffset() {

+ 3 - 2
packages/ckeditor5-engine/src/view/document.js

@@ -38,7 +38,7 @@ import ObservableMixin from '../../utils/observablemixin.js';
  * * {@link module:engine/view/observer/keyobserver~KeyObserver},
  * * {@link module:engine/view/observer/fakeselectionobserver~FakeSelectionObserver}.
  *
- * @mixes module:utils/emittermixin~EmitterMixin
+ * @mixes module:utils/observablemixin~ObservableMixin
  */
 export default class Document {
 	/**
@@ -182,7 +182,8 @@ export default class Document {
 	 *		document.createRoot( 'body' );
 	 *		document.attachDomRoot( document.querySelector( 'body#editor' ) );
 	 *
-	 * In both cases, {@link module:engine/view/rooteditableelement~RootEditableElement#name element name} is always transformed to lower case.
+	 * In both cases, {@link module:engine/view/rooteditableelement~RootEditableElement#rootName element name} is always transformed to lower
+	 * case.
 	 *
 	 * @param {Element|String} domRoot DOM root element or the tag name of view root element if the DOM element will be
 	 * attached later.

+ 1 - 1
packages/ckeditor5-engine/src/view/editableelement.js

@@ -17,7 +17,7 @@ import ObservableMixin from '../../utils/observablemixin.js';
  * or nested editable area in the editor.
  *
  * @extends module:engine/view/containerelement~ContainerElement
- * @mixes utils.ObservaleMixin
+ * @mixes module:utils/observablemixin~ObservaleMixin
  */
 export default class EditableElement extends ContainerElement {
 	/**

+ 2 - 2
packages/ckeditor5-engine/src/view/element.js

@@ -173,7 +173,7 @@ export default class Element extends Node {
 	 * {@link module:engine/view/element~Element#insertChildren Insert} a child node or a list of child nodes at the end of this node and sets
 	 * the parent of these nodes to this element.
 	 *
-	 * @fires change
+	 * @fires module:engine/view/node~Node#change
 	 * @param {module:engine/view/node~Node|Iterable.<module:engine/view/node~Node>} nodes Node or the list of nodes to be inserted.
 	 * @returns {Number} Number of appended nodes.
 	 */
@@ -655,7 +655,7 @@ export default class Element extends Node {
 	}
 
 	/**
-	 * Returns block {@link module:engine/view/filler~Filler filler} offset or `null` if block filler is not needed.
+	 * Returns block {@link module:engine/view/filler filler} offset or `null` if block filler is not needed.
 	 *
 	 * @abstract
 	 * @method module:engine/view/element~Element#getFillerOffset

+ 2 - 3
packages/ckeditor5-engine/src/view/observer/domeventobserver.js

@@ -68,10 +68,9 @@ export default class DomEventObserver extends Observer {
 	}
 
 	/**
-	 * Calls {@link module:engine/view/document~Document#fire} if observer
-	 * {@link #isEnabled is enabled}.
+	 * Calls `Document#fire()` if observer {@link #isEnabled is enabled}.
 	 *
-	 * @see module:engine/view/document~Document#fire
+	 * @see module:utils/emittermixin~EmitterMixin#fire
 	 * @param {String} eventType The event type (name).
 	 * @param {Event} domEvent The DOM event.
 	 * @param {Object} [additionalData] The additional data which should extend the

+ 1 - 1
packages/ckeditor5-engine/src/view/observer/fakeselectionobserver.js

@@ -15,7 +15,7 @@ import { keyCodes } from '../../../utils/keyboard.js';
  * Fake selection observer class. If view selection is fake it is placed in dummy DOM container. This observer listens
  * on {@link module:engine/view/document~Document#event:keydown keydown} events and handles moving fake view selection to the correct place
  * if arrow keys are pressed.
- * Fires {@link module:engine/view/document~Document#selectionChage selectionChange event} simulating natural behaviour of
+ * Fires {@link module:engine/view/document~Document#event:selectionChange selectionChange event} simulating natural behaviour of
  * {@link module:engine/view/observer/selectionobserver~SelectionObserver SelectionObserver}.
  *
  * @extends module:engine/view/observer/observer~Observer.Observer

+ 1 - 1
packages/ckeditor5-engine/src/view/observer/keyobserver.js

@@ -58,7 +58,7 @@ export default class KeyObserver extends DomEventObserver {
  *
  * @class module:engine/view/observer/keyobserver~KeyObserver.KeyEventData
  * @extends module:engine/view/observer/domeventdata~DomEventData
- * @implements module:utils/keyboard/keystorkedata~KeystrokeData
+ * @implements module:utils/keyboard~KeystrokeInfo
  */
 
 /**

+ 3 - 2
packages/ckeditor5-engine/src/view/observer/mutationobserver.js

@@ -15,8 +15,9 @@ import { startsWithFiller, getDataWithoutFiller } from '../filler.js';
 
 /**
  * Mutation observer class observes changes in the DOM, fires {@link module:engine/view/document~Document#event:mutations} event, mark view
- * elements as changed and call {@link module:engine/view/view~render}. Because all mutated nodes are marked as "to be rendered" and the
- * {@link module:engine/view/view.render} is called, all changes will be reverted, unless the mutation will be handled by the
+ * elements as changed and call {@link module:engine/view/renderer~Renderer#render}.
+ * Because all mutated nodes are marked as "to be rendered" and the
+ * {@link module:engine/view/renderer~Renderer#render} is called, all changes will be reverted, unless the mutation will be handled by the
  * {@link module:engine/view/document~Document#event:mutations} event listener. It means user will see only handled changes, and the editor
  * will block all changes which are not handled.
  *

+ 1 - 1
packages/ckeditor5-engine/src/view/rooteditableelement.js

@@ -32,7 +32,7 @@ export default class RootEditableElement extends EditableElement {
 		 * Name of this root inside {@link module:engine/view/document~Document} that is an owner of this root.
 		 *
 		 * @readonly
-		 * @member {String} module:engine/view/rooteditableelement~RootEditableElement#rootName
+		 * @member {String}
 		 */
 		this.rootName = rootName;
 	}