@ckeditor/ckeditor5-enter 35.2.1 → 35.3.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ckeditor/ckeditor5-enter",
-  "version": "35.2.1",
+  "version": "35.3.0",
   "description": "Enter feature for CKEditor 5.",
   "keywords": [
     "ckeditor",
@@ -12,18 +12,20 @@
   ],
   "main": "src/index.js",
   "dependencies": {
-    "@ckeditor/ckeditor5-core": "^35.2.1",
-    "@ckeditor/ckeditor5-engine": "^35.2.1",
-    "@ckeditor/ckeditor5-utils": "^35.2.1"
+    "@ckeditor/ckeditor5-core": "^35.3.0",
+    "@ckeditor/ckeditor5-engine": "^35.3.0"
   },
   "devDependencies": {
-    "@ckeditor/ckeditor5-basic-styles": "^35.2.1",
-    "@ckeditor/ckeditor5-editor-classic": "^35.2.1",
-    "@ckeditor/ckeditor5-heading": "^35.2.1",
-    "@ckeditor/ckeditor5-link": "^35.2.1",
-    "@ckeditor/ckeditor5-paragraph": "^35.2.1",
-    "@ckeditor/ckeditor5-typing": "^35.2.1",
-    "@ckeditor/ckeditor5-undo": "^35.2.1"
+    "@ckeditor/ckeditor5-basic-styles": "^35.3.0",
+    "@ckeditor/ckeditor5-editor-classic": "^35.3.0",
+    "@ckeditor/ckeditor5-heading": "^35.3.0",
+    "@ckeditor/ckeditor5-link": "^35.3.0",
+    "@ckeditor/ckeditor5-paragraph": "^35.3.0",
+    "@ckeditor/ckeditor5-typing": "^35.3.0",
+    "@ckeditor/ckeditor5-undo": "^35.3.0",
+    "typescript": "^4.8.4",
+    "webpack": "^5.58.1",
+    "webpack-cli": "^4.9.0"
   },
   "engines": {
     "node": ">=14.0.0",
@@ -40,9 +42,14 @@
   },
   "files": [
     "lang",
-    "src",
+    "src/**/*.js",
+    "src/**/*.d.ts",
     "theme",
     "ckeditor5-metadata.json",
     "CHANGELOG.md"
-  ]
+  ],
+  "scripts": {
+    "build": "tsc -p ./tsconfig.release.json",
+    "postversion": "npm run build"
+  }
 }

package/src/enter.js

@@ -2,15 +2,12 @@
  * @license Copyright (c) 2003-2022, CKSource Holding sp. z o.o. All rights reserved.
  * For licensing, see LICENSE.md or https://ckeditor.com/legal/ckeditor-oss-license
  */
-
 /**
  * @module enter/enter
  */
-
 import Plugin from '@ckeditor/ckeditor5-core/src/plugin';
 import EnterCommand from './entercommand';
 import EnterObserver from './enterobserver';
-
 /**
  * This plugin handles the <kbd>Enter</kbd> key (hard line break) in the editor.
  *
@@ -21,33 +18,30 @@ import EnterObserver from './enterobserver';
  * @extends module:core/plugin~Plugin
  */
 export default class Enter extends Plugin {
-	/**
-	 * @inheritDoc
-	 */
-	static get pluginName() {
-		return 'Enter';
-	}
-
-	init() {
-		const editor = this.editor;
-		const view = editor.editing.view;
-		const viewDocument = view.document;
-
-		view.addObserver( EnterObserver );
-
-		editor.commands.add( 'enter', new EnterCommand( editor ) );
-
-		this.listenTo( viewDocument, 'enter', ( evt, data ) => {
-			data.preventDefault();
-
-			// The soft enter key is handled by the ShiftEnter plugin.
-			if ( data.isSoft ) {
-				return;
-			}
-
-			editor.execute( 'enter' );
-
-			view.scrollToTheSelection();
-		}, { priority: 'low' } );
-	}
+    /**
+     * @inheritDoc
+     */
+    static get pluginName() {
+        return 'Enter';
+    }
+    init() {
+        const editor = this.editor;
+        const view = editor.editing.view;
+        const viewDocument = view.document;
+        view.addObserver(EnterObserver);
+        editor.commands.add('enter', new EnterCommand(editor));
+        this.listenTo(viewDocument, 'enter', (evt, data) => {
+            // When not in composition, we handle the action, so prevent the default one.
+            // When in composition, it's the browser who modify the DOM (renderer is disabled).
+            if (!viewDocument.isComposing) {
+                data.preventDefault();
+            }
+            // The soft enter key is handled by the ShiftEnter plugin.
+            if (data.isSoft) {
+                return;
+            }
+            editor.execute('enter');
+            view.scrollToTheSelection();
+        }, { priority: 'low' });
+    }
 }

