Remove unused function

Author: timtmok

src/vs/workbench/contrib/positronPlots/browser/components/dynamicPlotInstance.tsx

@@ -25,7 +25,7 @@ import { ZoomLevel } from '../../../../services/positronPlots/common/positronPlo
 interface DynamicPlotInstanceProps {
 	width: number;
 	height: number;
-	zoom?: ZoomLevel;
+	zoom: ZoomLevel;
 	plotClient: PlotClientInstance;
 }
 

src/vs/workbench/contrib/positronPlots/browser/components/panZoomImage.tsx

@@ -15,7 +15,7 @@ interface PanZoomImageProps {
 	height: number;
 	imageUri: string;
 	description: string;
-	zoom?: ZoomLevel;
+	zoom: ZoomLevel;
 }
 
 /**
@@ -37,13 +37,12 @@ export const PanZoomImage = (props: PanZoomImageProps) => {
 		if (!imageRef.current) {
 			return;
 		}
-		const zoomLevel = props.zoom ?? ZoomLevel.Fit;
 		// scale by the zoom level
 		// if the zoom level is Fill, then the image should fill the container using css
-		const adjustedWidth = props.zoom === ZoomLevel.Fit ? naturalWidth : naturalWidth * zoomLevel;
-		const adjustedHeight = props.zoom === ZoomLevel.Fit ? naturalHeight : naturalHeight * zoomLevel;
+		const adjustedWidth = props.zoom === ZoomLevel.Fit ? naturalWidth : naturalWidth * props.zoom;
+		const adjustedHeight = props.zoom === ZoomLevel.Fit ? naturalHeight : naturalHeight * props.zoom;
 
-		if (zoomLevel === ZoomLevel.Fit) {
+		if (props.zoom === ZoomLevel.Fit) {
 			imageRef.current.style.width = '100%';
 			imageRef.current.style.height = '100%';
 			imageRef.current.style.objectFit = 'contain';

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

@@ -564,17 +564,6 @@ export class PositronPlotsService extends Disposable implements IPositronPlotsSe
 		}
 	}
 
-	getEditorPlotZoom(plotId: string): ZoomLevel | undefined {
-		const plot = this._editorPlots.get(plotId);
-		if (plot instanceof PlotClientInstance) {
-			// Return the zoom level for the plot, or the default zoom level if not set
-			return plot.metadata.zoom_level;
-		} else {
-			this._notificationService.error(localize('positronPlots.zoom.getInvalidPlotType', 'Cannot get zoom for this plot type'));
-			return undefined;
-		}
-	}
-
 	/**
 	 * Sets a custom plot size and applies it as a custom sizing policy.
 	 *

src/vs/workbench/contrib/positronPlotsEditor/browser/editorPlotsContainer.tsx

@@ -26,7 +26,7 @@ interface EditorPlotsContainerProps {
 
 export const EditorPlotsContainer = (props: EditorPlotsContainerProps) => {
 
-	const [zoom, setZoom] = useState<ZoomLevel>(props.plotClient instanceof PlotClientInstance ? props.plotClient.zoomLevel : ZoomLevel.Fit);
+	const [zoom, setZoom] = useState<ZoomLevel>(ZoomLevel.Fit);
 	const [darkFilterMode, setDarkFilterMode] = useState(props.positronPlotsService.darkFilterMode);
 
 	const render = () => {
@@ -79,7 +79,7 @@ export const EditorPlotsContainer = (props: EditorPlotsContainerProps) => {
 	}, [props.plotClient]);
 
 	return (
-		<div className={`dark-filter-${darkFilterMode}`} style={
+		<div className={'dark-filter-' + darkFilterMode} style={
 			{
 				width: props.width,
 				height: props.height

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

@@ -5,7 +5,7 @@
 
 import { TestPositronPlotsService } from './testPositronPlotsService.js';
 import { TestPositronPlotClient } from './testPositronPlotClient.js';
-import { HistoryPolicy } from '../../common/positronPlots.js';
+import { HistoryPolicy, ZoomLevel } from '../../common/positronPlots.js';
 
 /**
  * Example of how to use the TestPositronPlotsService in tests.
@@ -21,7 +21,7 @@ export function createTestPlotsServiceWithPlots(): TestPositronPlotsService {
 		created: Date.now(),
 		parent_id: '',
 		code: 'plot(1:10)',
-		zoom_level: 0,
+		zoom_level: ZoomLevel.Fit,
 	});
 
 	const plotClient2 = new TestPositronPlotClient({
@@ -30,7 +30,7 @@ export function createTestPlotsServiceWithPlots(): TestPositronPlotsService {
 		created: Date.now() + 1000, // Created later
 		parent_id: '',
 		code: 'hist(rnorm(100))',
-		zoom_level: 0,
+		zoom_level: ZoomLevel.Fit,
 	});
 
 	// Add the plot clients to the service

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

@@ -5,9 +5,9 @@
 
 import { Emitter } from '../../../../../base/common/event.js';
 import { Disposable, DisposableMap } from '../../../../../base/common/lifecycle.js';
-import { IPositronPlotsService, IPositronPlotClient, HistoryPolicy, DarkFilter, PlotRenderSettings, ZoomLevel } from '../../common/positronPlots.js';
+import { IPositronPlotsService, IPositronPlotClient, HistoryPolicy, DarkFilter, PlotRenderSettings } from '../../common/positronPlots.js';
 import { IPositronPlotSizingPolicy } from '../../common/sizingPolicy.js';
-import { IPositronPlotMetadata, PlotClientLocation } from '../../../languageRuntime/common/languageRuntimePlotClient.js';
+import { IPositronPlotMetadata } from '../../../languageRuntime/common/languageRuntimePlotClient.js';
 
 /**
  * TestPositronPlotsService class.
@@ -28,9 +28,6 @@ export class TestPositronPlotsService extends Disposable implements IPositronPlo
 	 */
 	private readonly _editorPlots = new Map<string, IPositronPlotClient>();
 
