From 91b0cb0b873a63ee0632ef8fa9caa311e531f365 Mon Sep 17 00:00:00 2001 From: Benjin Dubishar Date: Wed, 13 Sep 2023 11:18:52 -0700 Subject: [PATCH] removing unused ChartClickEvent type until data selection eventing is implemented --- src/sql/azdata.proposed.d.ts | 4 +--- src/sql/workbench/api/common/extHostModelView.ts | 4 ++-- 2 files changed, 3 insertions(+), 5 deletions(-) diff --git a/src/sql/azdata.proposed.d.ts b/src/sql/azdata.proposed.d.ts index 2282e07c1155..9026b615839e 100644 --- a/src/sql/azdata.proposed.d.ts +++ b/src/sql/azdata.proposed.d.ts @@ -2082,11 +2082,9 @@ declare module 'azdata' { } export interface ChartComponent, TOptions extends ChartOptions> extends Component, ChartComponentProperties { - onDidClick: vscode.Event; + onDidClick: vscode.Event; } - export type ChartClickEvent = { label: string }; - export interface ChartComponentProperties, TOptions extends ChartOptions> 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 index ac9bf3759321..df970257655b 100644 --- a/src/sql/workbench/api/common/extHostModelView.ts +++ b/src/sql/workbench/api/common/extHostModelView.ts @@ -2285,7 +2285,7 @@ class ChartComponentWrapper()); + this._emitterMap.set(ComponentEventType.onDidClick, new Emitter()); } public set chartType(v: TChartType) { @@ -2304,7 +2304,7 @@ class ChartComponentWrapper { + public get onDidClick(): vscode.Event { let emitter = this._emitterMap.get(ComponentEventType.onDidClick); return emitter && emitter.event; }