From 56fcba98e9f42e67711ab26a2da8ce2cd300b658 Mon Sep 17 00:00:00 2001
From: Alex Dima <alexdima@microsoft.com>
Date: Fri, 6 Mar 2020 13:32:46 +0100
Subject: [PATCH] Fixes #87614: Ignore composition that doesn't do anything

---
 .../editor/common/controller/cursorTypeOperations.ts | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/src/vs/editor/common/controller/cursorTypeOperations.ts b/src/vs/editor/common/controller/cursorTypeOperations.ts
index 9ae8b986bce..0155bf76b6e 100644
--- a/src/vs/editor/common/controller/cursorTypeOperations.ts
+++ b/src/vs/editor/common/controller/cursorTypeOperations.ts
@@ -269,9 +269,15 @@ export class TypeOperations {
 				commands[i] = null;
 				continue;
 			}
-			let pos = selection.getPosition();
-			let startColumn = Math.max(1, pos.column - replaceCharCnt);
-			let range = new Range(pos.lineNumber, startColumn, pos.lineNumber, pos.column);
+			const pos = selection.getPosition();
+			const startColumn = Math.max(1, pos.column - replaceCharCnt);
+			const range = new Range(pos.lineNumber, startColumn, pos.lineNumber, pos.column);
+			const oldText = model.getValueInRange(range);
+			if (oldText === txt) {
+				// => ignore composition that doesn't do anything
+				commands[i] = null;
+				continue;
+			}
 			commands[i] = new ReplaceCommand(range, txt);
 		}
 		return new EditOperationResult(EditOperationType.Typing, commands, {
-- 
GitLab