removing unused ChartClickEvent type until data selection eventing is…

… implemented
microsoft · Sep 13, 2023 · 91b0cb0 · 91b0cb0
1 parent 95f2605
commit 91b0cb0
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/src/sql/azdata.proposed.d.ts b/src/sql/azdata.proposed.d.ts
@@ -2082,11 +2082,9 @@ declare module 'azdata' {
 	}
 
 	export interface ChartComponent<TChartType extends ChartType, TData extends ChartData<TChartType>, TOptions extends ChartOptions<TChartType>> extends Component, ChartComponentProperties<TChartType, TData, TOptions> {
-		onDidClick: vscode.Event<ChartClickEvent>;
+		onDidClick: vscode.Event<any>;
 	}
 
-	export type ChartClickEvent = { label: string };
-
 	export interface ChartComponentProperties<TChartType extends ChartType, TData extends ChartData<TChartType>, TOptions extends ChartOptions<TChartType>> extends ComponentProperties {
 		/**
 		 * Type of chart to build.  Must match the ChartType parameter used to construct the chart.

diff --git a/src/sql/workbench/api/common/extHostModelView.ts b/src/sql/workbench/api/common/extHostModelView.ts
@@ -2285,7 +2285,7 @@ class ChartComponentWrapper<TChartType extends azdata.ChartType, TData extends a
 		super(proxy, handle, ModelComponentTypes.Chart, id, logService);
 		this.properties = {};
 
-		this._emitterMap.set(ComponentEventType.onDidClick, new Emitter<azdata.ChartClickEvent>());
+		this._emitterMap.set(ComponentEventType.onDidClick, new Emitter<any>());
 	}
 
 	public set chartType(v: TChartType) {
@@ -2304,7 +2304,7 @@ class ChartComponentWrapper<TChartType extends azdata.ChartType, TData extends a
 		return this.properties['data'];
 	}
 
-	public get onDidClick(): vscode.Event<azdata.ChartClickEvent> {
+	public get onDidClick(): vscode.Event<any> {
 		let emitter = this._emitterMap.get(ComponentEventType.onDidClick);
 		return emitter && emitter.event;
 	}