From d26daaf8dc3acd31ef5ace2a7f72e6cbb90aa779 Mon Sep 17 00:00:00 2001
From: Aviv Turgeman <aturgema@redhat.com>
Date: Wed, 25 Sep 2024 13:03:57 +0300
Subject: [PATCH] CNV-43588: Start/Restart/Stop icon controls disappear during
 shutdown

Signed-off-by: Aviv Turgeman <aturgema@redhat.com>
---
 .../VMActionsIconBar/components/ActionIconButton.tsx   | 10 ++++++++--
 .../actions/components/VMActionsIconBar/utils/types.ts |  3 ++-
 .../actions/components/VMActionsIconBar/utils/utils.ts |  9 +++++----
 3 files changed, 15 insertions(+), 7 deletions(-)

diff --git a/src/views/virtualmachines/actions/components/VMActionsIconBar/components/ActionIconButton.tsx b/src/views/virtualmachines/actions/components/VMActionsIconBar/components/ActionIconButton.tsx
index 78cc2b4f6..dbeadede8 100644
--- a/src/views/virtualmachines/actions/components/VMActionsIconBar/components/ActionIconButton.tsx
+++ b/src/views/virtualmachines/actions/components/VMActionsIconBar/components/ActionIconButton.tsx
@@ -7,7 +7,13 @@ import { VMActionIconDetails } from '@virtualmachines/actions/components/VMActio
 
 import '../VMActionsIconBar.scss';
 
-const ActionIconButton: FC<VMActionIconDetails> = ({ action, Icon, iconClassname, isHidden }) => {
+const ActionIconButton: FC<VMActionIconDetails> = ({
+  action,
+  Icon,
+  iconClassname,
+  isDisabled,
+  isHidden,
+}) => {
   const [actionAllowed] = useAccessReview(action?.accessReview);
 
   const handleClick = () => {
@@ -23,7 +29,7 @@ const ActionIconButton: FC<VMActionIconDetails> = ({ action, Icon, iconClassname
           <Button
             className="vm-actions-icon-bar__button"
             data-test-id={`${action?.id}-button`}
-            isDisabled={!actionAllowed}
+            isDisabled={!actionAllowed || isDisabled}
             onClick={handleClick}
             variant={ButtonVariant.link}
           >
diff --git a/src/views/virtualmachines/actions/components/VMActionsIconBar/utils/types.ts b/src/views/virtualmachines/actions/components/VMActionsIconBar/utils/types.ts
index 77e02db60..a95b1d0f6 100644
--- a/src/views/virtualmachines/actions/components/VMActionsIconBar/utils/types.ts
+++ b/src/views/virtualmachines/actions/components/VMActionsIconBar/utils/types.ts
@@ -7,5 +7,6 @@ export type VMActionIconDetails = {
   action: Action;
   Icon: ComponentClass<SVGIconProps, any>;
   iconClassname?: string;
-  isHidden: boolean;
+  isDisabled?: boolean;
+  isHidden?: boolean;
 };
diff --git a/src/views/virtualmachines/actions/components/VMActionsIconBar/utils/utils.ts b/src/views/virtualmachines/actions/components/VMActionsIconBar/utils/utils.ts
index 380736c55..3dcd1bb90 100644
--- a/src/views/virtualmachines/actions/components/VMActionsIconBar/utils/utils.ts
+++ b/src/views/virtualmachines/actions/components/VMActionsIconBar/utils/utils.ts
@@ -17,17 +17,18 @@ export const getVMActionIconsDetails = (vm: V1VirtualMachine): VMActionIconDetai
     {
       action: VirtualMachineActionFactory.stop(vm),
       Icon: SquareIcon,
-      isHidden: !isRunning(vm) && !isPaused(vm),
+      isDisabled: !isRunning(vm) && !isPaused(vm),
     },
     {
       action: VirtualMachineActionFactory.restart(vm),
       Icon: RedoIcon,
-      isHidden: !isRunning(vm) && !isPaused(vm),
+      isDisabled: !isRunning(vm) && !isPaused(vm),
     },
     {
       action: VirtualMachineActionFactory.pause(vm),
       Icon: PauseIcon,
-      isHidden: !isRunning(vm),
+      isDisabled: !isRunning(vm),
+      isHidden: isPaused(vm),
     },
     {
       action: VirtualMachineActionFactory.unpause(vm),
@@ -38,7 +39,7 @@ export const getVMActionIconsDetails = (vm: V1VirtualMachine): VMActionIconDetai
     {
       action: VirtualMachineActionFactory.start(vm),
       Icon: PlayIcon,
-      isHidden: !isStopped(vm),
+      isDisabled: !isStopped(vm),
     },
   ];
 };