setting the margins inside of the case of initial render

aiday-mar · aiday-mar · commit 79c611d7b7a5 · 2023-06-14T09:33:54.000+02:00
diff --git a/src/vs/workbench/contrib/inlineChat/browser/inlineChatController.ts b/src/vs/workbench/contrib/inlineChat/browser/inlineChatController.ts
@@ -202,13 +202,11 @@ export class InlineChatController implements IEditorContribution {
 		let widgetPosition: Position | undefined;
 		if (initialRender) {
 			widgetPosition = this._editor.getPosition();
+			this._zone.value.setMargins(widgetPosition);
 		} else {
 			widgetPosition = this._strategy.getWidgetPosition() ?? this._zone.value.position ?? this._activeSession.wholeRange.value.getEndPosition();
 		}
 		this._zone.value.show(widgetPosition);
-		if (initialRender) {
-			this._zone.value.setMargins(widgetPosition);
-		}
 	}
 
 	protected async _nextState(state: State, options: InlineChatRunOptions | undefined): Promise<void> {

Original file line number	Diff line number	Diff line change
`@@ -202,13 +202,11 @@ export class InlineChatController implements IEditorContribution {`
`202`	`202`	`let widgetPosition: Position \| undefined;`
`203`	`203`	`if (initialRender) {`
`204`	`204`	`widgetPosition = this._editor.getPosition();`
	`205`	`+ this._zone.value.setMargins(widgetPosition);`
`205`	`206`	`} else {`
`206`	`207`	`widgetPosition = this._strategy.getWidgetPosition() ?? this._zone.value.position ?? this._activeSession.wholeRange.value.getEndPosition();`
`207`	`208`	`}`
`208`	`209`	`this._zone.value.show(widgetPosition);`
`209`		`- if (initialRender) {`
`210`		`- this._zone.value.setMargins(widgetPosition);`
`211`		`- }`
`212`	`210`	`}`
`213`	`211`
`214`	`212`	`protected async _nextState(state: State, options: InlineChatRunOptions \| undefined): Promise<void> {`