From 8f93fd9d094a9d56fd75af97456af0dbeb8f9d1a Mon Sep 17 00:00:00 2001 From: Lewis Sanchez <87730006+lewis-sanchez@users.noreply.github.com> Date: Tue, 12 Jul 2022 17:14:49 -0700 Subject: [PATCH] Updates Actual Execution Plan button icon (#20018) --- .../disabled-actual-execution-plan-inverse.svg | 18 +++++++++++++++++- .../media/disabled-actual-execution-plan.svg | 11 ++++++++++- .../enabled-actual-execution-plan-inverse.svg | 16 +++++++++++++++- .../media/enabled-actual-execution-plan.svg | 10 +++++++++- .../contrib/query/browser/queryActions.ts | 7 +++++++ 5 files changed, 58 insertions(+), 4 deletions(-) diff --git a/src/sql/base/browser/ui/taskbar/media/disabled-actual-execution-plan-inverse.svg b/src/sql/base/browser/ui/taskbar/media/disabled-actual-execution-plan-inverse.svg index 1c7c0724d8..d10036a68a 100644 --- a/src/sql/base/browser/ui/taskbar/media/disabled-actual-execution-plan-inverse.svg +++ b/src/sql/base/browser/ui/taskbar/media/disabled-actual-execution-plan-inverse.svg @@ -1 +1,17 @@ -query_plan_inverse_16x16 \ No newline at end of file + + + + + + + + + + + + + diff --git a/src/sql/base/browser/ui/taskbar/media/disabled-actual-execution-plan.svg b/src/sql/base/browser/ui/taskbar/media/disabled-actual-execution-plan.svg index d1c246bf81..7d932aca19 100644 --- a/src/sql/base/browser/ui/taskbar/media/disabled-actual-execution-plan.svg +++ b/src/sql/base/browser/ui/taskbar/media/disabled-actual-execution-plan.svg @@ -1 +1,10 @@ -query_plan_16x16 \ No newline at end of file + + + + + + + + + + diff --git a/src/sql/base/browser/ui/taskbar/media/enabled-actual-execution-plan-inverse.svg b/src/sql/base/browser/ui/taskbar/media/enabled-actual-execution-plan-inverse.svg index 1c7c0724d8..7883a62651 100644 --- a/src/sql/base/browser/ui/taskbar/media/enabled-actual-execution-plan-inverse.svg +++ b/src/sql/base/browser/ui/taskbar/media/enabled-actual-execution-plan-inverse.svg @@ -1 +1,15 @@ -query_plan_inverse_16x16 \ No newline at end of file + + + + + + + + + + + + diff --git a/src/sql/base/browser/ui/taskbar/media/enabled-actual-execution-plan.svg b/src/sql/base/browser/ui/taskbar/media/enabled-actual-execution-plan.svg index d1c246bf81..722a03fbc3 100644 --- a/src/sql/base/browser/ui/taskbar/media/enabled-actual-execution-plan.svg +++ b/src/sql/base/browser/ui/taskbar/media/enabled-actual-execution-plan.svg @@ -1 +1,9 @@ -query_plan_16x16 \ No newline at end of file + + + + + + + + + diff --git a/src/sql/workbench/contrib/query/browser/queryActions.ts b/src/sql/workbench/contrib/query/browser/queryActions.ts index 0802875ac5..ce745767cb 100644 --- a/src/sql/workbench/contrib/query/browser/queryActions.ts +++ b/src/sql/workbench/contrib/query/browser/queryActions.ts @@ -341,6 +341,7 @@ export class EstimatedQueryPlanAction extends QueryTaskbarAction { */ export class ToggleActualExecutionPlanModeAction extends QueryTaskbarAction { public static EnabledClass = 'enabledActualExecutionPlan'; + public static DisabledClass = 'disabledActualExecutionPlan'; public static ID = 'toggleActualExecutionPlanModeAction'; private static readonly EnableActualPlanLabel = nls.localize('enableActualPlanLabel', "Enable Actual Plan"); @@ -370,6 +371,12 @@ export class ToggleActualExecutionPlanModeAction extends QueryTaskbarAction { private updateLabel(): void { // show option to disable actual plan mode if already enabled this.label = this.isActualExecutionPlanMode ? ToggleActualExecutionPlanModeAction.DisableActualPlanLabel : ToggleActualExecutionPlanModeAction.EnableActualPlanLabel; + if (this.isActualExecutionPlanMode) { + this.updateCssClass(ToggleActualExecutionPlanModeAction.DisabledClass); + } + else { + this.updateCssClass(ToggleActualExecutionPlanModeAction.EnabledClass); + } } public override async run(): Promise {