Add onDidChangesizingPolicy event on plot service

Author: lionel-

src/vs/workbench/contrib/positronPlots/browser/positronPlotsService.ts

@@ -118,6 +118,9 @@ export class PositronPlotsService extends Disposable implements IPositronPlotsSe
 	/** The emitter for the onDidChangePlotsRenderSettings event */
 	private readonly _onDidChangePlotsRenderSettings = new Emitter<PlotRenderSettings>();
 
+	/** The emitter for the _sizingPolicyEmitter event */
+	private readonly _onDidChangeSizingPolicyEmitter = new Emitter<IPositronPlotSizingPolicy>;
+
 	/** The ID Of the currently selected plot, if any */
 	private _selectedPlotId: string | undefined;
 
@@ -205,7 +208,7 @@ export class PositronPlotsService extends Disposable implements IPositronPlotsSe
 			this._selectedPlotId = id;
 			const selectedPlot = this._plots.find((plot) => plot.id === id);
 			if (selectedPlot instanceof PlotClientInstance) {
-				this._selectedSizingPolicy = selectedPlot.sizingPolicy;
+				this.setSelectedSizingPolicy(selectedPlot.sizingPolicy);
 			}
 		}));
 
@@ -440,7 +443,8 @@ export class PositronPlotsService extends Disposable implements IPositronPlotsSe
 		if (!policy) {
 			throw new Error(`Invalid sizing policy ID: ${id}`);
 		}
-		this._selectedSizingPolicy = policy;
+
+		this.setSelectedSizingPolicy(policy);
 		const selectedPlot = this._plots.find((plot) => this.selectedPlotId === plot.id);
 		if (selectedPlot instanceof PlotClientInstance) {
 			selectedPlot.sizingPolicy = policy;
@@ -863,6 +867,7 @@ export class PositronPlotsService extends Disposable implements IPositronPlotsSe
 	onDidChangeHistoryPolicy: Event<HistoryPolicy> = this._onDidChangeHistoryPolicy.event;
 	onDidChangeDarkFilterMode: Event<DarkFilter> = this._onDidChangeDarkFilterMode.event;
 	onDidChangePlotsRenderSettings: Event<PlotRenderSettings> = this._onDidChangePlotsRenderSettings.event;
+	onDidChangeSizingPolicy: Event<IPositronPlotSizingPolicy> = this._onDidChangeSizingPolicyEmitter.event;
 
 	// Gets the individual plot instances.
 	get positronPlotInstances(): IPositronPlotClient[] {
@@ -1353,6 +1358,11 @@ export class PositronPlotsService extends Disposable implements IPositronPlotsSe
 	 */
 	initialize() {
 	}
+
+	private setSelectedSizingPolicy(policy: IPositronPlotSizingPolicy) {
+		this._selectedSizingPolicy = policy;
+		this._onDidChangeSizingPolicyEmitter.fire(policy);
+	}
 }
 
 function isActiveWebviewPlot(plot: IPositronPlotClient): plot is WebviewPlotClient {

src/vs/workbench/services/positronPlots/common/positronPlots.ts

@@ -145,6 +145,12 @@ export interface IPositronPlotsService {
 	 */
 	readonly onDidChangePlotsRenderSettings: Event<PlotRenderSettings>;
 
+	/**
+	 * Notifies subscribers when the settings for rendering a plot have changed.
+	 * This typically happens when the plot viewpane has been resized.
+	 */
+	readonly onDidChangeSizingPolicy: Event<IPositronPlotSizingPolicy>;
+
 	/**
 	 * Selects the plot with the specified ID.
 	 *

src/vs/workbench/services/positronPlots/test/common/testPositronPlotsService.ts

@@ -95,6 +95,10 @@ export class TestPositronPlotsService extends Disposable implements IPositronPlo
 	private readonly _onDidChangePlotsRenderSettingsEmitter =
 		this._register(new Emitter<PlotRenderSettings>());
 
+	/** The emitter for the _sizingPolicyEmitter event */
+	private readonly _onDidChangeSizingPolicyEmitter =
+		this._register(new Emitter<IPositronPlotSizingPolicy>());
+
 	//#endregion Private Properties
 
 	//#region Constructor
@@ -106,12 +110,12 @@ export class TestPositronPlotsService extends Disposable implements IPositronPlo
 		super();
 	}
 
-    getPlotsRenderSettings(): PlotRenderSettings {
-        throw new Error('Method not implemented.');
-    }
-    setPlotsRenderSettings(settings: PlotRenderSettings): void {
-        throw new Error('Method not implemented.');
-    }
+	getPlotsRenderSettings(): PlotRenderSettings {
+		throw new Error('Method not implemented.');
+	}
+	setPlotsRenderSettings(settings: PlotRenderSettings): void {
+		throw new Error('Method not implemented.');
+	}
 
 	//#endregion Constructor
 
@@ -199,6 +203,12 @@ export class TestPositronPlotsService extends Disposable implements IPositronPlo
 	 */
 	readonly onDidChangePlotsRenderSettings = this._onDidChangePlotsRenderSettingsEmitter.event;
 
+	/**
+	 * The onDidChangeSizingPolicy event.
+	 */
+	readonly onDidChangeSizingPolicy = this._onDidChangeSizingPolicyEmitter.event;
+
+
 	/**
 	 * Selects the plot with the specified ID.
 	 *