package/src/entercommand.js

@@ -2,34 +2,29 @@
  * @license Copyright (c) 2003-2022, CKSource Holding sp. z o.o. All rights reserved.
  * For licensing, see LICENSE.md or https://ckeditor.com/legal/ckeditor-oss-license
  */
-
 /**
  * @module enter/entercommand
  */
-
 import Command from '@ckeditor/ckeditor5-core/src/command';
 import { getCopyOnEnterAttributes } from './utils';
-
 /**
  * Enter command. It is used by the {@link module:enter/enter~Enter Enter feature} to handle the <kbd>Enter</kbd> key.
  *
  * @extends module:core/command~Command
  */
 export default class EnterCommand extends Command {
-	/**
-	 * @inheritDoc
-	 */
-	execute() {
-		const model = this.editor.model;
-		const doc = model.document;
-
-		model.change( writer => {
-			enterBlock( this.editor.model, writer, doc.selection, model.schema );
-			this.fire( 'afterExecute', { writer } );
-		} );
-	}
+    /**
+     * @inheritDoc
+     */
+    execute() {
+        const model = this.editor.model;
+        const doc = model.document;
+        model.change(writer => {
+            enterBlock(this.editor.model, writer, doc.selection, model.schema);
+            this.fire('afterExecute', { writer });
+        });
+    }
 }
-
 // Creates a new block in the way that the <kbd>Enter</kbd> key is expected to work.
 //
 // @param {module:engine/model~Model} model
