Browse Source

Merge pull request #70 from ckeditor/t/69

Other: Use model.insertContent instead of model.Writer.insertText. Closes #69.
Piotr Jasiun 6 years ago
parent
commit
b7036639cd
1 changed files with 2 additions and 5 deletions
  1. 2 5
      packages/ckeditor5-mention/src/mentioncommand.js

+ 2 - 5
packages/ckeditor5-mention/src/mentioncommand.js

@@ -137,11 +137,8 @@ export default class MentionCommand extends Command {
 			attributesWithMention.set( 'mention', mention );
 
 			// Replace a range with the text with a mention.
-			writer.remove( range );
-			writer.insertText( mentionText, attributesWithMention, range.start );
-
-			// Insert a space after the mention.
-			writer.insertText( ' ', currentAttributes, model.document.selection.focus );
+			model.insertContent( writer.createText( mentionText, attributesWithMention ), range );
+			model.insertContent( writer.createText( ' ', currentAttributes ), range.start.getShiftedBy( mentionText.length ) );
 		} );
 	}
 }