Merge pull request #313443 from mjbvz/dev/mjbvz/inquisitive-tuna

Remove some now redundant webview layout logic

Author: mjbvz

src/vs/workbench/contrib/notebook/browser/notebookEditorWidget.ts

@@ -416,13 +416,6 @@ export class NotebookEditorWidget extends Disposable implements INotebookEditorD
 		}));
 
 		const container = creationOptions.codeWindow ? this.layoutService.getContainer(creationOptions.codeWindow) : this.layoutService.mainContainer;
-		this._register(editorGroupsService.getPart(container).onDidScroll(e => {
-			if (!this._shadowElement || !this._isVisible) {
-				return;
-			}
-
-			this.layoutContainerOverShadowElement(this._shadowElement);
-		}));
 
 		this.notebookEditorService.addNotebookEditor(this);
 

src/vs/workbench/contrib/webviewPanel/browser/webviewEditor.ts

@@ -40,7 +40,6 @@ export class WebviewEditor extends EditorPane {
 	public static readonly ID = 'WebviewEditor';
 
 	private _element?: HTMLElement;
-	private _dimension?: DOM.Dimension;
 	private _visible = false;
 	private _isDisposed = false;
 	private _clippingContainer?: HTMLElement;
@@ -65,15 +64,6 @@ export class WebviewEditor extends EditorPane {
 		@IContextKeyService private readonly _contextKeyService: IContextKeyService,
 	) {
 		super(WebviewEditor.ID, group, telemetryService, themeService, storageService);
-
-		const part = _editorGroupsService.getPart(group);
-		this._register(Event.any(part.onDidScroll, part.onDidAddGroup, part.onDidRemoveGroup, part.onDidMoveGroup)(() => {
-			if (this.webview && this._visible) {
-				this.setWebviewAnchorElement(this.webview);
-			}
-		}));
-
-
 	}
 
 	private get webview(): IOverlayWebview | undefined {
@@ -103,11 +93,6 @@ export class WebviewEditor extends EditorPane {
 	}
 
 	public override layout(dimension: DOM.Dimension): void {
-		this._dimension = dimension;
-		if (this.webview && this._visible) {
-			this.setWebviewAnchorElement(this.webview);
-		}
-
 		this.setEditorVisible(dimension.width > 0 && dimension.height > 0);
 	}
 
@@ -172,9 +157,6 @@ export class WebviewEditor extends EditorPane {
 			if (!alreadyOwnsWebview) {
 				this.claimWebview(input);
 			}
-			if (this._dimension) {
-				this.layout(this._dimension);
-			}
 		}
 	}
 

src/vs/workbench/contrib/webviewView/browser/webviewViewPane.ts

@@ -137,11 +137,6 @@ export class WebviewViewPane extends ViewPane {
 		super.saveState();
 	}
 
-	protected override layoutBody(height: number, width: number): void {
-		super.layoutBody(height, width);
-		this.layoutWebview();
-	}
-
 	private updateTreeVisibility() {
 		if (this.isBodyVisible()) {
 			this.activate();