From 800f2cebb507da8133f768432c6eb45b0fde36cb Mon Sep 17 00:00:00 2001 From: Lewis Sanchez <87730006+lewis-sanchez@users.noreply.github.com> Date: Wed, 22 Jun 2022 14:59:13 -0700 Subject: [PATCH] Updates actual plan text to enable/disable instead of include/exclude (#19796) * Updates actual plan text to enable/disable instead of include/exclude * Makes actual plan button text static and readonly --- src/sql/workbench/contrib/query/browser/queryActions.ts | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/sql/workbench/contrib/query/browser/queryActions.ts b/src/sql/workbench/contrib/query/browser/queryActions.ts index 9dfe4a3757..ae3140ff89 100644 --- a/src/sql/workbench/contrib/query/browser/queryActions.ts +++ b/src/sql/workbench/contrib/query/browser/queryActions.ts @@ -343,8 +343,8 @@ export class ToggleActualExecutionPlanModeAction extends QueryTaskbarAction { public static EnabledClass = 'enabledActualExecutionPlan'; public static ID = 'toggleActualExecutionPlanModeAction'; - private _enableActualPlanLabel = nls.localize('enableActualPlanLabel', "Include Actual Plan"); - private _disableActualPlanLabel = nls.localize('disableActualPlanLabel', "Exclude Actual Plan"); + private static readonly EnableActualPlanLabel = nls.localize('enableActualPlanLabel', "Enable Actual Plan"); + private static readonly DisableActualPlanLabel = nls.localize('disableActualPlanLabel', "Disable Actual Plan"); constructor( editor: QueryEditor, @@ -369,7 +369,7 @@ export class ToggleActualExecutionPlanModeAction extends QueryTaskbarAction { private updateLabel(): void { // show option to disable actual plan mode if already enabled - this.label = this.isActualExecutionPlanMode ? this._disableActualPlanLabel : this._enableActualPlanLabel; + this.label = this.isActualExecutionPlanMode ? ToggleActualExecutionPlanModeAction.DisableActualPlanLabel : ToggleActualExecutionPlanModeAction.EnableActualPlanLabel; } public override async run(): Promise {