From ef551da7c2353e7d8fdce3e15e05d3247edc640b Mon Sep 17 00:00:00 2001
From: Ugo Palatucci <upalatuc@redhat.com>
Date: Wed, 22 Mar 2023 12:50:55 +0100
Subject: [PATCH] Bug 2177091: hide edit buttons on catalog drawer

---
 .../HardwareDevices/HardwareDeviceTitle.tsx   | 12 +++++++++--
 .../HardwareDevices/HardwareDevices.tsx       | 21 +++++++++++++++----
 .../TemplatesCatalogDrawerPanel.tsx           |  2 +-
 3 files changed, 28 insertions(+), 7 deletions(-)

diff --git a/src/utils/components/HardwareDevices/HardwareDeviceTitle.tsx b/src/utils/components/HardwareDevices/HardwareDeviceTitle.tsx
index a73f0ec1c..1126e4263 100644
--- a/src/utils/components/HardwareDevices/HardwareDeviceTitle.tsx
+++ b/src/utils/components/HardwareDevices/HardwareDeviceTitle.tsx
@@ -1,4 +1,4 @@
-import * as React from 'react';
+import React, { FC } from 'react';
 
 import { useKubevirtTranslation } from '@kubevirt-utils/hooks/useKubevirtTranslation';
 import { Button, DescriptionListTerm } from '@patternfly/react-core';
@@ -7,12 +7,20 @@ import { PencilAltIcon } from '@patternfly/react-icons';
 type HardwareDeviceTitleProps = {
   title: string;
   canEdit: boolean;
+  hideEdit?: boolean;
   onClick?: () => void;
 };
 
-const HardwareDeviceTitle: React.FC<HardwareDeviceTitleProps> = ({ title, canEdit, onClick }) => {
+const HardwareDeviceTitle: FC<HardwareDeviceTitleProps> = ({
+  title,
+  canEdit,
+  hideEdit = false,
+  onClick,
+}) => {
   const { t } = useKubevirtTranslation();
 
+  if (hideEdit) return <DescriptionListTerm>{title}</DescriptionListTerm>;
+
   return (
     <DescriptionListTerm>
       <Button
diff --git a/src/utils/components/HardwareDevices/HardwareDevices.tsx b/src/utils/components/HardwareDevices/HardwareDevices.tsx
index 622848a8c..3a8c4dc61 100644
--- a/src/utils/components/HardwareDevices/HardwareDevices.tsx
+++ b/src/utils/components/HardwareDevices/HardwareDevices.tsx
@@ -1,4 +1,4 @@
-import * as React from 'react';
+import React, { FC } from 'react';
 
 import { V1VirtualMachine, V1VirtualMachineInstance } from '@kubevirt-ui/kubevirt-api/kubevirt';
 import { useKubevirtTranslation } from '@kubevirt-utils/hooks/useKubevirtTranslation';
@@ -21,10 +21,17 @@ type HardwareDevicesProps = {
   vm: V1VirtualMachine;
   onSubmit?: (vm: V1VirtualMachine) => Promise<void | V1VirtualMachine>;
   canEdit?: boolean;
+  hideEdit?: boolean;
   vmi?: V1VirtualMachineInstance;
 };
 
-const HardwareDevices: React.FC<HardwareDevicesProps> = ({ vm, vmi, onSubmit, canEdit = true }) => {
+const HardwareDevices: FC<HardwareDevicesProps> = ({
+  vm,
+  vmi,
+  onSubmit,
+  canEdit = true,
+  hideEdit = false,
+}) => {
   const { t } = useKubevirtTranslation();
   const { createModal } = useModal();
 
@@ -66,7 +73,12 @@ const HardwareDevices: React.FC<HardwareDevicesProps> = ({ vm, vmi, onSubmit, ca
   return (
     <DescriptionList>
       <DescriptionListGroup>
-        <HardwareDeviceTitle title={t('GPU devices')} onClick={onEditGPU} canEdit={canEdit} />
+        <HardwareDeviceTitle
+          title={t('GPU devices')}
+          onClick={onEditGPU}
+          canEdit={canEdit}
+          hideEdit={hideEdit}
+        />
         <DescriptionListDescription>
           <HardwareDevicesTable devices={gpus} />
         </DescriptionListDescription>
@@ -77,12 +89,13 @@ const HardwareDevices: React.FC<HardwareDevicesProps> = ({ vm, vmi, onSubmit, ca
           title={t('Host devices')}
           onClick={onEditHostDevices}
           canEdit={canEdit}
+          hideEdit={hideEdit}
         />
         <DescriptionListDescription>
           <HardwareDevicesTable devices={hostDevices} />
         </DescriptionListDescription>
       </DescriptionListGroup>
-      {canEdit && (
+      {canEdit && !hideEdit && (
         <DescriptionListGroup>
           <HardwareDevicesHeadlessMode vm={vm} vmi={vmi} onSubmit={onSubmit} />
         </DescriptionListGroup>
diff --git a/src/views/catalog/templatescatalog/components/TemplatesCatalogDrawer/TemplatesCatalogDrawerPanel.tsx b/src/views/catalog/templatescatalog/components/TemplatesCatalogDrawer/TemplatesCatalogDrawerPanel.tsx
index 4bcf12ab6..db51be9be 100644
--- a/src/views/catalog/templatescatalog/components/TemplatesCatalogDrawer/TemplatesCatalogDrawerPanel.tsx
+++ b/src/views/catalog/templatescatalog/components/TemplatesCatalogDrawer/TemplatesCatalogDrawerPanel.tsx
@@ -172,7 +172,7 @@ export const TemplatesCatalogDrawerPanel: React.FC<TemplatesCatalogDrawerPanelPr
                           {` (${hardwareDevicesCount})`}
                         </DescriptionListTerm>
                         <DescriptionListDescription>
-                          <HardwareDevices canEdit={false} vm={vmObject} />
+                          <HardwareDevices hideEdit vm={vmObject} />
                         </DescriptionListDescription>
                       </DescriptionListGroup>
                     </DescriptionList>