fixes microsoft#36340

joaomoreno · joaomoreno · commit 828dd97c2dc6 · 2017-10-17T14:39:54.000+02:00
diff --git a/src/vs/editor/contrib/zoneWidget/browser/zoneWidget.ts b/src/vs/editor/contrib/zoneWidget/browser/zoneWidget.ts
@@ -395,7 +395,11 @@ export abstract class ZoneWidget implements IHorizontalSashLayoutProvider {
 
 		// Reveal the line above or below the zone widget, to get the zone widget in the viewport
 		const revealLineNumber = Math.min(this.editor.getModel().getLineCount(), Math.max(1, where.endLineNumber + 1));
-		this.editor.revealLineInCenterIfOutsideViewport(revealLineNumber, ScrollType.Smooth);
+		this.revealLine(revealLineNumber);
+	}
+
+	protected revealLine(lineNumber: number) {
+		this.editor.revealLine(lineNumber, ScrollType.Smooth);
 	}
 
 	protected setCssClass(className: string, classToReplace?: string): void {
diff --git a/src/vs/workbench/parts/scm/electron-browser/dirtydiffDecorator.ts b/src/vs/workbench/parts/scm/electron-browser/dirtydiffDecorator.ts
@@ -315,6 +315,10 @@ class DirtyDiffWidget extends PeekViewWidget {
 			secondaryHeadingColor: theme.getColor(peekViewTitleInfoForeground)
 		});
 	}
+
+	protected revealLine(lineNumber: number) {
+		this.editor.revealLineInCenterIfOutsideViewport(lineNumber, ScrollType.Smooth);
+	}
 }
 
 @editorAction

Original file line number	Diff line number	Diff line change
`@@ -315,6 +315,10 @@ class DirtyDiffWidget extends PeekViewWidget {`
`315`	`315`	`secondaryHeadingColor: theme.getColor(peekViewTitleInfoForeground)`
`316`	`316`	`});`
`317`	`317`	`}`
	`318`	`+`
	`319`	`+ protected revealLine(lineNumber: number) {`
	`320`	`+ this.editor.revealLineInCenterIfOutsideViewport(lineNumber, ScrollType.Smooth);`
	`321`	`+ }`
`318`	`322`	`}`
`319`	`323`
`320`	`324`	`@editorAction`