@@ -37,53 +32,48 @@ export default class EnterCommand extends Command {
 // @param {module:engine/model/selection~Selection|module:engine/model/documentselection~DocumentSelection} selection
 // Selection on which the action should be performed.
 // @param {module:engine/model/schema~Schema} schema
-function enterBlock( model, writer, selection, schema ) {
-	const isSelectionEmpty = selection.isCollapsed;
-	const range = selection.getFirstRange();
-	const startElement = range.start.parent;
-	const endElement = range.end.parent;
-
-	// Don't touch the roots and other limit elements.
-	if ( schema.isLimit( startElement ) || schema.isLimit( endElement ) ) {
-		// Delete the selected content but only if inside a single limit element.
-		// Abort, when crossing limit elements boundary (e.g. <limit1>x[x</limit1>donttouchme<limit2>y]y</limit2>).
-		// This is an edge case and it's hard to tell what should actually happen because such a selection
-		// is not entirely valid.
-		if ( !isSelectionEmpty && startElement == endElement ) {
-			model.deleteContent( selection );
-		}
-
-		return;
-	}
-
-	if ( isSelectionEmpty ) {
-		const attributesToCopy = getCopyOnEnterAttributes( writer.model.schema, selection.getAttributes() );
-		splitBlock( writer, range.start );
-		writer.setSelectionAttribute( attributesToCopy );
-	} else {
-		const leaveUnmerged = !( range.start.isAtStart && range.end.isAtEnd );
-		const isContainedWithinOneElement = ( startElement == endElement );
-
-		model.deleteContent( selection, { leaveUnmerged } );
-
-		if ( leaveUnmerged ) {
-			// Partially selected elements.
-			//
-			// <h>x[xx]x</h>		-> <h>x^x</h>			-> <h>x</h><h>^x</h>
-			if ( isContainedWithinOneElement ) {
-				splitBlock( writer, selection.focus );
-			}
-			// Selection over multiple elements.
-			//
-			// <h>x[x</h><p>y]y<p>	-> <h>x^</h><p>y</p>	-> <h>x</h><p>^y</p>
-			else {
-				writer.setSelection( endElement, 0 );
-			}
-		}
-	}
+function enterBlock(model, writer, selection, schema) {
+    const isSelectionEmpty = selection.isCollapsed;
+    const range = selection.getFirstRange();
+    const startElement = range.start.parent;
+    const endElement = range.end.parent;
+    // Don't touch the roots and other limit elements.
+    if (schema.isLimit(startElement) || schema.isLimit(endElement)) {
+        // Delete the selected content but only if inside a single limit element.
+        // Abort, when crossing limit elements boundary (e.g. <limit1>x[x</limit1>donttouchme<limit2>y]y</limit2>).
+        // This is an edge case and it's hard to tell what should actually happen because such a selection
+        // is not entirely valid.
+        if (!isSelectionEmpty && startElement == endElement) {
+            model.deleteContent(selection);
+        }
+        return;
+    }
+    if (isSelectionEmpty) {
+        const attributesToCopy = getCopyOnEnterAttributes(writer.model.schema, selection.getAttributes());
+        splitBlock(writer, range.start);
+        writer.setSelectionAttribute(attributesToCopy);
+    }
+    else {
+        const leaveUnmerged = !(range.start.isAtStart && range.end.isAtEnd);
+        const isContainedWithinOneElement = (startElement == endElement);
+        model.deleteContent(selection, { leaveUnmerged });
+        if (leaveUnmerged) {
+            // Partially selected elements.
+            //
+            // <h>x[xx]x</h>		-> <h>x^x</h>			-> <h>x</h><h>^x</h>
+            if (isContainedWithinOneElement) {
+                splitBlock(writer, selection.focus);
+            }
+            // Selection over multiple elements.
+            //
+            // <h>x[x</h><p>y]y<p>	-> <h>x^</h><p>y</p>	-> <h>x</h><p>^y</p>
+            else {
+                writer.setSelection(endElement, 0);
+            }
+        }
+    }
 }
-
-function splitBlock( writer, splitPos ) {
-	writer.split( splitPos );
-	writer.setSelection( splitPos.parent.nextSibling, 0 );
+function splitBlock(writer, splitPos) {
+    writer.split(splitPos);
+    writer.setSelection(splitPos.parent.nextSibling, 0);
 }

package/src/enterobserver.js

@@ -2,61 +2,50 @@
  * @license Copyright (c) 2003-2022, CKSource Holding sp. z o.o. All rights reserved.
  * For licensing, see LICENSE.md or https://ckeditor.com/legal/ckeditor-oss-license
  */
-
 /**
  * @module enter/enterobserver
  */
-
 import Observer from '@ckeditor/ckeditor5-engine/src/view/observer/observer';
 import DomEventData from '@ckeditor/ckeditor5-engine/src/view/observer/domeventdata';
 import BubblingEventInfo from '@ckeditor/ckeditor5-engine/src/view/observer/bubblingeventinfo';
-import { keyCodes } from '@ckeditor/ckeditor5-utils/src/keyboard';
-
+const ENTER_EVENT_TYPES = {
+    insertParagraph: { isSoft: false },
+    insertLineBreak: { isSoft: true }
+};
 /**
  * Enter observer introduces the {@link module:engine/view/document~Document#event:enter `Document#enter`} event.
  *
  * @extends module:engine/view/observer/observer~Observer
  */
 export default class EnterObserver extends Observer {
-	/**
-	 * @inheritDoc
-	 */
-	constructor( view ) {
-		super( view );
-
-		const doc = this.document;
-
-		doc.on( 'keydown', ( evt, data ) => {
-			if ( this.isEnabled && data.keyCode == keyCodes.enter ) {
-				const event = new BubblingEventInfo( doc, 'enter', doc.selection.getFirstRange() );
-
-				doc.fire( event, new DomEventData( doc, data.domEvent, {
-					isSoft: data.shiftKey
-				} ) );
-
-				// Stop `keydown` event if `enter` event was stopped.
-				// https://github.com/ckeditor/ckeditor5/issues/753
-				if ( event.stop.called ) {
-					evt.stop();
-				}
-			}
-		} );
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	observe() {}
+    /**
+     * @inheritDoc
+     */
+    constructor(view) {
+        super(view);
+        const doc = this.document;
+        doc.on('beforeinput', (evt, data) => {
+            if (!this.isEnabled) {
+                return;
+            }
+            const domEvent = data.domEvent;
+            const enterEventSpec = ENTER_EVENT_TYPES[data.inputType];
+            if (!enterEventSpec) {
+                return;
+            }
+            const event = new BubblingEventInfo(doc, 'enter', data.targetRanges[0]);
+            doc.fire(event, new DomEventData(view, domEvent, {
+                isSoft: enterEventSpec.isSoft
+            }));
+            // Stop `beforeinput` event if `enter` event was stopped.
+            // https://github.com/ckeditor/ckeditor5/issues/753
+            if (event.stop.called) {
+                evt.stop();
+            }
+        });
+    }
+    /**
+     * @inheritDoc
+     */
+    observe() { }
 }
-
-/**
- * Event fired when the user presses the <kbd>Enter</kbd> key.
- *
- * Note: This event is fired by the {@link module:enter/enterobserver~EnterObserver observer}
- * (usually registered by the {@link module:enter/enter~Enter Enter feature} and
- * {@link module:enter/shiftenter~ShiftEnter ShiftEnter feature}).
- *
- * @event module:engine/view/document~Document#event:enter
- * @param {module:engine/view/observer/domeventdata~DomEventData} data
- * @param {Boolean} data.isSoft Whether it's a soft enter (<kbd>Shift</kbd>+<kbd>Enter</kbd>) or hard enter (<kbd>Enter</kbd>).
- */

package/src/index.js

@@ -2,10 +2,8 @@
  * @license Copyright (c) 2003-2022, CKSource Holding sp. z o.o. All rights reserved.
  * For licensing, see LICENSE.md or https://ckeditor.com/legal/ckeditor-oss-license
  */
-
 /**
  * @module enter
  */
-
 export { default as Enter } from './enter';
 export { default as ShiftEnter } from './shiftenter';

package/src/shiftenter.js

@@ -2,15 +2,12 @@
  * @license Copyright (c) 2003-2022, CKSource Holding sp. z o.o. All rights reserved.
  * For licensing, see LICENSE.md or https://ckeditor.com/legal/ckeditor-oss-license
  */
-
 /**
  * @module enter/shiftenter
  */
-
 import ShiftEnterCommand from './shiftentercommand';
 import EnterObserver from './enterobserver';
 import Plugin from '@ckeditor/ckeditor5-core/src/plugin';
-
 /**
  * This plugin handles the <kbd>Shift</kbd>+<kbd>Enter</kbd> keystroke (soft line break) in the editor.
  *
@@ -21,53 +18,48 @@ import Plugin from '@ckeditor/ckeditor5-core/src/plugin';
  * @extends module:core/plugin~Plugin
  */
 export default class ShiftEnter extends Plugin {
-	/**
-	 * @inheritDoc
-	 */
-	static get pluginName() {
-		return 'ShiftEnter';
-	}
-
-	init() {
-		const editor = this.editor;
-		const schema = editor.model.schema;
-		const conversion = editor.conversion;
-		const view = editor.editing.view;
-		const viewDocument = view.document;
-
-		// Configure the schema.
-		schema.register( 'softBreak', {
-			allowWhere: '$text',
-			isInline: true
-		} );
-
-		// Configure converters.
-		conversion.for( 'upcast' )
-			.elementToElement( {
-				model: 'softBreak',
-				view: 'br'
-			} );
-
-		conversion.for( 'downcast' )
-			.elementToElement( {
-				model: 'softBreak',
-				view: ( modelElement, { writer } ) => writer.createEmptyElement( 'br' )
-			} );
-
-		view.addObserver( EnterObserver );
-
-		editor.commands.add( 'shiftEnter', new ShiftEnterCommand( editor ) );
-
-		this.listenTo( viewDocument, 'enter', ( evt, data ) => {
-			data.preventDefault();
-
-			// The hard enter key is handled by the Enter plugin.
-			if ( !data.isSoft ) {
-				return;
-			}
-
-			editor.execute( 'shiftEnter' );
-			view.scrollToTheSelection();
-		}, { priority: 'low' } );
-	}
+    /**
+     * @inheritDoc
+     */
+    static get pluginName() {
+        return 'ShiftEnter';
+    }
+    init() {
+        const editor = this.editor;
+        const schema = editor.model.schema;
+        const conversion = editor.conversion;
+        const view = editor.editing.view;
+        const viewDocument = view.document;
+        // Configure the schema.
+        schema.register('softBreak', {
+            allowWhere: '$text',
+            isInline: true
+        });
+        // Configure converters.
+        conversion.for('upcast')
+            .elementToElement({
+            model: 'softBreak',
+            view: 'br'
+        });
+        conversion.for('downcast')
+            .elementToElement({
+            model: 'softBreak',
+            view: (modelElement, { writer }) => writer.createEmptyElement('br')
+        });
+        view.addObserver(EnterObserver);
+        editor.commands.add('shiftEnter', new ShiftEnterCommand(editor));
+        this.listenTo(viewDocument, 'enter', (evt, data) => {
+            // When not in composition, we handle the action, so prevent the default one.
+            // When in composition, it's the browser who modify the DOM (renderer is disabled).
+            if (!viewDocument.isComposing) {
+                data.preventDefault();
+            }
+            // The hard enter key is handled by the Enter plugin.
+            if (!data.isSoft) {
+                return;
+            }
+            editor.execute('shiftEnter');
+            view.scrollToTheSelection();
+        }, { priority: 'low' });
+    }
 }

package/src/shiftentercommand.js

@@ -2,14 +2,11 @@
  * @license Copyright (c) 2003-2022, CKSource Holding sp. z o.o. All rights reserved.
  * For licensing, see LICENSE.md or https://ckeditor.com/legal/ckeditor-oss-license
  */
-
 /**
  * @module enter/shiftentercommand
  */
-
 import Command from '@ckeditor/ckeditor5-core/src/command';
 import { getCopyOnEnterAttributes } from './utils';
-
 /**
  * ShiftEnter command. It is used by the {@link module:enter/shiftenter~ShiftEnter ShiftEnter feature} to handle
  * the <kbd>Shift</kbd>+<kbd>Enter</kbd> keystroke.
@@ -17,112 +14,97 @@ import { getCopyOnEnterAttributes } from './utils';
  * @extends module:core/command~Command
  */
 export default class ShiftEnterCommand extends Command {
-	/**
-	 * @inheritDoc
-	 */
-	execute() {
-		const model = this.editor.model;
-		const doc = model.document;
-
-		model.change( writer => {
-			softBreakAction( model, writer, doc.selection );
-			this.fire( 'afterExecute', { writer } );
-		} );
-	}
-
-	refresh() {
-		const model = this.editor.model;
-		const doc = model.document;
-
-		this.isEnabled = isEnabled( model.schema, doc.selection );
-	}
+    /**
+     * @inheritDoc
+     */
+    execute() {
+        const model = this.editor.model;
+        const doc = model.document;
+        model.change(writer => {
+            softBreakAction(model, writer, doc.selection);
+            this.fire('afterExecute', { writer });
+        });
+    }
+    refresh() {
+        const model = this.editor.model;
+        const doc = model.document;
+        this.isEnabled = isEnabled(model.schema, doc.selection);
+    }
 }
-
 // Checks whether the ShiftEnter command should be enabled in the specified selection.
 //
 // @param {module:engine/model/schema~Schema} schema
 // @param {module:engine/model/selection~Selection|module:engine/model/documentselection~DocumentSelection} selection
-function isEnabled( schema, selection ) {
-	// At this moment it is okay to support single range selections only.
-	// But in the future we may need to change that.
-	if ( selection.rangeCount > 1 ) {
-		return false;
-	}
-
-	const anchorPos = selection.anchor;
-
-	// Check whether the break element can be inserted in the current selection anchor.
-	if ( !anchorPos || !schema.checkChild( anchorPos, 'softBreak' ) ) {
-		return false;
-	}
-
-	const range = selection.getFirstRange();
-	const startElement = range.start.parent;
-	const endElement = range.end.parent;
-
-	// Do not modify the content if selection is cross-limit elements.
-	if ( ( isInsideLimitElement( startElement, schema ) || isInsideLimitElement( endElement, schema ) ) && startElement !== endElement ) {
-		return false;
-	}
-
-	return true;
+function isEnabled(schema, selection) {
+    // At this moment it is okay to support single range selections only.
+    // But in the future we may need to change that.
+    if (selection.rangeCount > 1) {
+        return false;
+    }
+    const anchorPos = selection.anchor;
+    // Check whether the break element can be inserted in the current selection anchor.
+    if (!anchorPos || !schema.checkChild(anchorPos, 'softBreak')) {
+        return false;
+    }
+    const range = selection.getFirstRange();
+    const startElement = range.start.parent;
+    const endElement = range.end.parent;
+    // Do not modify the content if selection is cross-limit elements.
+    if ((isInsideLimitElement(startElement, schema) || isInsideLimitElement(endElement, schema)) && startElement !== endElement) {
+        return false;
+    }
+    return true;
 }
-
 // Creates a break in the way that the <kbd>Shift</kbd>+<kbd>Enter</kbd> keystroke is expected to work.
 //
 // @param {module:engine/model~Model} model
 // @param {module:engine/model/writer~Writer} writer
 // @param {module:engine/model/selection~Selection|module:engine/model/documentselection~DocumentSelection} selection
 // Selection on which the action should be performed.
-function softBreakAction( model, writer, selection ) {
-	const isSelectionEmpty = selection.isCollapsed;
-	const range = selection.getFirstRange();
-	const startElement = range.start.parent;
-	const endElement = range.end.parent;
-	const isContainedWithinOneElement = ( startElement == endElement );
-
-	if ( isSelectionEmpty ) {
-		const attributesToCopy = getCopyOnEnterAttributes( model.schema, selection.getAttributes() );
-		insertBreak( model, writer, range.end );
-
-		writer.removeSelectionAttribute( selection.getAttributeKeys() );
-		writer.setSelectionAttribute( attributesToCopy );
-	} else {
-		const leaveUnmerged = !( range.start.isAtStart && range.end.isAtEnd );
-		model.deleteContent( selection, { leaveUnmerged } );
-
-		// Selection within one element:
-		//
-		// <h>x[xx]x</h>		-> <h>x^x</h>			-> <h>x<br>^x</h>
-		if ( isContainedWithinOneElement ) {
-			insertBreak( model, writer, selection.focus );
-		}
-		// Selection over multiple elements.
-		//
-		// <h>x[x</h><p>y]y<p>	-> <h>x^</h><p>y</p>	-> <h>x</h><p>^y</p>
-		//
-		// We chose not to insert a line break in this case because:
-		//
-		// * it's not a very common scenario,
-		// * it actually surprised me when I saw the "expected behavior" in real life.
-		//
-		// It's ok if the user will need to be more specific where they want the <br> to be inserted.
-		else {
-			// Move the selection to the 2nd element (last step of the example above).
-			if ( leaveUnmerged ) {
-				writer.setSelection( endElement, 0 );
-			}
-		}
-	}
+function softBreakAction(model, writer, selection) {
+    const isSelectionEmpty = selection.isCollapsed;
+    const range = selection.getFirstRange();
+    const startElement = range.start.parent;
+    const endElement = range.end.parent;
+    const isContainedWithinOneElement = (startElement == endElement);
+    if (isSelectionEmpty) {
+        const attributesToCopy = getCopyOnEnterAttributes(model.schema, selection.getAttributes());
+        insertBreak(model, writer, range.end);
+        writer.removeSelectionAttribute(selection.getAttributeKeys());
+        writer.setSelectionAttribute(attributesToCopy);
+    }
+    else {
+        const leaveUnmerged = !(range.start.isAtStart && range.end.isAtEnd);
+        model.deleteContent(selection, { leaveUnmerged });
+        // Selection within one element:
+        //
+        // <h>x[xx]x</h>		-> <h>x^x</h>			-> <h>x<br>^x</h>
+        if (isContainedWithinOneElement) {
+            insertBreak(model, writer, selection.focus);
+        }
+        // Selection over multiple elements.
+        //
+        // <h>x[x</h><p>y]y<p>	-> <h>x^</h><p>y</p>	-> <h>x</h><p>^y</p>
+        //
+        // We chose not to insert a line break in this case because:
+        //
+        // * it's not a very common scenario,
+        // * it actually surprised me when I saw the "expected behavior" in real life.
+        //
+        // It's ok if the user will need to be more specific where they want the <br> to be inserted.
+        else {
+            // Move the selection to the 2nd element (last step of the example above).
+            if (leaveUnmerged) {
+                writer.setSelection(endElement, 0);
+            }
+        }
+    }
 }
-
-function insertBreak( model, writer, position ) {
-	const breakLineElement = writer.createElement( 'softBreak' );
-
-	model.insertContent( breakLineElement, position );
-	writer.setSelection( breakLineElement, 'after' );
+function insertBreak(model, writer, position) {
+    const breakLineElement = writer.createElement('softBreak');
+    model.insertContent(breakLineElement, position);
+    writer.setSelection(breakLineElement, 'after');
 }
-
 // Checks whether the specified `element` is a child of the limit element.
 //
 // Checking whether the `<p>` element is inside a limit element:
@@ -132,11 +114,10 @@ function insertBreak( model, writer, position ) {
 // @param {module:engine/model/element~Element} element
 // @param {module:engine/schema~Schema} schema
 // @returns {Boolean}
-function isInsideLimitElement( element, schema ) {
-	// `$root` is a limit element but in this case is an invalid element.
-	if ( element.is( 'rootElement' ) ) {
-		return false;
-	}
-
-	return schema.isLimit( element ) || isInsideLimitElement( element.parent, schema );
+function isInsideLimitElement(element, schema) {
+    // `$root` is a limit element but in this case is an invalid element.
+    if (element.is('rootElement')) {
+        return false;
+    }
+    return schema.isLimit(element) || isInsideLimitElement(element.parent, schema);
 }

package/src/utils.js

@@ -2,11 +2,6 @@
  * @license Copyright (c) 2003-2022, CKSource Holding sp. z o.o. All rights reserved.
  * For licensing, see LICENSE.md or https://ckeditor.com/legal/ckeditor-oss-license
  */
-
-/**
- * @module enter/utils
- */
-
 /**
  * Returns attributes that should be preserved on the enter key.
  *
@@ -17,10 +12,10 @@
  * @param {Iterable.<*>} allAttributes attributes to filter.
  * @returns {Iterable.<*>}
  */
-export function* getCopyOnEnterAttributes( schema, allAttributes ) {
-	for ( const attribute of allAttributes ) {
-		if ( attribute && schema.getAttributeProperties( attribute[ 0 ] ).copyOnEnter ) {
-			yield attribute;
-		}
-	}
+export function* getCopyOnEnterAttributes(schema, allAttributes) {
+    for (const attribute of allAttributes) {
+        if (attribute && schema.getAttributeProperties(attribute[0]).copyOnEnter) {
+            yield attribute;
+        }
+    }
 }