From f91e2bb27e894dfe1cbe6a7170cf19ba48d1772b Mon Sep 17 00:00:00 2001 From: isidor Date: Wed, 23 Nov 2016 12:52:13 +0100 Subject: [PATCH] debug: last new line should be ignored since the repl elements are by design split by rows --- src/vs/workbench/parts/debug/electron-browser/replViewer.ts | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/vs/workbench/parts/debug/electron-browser/replViewer.ts b/src/vs/workbench/parts/debug/electron-browser/replViewer.ts index 98279d0a404..b8bb48d9dc2 100644 --- a/src/vs/workbench/parts/debug/electron-browser/replViewer.ts +++ b/src/vs/workbench/parts/debug/electron-browser/replViewer.ts @@ -6,7 +6,7 @@ import * as nls from 'vs/nls'; import { TPromise } from 'vs/base/common/winjs.base'; import { IAction } from 'vs/base/common/actions'; -import { isFullWidthCharacter, removeAnsiEscapeCodes } from 'vs/base/common/strings'; +import { isFullWidthCharacter, removeAnsiEscapeCodes, endsWith } from 'vs/base/common/strings'; import uri from 'vs/base/common/uri'; import { isMacintosh } from 'vs/base/common/platform'; import { IKeyboardEvent } from 'vs/base/browser/keyboardEvent'; @@ -118,6 +118,10 @@ export class ReplExpressionsRenderer implements IRenderer { return ReplExpressionsRenderer.LINE_HEIGHT_PX; } + // Last new line should be ignored since the repl elements are by design split by rows + if (endsWith(s, '\n')) { + s = s.substr(0, s.length - 1); + } const lines = removeAnsiEscapeCodes(s).split('\n'); const numLines = lines.reduce((lineCount: number, line: string) => { let lineLength = 0; -- GitLab