Added small improvements to model/writer.js.

ckeditor · Jan 26, 2018 · 4e80fa9 · 4e80fa9
1 parent 92fda51
commit 4e80fa9
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/model/writer.js b/src/model/writer.js
@@ -872,8 +872,8 @@ export default class Writer {
 
 	/**
 	 * @private
-	 * @param {String} key
-	 * @param {*} value
+	 * @param {String} key Key of the attribute to remove.
+	 * @param {*} value Attribute value.
 	 */
 	_setSelectionAttribute( key, value ) {
 		const selection = this.model.document.selection;
@@ -891,7 +891,7 @@ export default class Writer {
 	/**
 	 * Removes an attribute with given key from the selection.
 	 *
-	 * @param {String|Iterable.<String>} keyOrIterableOfKeys Key of the attribute to remove.
+	 * @param {String|Iterable.<String>} keyOrIterableOfKeys Key of the attribute to remove or an iterable of attribute keys to remove.
 	 */
 	removeSelectionAttribute( keyOrIterableOfKeys ) {
 		this._assertWriterUsedCorrectly();

diff --git a/src/view/writer.js b/src/view/writer.js
@@ -496,7 +496,7 @@ export function wrap( range, attribute, viewSelection = null ) {
 
 		// If wrapping position is equal to view selection, move view selection inside wrapping attribute element.
 		if ( viewSelection && viewSelection.isCollapsed && viewSelection.getFirstPosition().isEqual( range.start ) ) {
-			viewSelection.setTo( new Range( position ) );
+			viewSelection.setTo( position );
 		}
 
 		return new Range( position );