From d73ccaa856ec5774f2ef05e4fa9d4cc710da1906 Mon Sep 17 00:00:00 2001 From: Alex Ross Date: Thu, 1 Sep 2022 16:55:48 +0200 Subject: [PATCH] Adding a multi-line comment is jarring (#159766) Fixes #159765 --- .../contrib/comments/browser/commentThreadZoneWidget.ts | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/vs/workbench/contrib/comments/browser/commentThreadZoneWidget.ts b/src/vs/workbench/contrib/comments/browser/commentThreadZoneWidget.ts index e47f427b4f6..d453166e071 100644 --- a/src/vs/workbench/contrib/comments/browser/commentThreadZoneWidget.ts +++ b/src/vs/workbench/contrib/comments/browser/commentThreadZoneWidget.ts @@ -173,7 +173,7 @@ export class ReviewZoneWidget extends ZoneWidget implements ICommentThreadWidget public reveal(commentUniqueId?: number, focus: boolean = false) { if (!this._isExpanded) { - this.show({ lineNumber: this._commentThread.range.startLineNumber, column: 1 }, 2); + this.show({ lineNumber: this._commentThread.range.endLineNumber, column: 1 }, 2); } if (commentUniqueId !== undefined) { @@ -258,7 +258,7 @@ export class ReviewZoneWidget extends ZoneWidget implements ICommentThreadWidget public expand(): Promise { this._commentThread.collapsibleState = languages.CommentThreadCollapsibleState.Expanded; - const lineNumber = this._commentThread.range.startLineNumber; + const lineNumber = this._commentThread.range.endLineNumber; this.show({ lineNumber, column: 1 }, 2); return Promise.resolve();