From 0fd2b0ce5d941c8c24713076ec07c6f794a1d860 Mon Sep 17 00:00:00 2001
From: Aiday Marlen Kyzy <amarlenkyzy@microsoft.com>
Date: Fri, 16 Feb 2024 11:34:48 +0100
Subject: [PATCH] inlining the resource and the range

---
 .../widget/multiDiffEditorWidget/multiDiffEditorWidget.ts    | 4 ++--
 .../multiDiffEditorWidget/multiDiffEditorWidgetImpl.ts       | 5 ++---
 .../contrib/multiDiffEditor/browser/multiDiffEditor.ts       | 2 +-
 3 files changed, 5 insertions(+), 6 deletions(-)

diff --git a/src/vs/editor/browser/widget/multiDiffEditorWidget/multiDiffEditorWidget.ts b/src/vs/editor/browser/widget/multiDiffEditorWidget/multiDiffEditorWidget.ts
index 60adf436d1502..5a869f83bd2f7 100644
--- a/src/vs/editor/browser/widget/multiDiffEditorWidget/multiDiffEditorWidget.ts
+++ b/src/vs/editor/browser/widget/multiDiffEditorWidget/multiDiffEditorWidget.ts
@@ -46,8 +46,8 @@ export class MultiDiffEditorWidget extends Disposable {
 		this._register(recomputeInitiallyAndOnChange(this._widgetImpl));
 	}
 
-	public reveal(revealData: { resource: IMultiDiffResource; range: Range }): void {
-		this._widgetImpl.get().reveal(revealData);
+	public reveal(resource: IMultiDiffResource, range: Range): void {
+		this._widgetImpl.get().reveal(resource, range);
 	}
 
 	public createViewModel(model: IMultiDiffEditorModel): MultiDiffEditorViewModel {
diff --git a/src/vs/editor/browser/widget/multiDiffEditorWidget/multiDiffEditorWidgetImpl.ts b/src/vs/editor/browser/widget/multiDiffEditorWidget/multiDiffEditorWidgetImpl.ts
index e423974d4fb70..ca4e240c1dace 100644
--- a/src/vs/editor/browser/widget/multiDiffEditorWidget/multiDiffEditorWidgetImpl.ts
+++ b/src/vs/editor/browser/widget/multiDiffEditorWidget/multiDiffEditorWidgetImpl.ts
@@ -191,8 +191,7 @@ export class MultiDiffEditorWidgetImpl extends Disposable {
 	}
 
 	// todo@aiday-mar need to reveal the range instead of just the start line number
-	public reveal(revealData: { resource: IMultiDiffResource; range: Range }): void {
-		const resource = revealData.resource;
+	public reveal(resource: IMultiDiffResource, range: Range): void {
 		const viewItems = this._viewItems.get();
 		let searchCallback: (item: VirtualizedViewItem) => boolean;
 		if ('original' in resource) {
@@ -201,7 +200,7 @@ export class MultiDiffEditorWidgetImpl extends Disposable {
 			searchCallback = (item) => item.viewModel.modifiedUri?.toString() === resource.modified.toString();
 		}
 		const index = viewItems.findIndex(searchCallback);
-		let scrollTop = (revealData.range.startLineNumber - 1) * this._configurationService.getValue<number>('editor.lineHeight');
+		let scrollTop = (range.startLineNumber - 1) * this._configurationService.getValue<number>('editor.lineHeight');
 		for (let i = 0; i < index; i++) {
 			scrollTop += viewItems[i].contentHeight.get() + this._spaceBetweenPx;
 		}
diff --git a/src/vs/workbench/contrib/multiDiffEditor/browser/multiDiffEditor.ts b/src/vs/workbench/contrib/multiDiffEditor/browser/multiDiffEditor.ts
index ed582ca29b635..bb8bdc7c16e53 100644
--- a/src/vs/workbench/contrib/multiDiffEditor/browser/multiDiffEditor.ts
+++ b/src/vs/workbench/contrib/multiDiffEditor/browser/multiDiffEditor.ts
@@ -92,7 +92,7 @@ export class MultiDiffEditor extends AbstractEditorWithViewState<IMultiDiffEdito
 		if (!viewState || !viewState.revealData) {
 			return;
 		}
-		this._multiDiffEditorWidget?.reveal(viewState.revealData);
+		this._multiDiffEditorWidget?.reveal(viewState.revealData.resource, viewState.revealData.range);
 	}
 
 	override async clearInput(): Promise<void> {