-	/** Map of zoom levels for editor plots, keyed by plot ID. */
-	private readonly _editorPlotZoomLevels = new Map<string, number>();
-
 	/**
 	 * Gets or sets the ID of the currently selected plot.
 	 */
@@ -102,10 +99,6 @@ export class TestPositronPlotsService extends Disposable implements IPositronPlo
 	private readonly _onDidChangeSizingPolicyEmitter =
 		this._register(new Emitter<IPositronPlotSizingPolicy>());
 
-	/** The emitter for the _plotZoomEmitter event */
-	private readonly _onDidChangePlotZoomEmitter =
-		this._register(new Emitter<{ plotId: string; zoomLevel: number; location: PlotClientLocation }>());
-
 	//#endregion Private Properties
 
 	//#region Constructor
@@ -215,11 +208,6 @@ export class TestPositronPlotsService extends Disposable implements IPositronPlo
 	 */
 	readonly onDidChangeSizingPolicy = this._onDidChangeSizingPolicyEmitter.event;
 
-	/**
-	 * The onDidChangePlotZoom event.
-	 */
-	readonly onDidChangePlotZoom = this._onDidChangePlotZoomEmitter.event;
-
 	/**
 	 * Gets the cached plot thumbnail URI for a given plot ID.
 	 * @param plotId The plot ID to get the thumbnail URI for.
@@ -474,33 +462,6 @@ export class TestPositronPlotsService extends Disposable implements IPositronPlo
 		plotClient.dispose();
 	}
 
-	/**
-	 * Sets the zoom level for the editor plot with the given ID.
-	 * @param plotId The ID of the plot to set the zoom level for.
-	 * @param zoomLevel The zoom level to set for the plot.
-	 */
-	setEditorPlotZoom(plotId: string, zoomLevel: number): void {
-		if (this._editorPlots.has(plotId)) {
-			this._editorPlotZoomLevels.set(plotId, zoomLevel);
-		} else {
-			throw new Error(`Plot with ID ${plotId} does not exist.`);
-		}
-	}
-
-	/**
-	 * Gets the zoom level for the editor plot with the given ID.
-	 * @param plotId The ID of the plot to get the zoom level for.
-	 * @returns The zoom level for the plot, or undefined if not found.
-	 */
-	getEditorPlotZoomLevel(plotId: string): number | undefined {
-		const plotClient = this._editorPlots.get(plotId);
-		if (plotClient) {
-			return this._editorPlotZoomLevels.get(plotId) ?? ZoomLevel.Fit;
-		}
-
-		return ZoomLevel.Fit; // Default to Fit for test implementation
-	}
-
 	/**
 	 * Placeholder for service initialization.
 	 */