Fix auto-surround closes with an open character (microsoft#147187)

yamachig · alexdima · web-flow · commit ce6aa98b4a4a · 2022-10-24T11:24:16.000-07:00
Co-authored-by: Alexandru Dima &lt;alexdima@microsoft.com&gt;
diff --git a/src/vs/editor/common/cursor/cursorTypeOperations.ts b/src/vs/editor/common/cursor/cursorTypeOperations.ts
@@ -891,7 +891,7 @@ export class TypeOperations {
 
 			const commands: ICommand[] = [];
 			for (let i = 0, len = positions.length; i < len; i++) {
-				commands.push(new CompositionSurroundSelectionCommand(positions[i], compositions[i].deletedText, ch));
+				commands.push(new CompositionSurroundSelectionCommand(positions[i], compositions[i].deletedText, config.surroundingPairs[ch]));
 			}
 			return new EditOperationResult(EditOperationType.TypingOther, commands, {
 				shouldPushStackElementBefore: true,

Original file line number	Diff line number	Diff line change
`@@ -891,7 +891,7 @@ export class TypeOperations {`
`891`	`891`
`892`	`892`	`const commands: ICommand[] = [];`
`893`	`893`	`for (let i = 0, len = positions.length; i < len; i++) {`
`894`		`- commands.push(new CompositionSurroundSelectionCommand(positions[i], compositions[i].deletedText, ch));`
	`894`	`+ commands.push(new CompositionSurroundSelectionCommand(positions[i], compositions[i].deletedText, config.surroundingPairs[ch]));`
`895`	`895`	`}`
`896`	`896`	`return new EditOperationResult(EditOperationType.TypingOther, commands, {`
`897`	`897`	`shouldPushStackElementBefore: true,`