From 6794f331c33b74bebac8cf3a033d5be8bc5d4ea8 Mon Sep 17 00:00:00 2001
From: nixocio <nixocio@gmail.com>
Date: Tue, 25 Aug 2020 10:10:52 -0400
Subject: [PATCH] Update styles to delete button to be secondary

Update styles to delete button to be secondary.

See: https://github.com/ansible/awx/issues/7041
Also: https://github.com/ansible/awx/issues/7984
---
 .../InventoryGroupDetail/InventoryGroupDetail.jsx         | 2 +-
 .../Inventory/InventoryGroups/InventoryGroupsList.jsx     | 2 +-
 .../src/screens/Template/Survey/SurveyList.test.jsx       | 8 ++++----
 awx/ui_next/src/screens/Template/Survey/SurveyToolbar.jsx | 2 +-
 .../src/screens/Template/Survey/SurveyToolbar.test.jsx    | 6 +++---
 5 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/awx/ui_next/src/screens/Inventory/InventoryGroupDetail/InventoryGroupDetail.jsx b/awx/ui_next/src/screens/Inventory/InventoryGroupDetail/InventoryGroupDetail.jsx
index 7b5355d201..070e64d0ae 100644
--- a/awx/ui_next/src/screens/Inventory/InventoryGroupDetail/InventoryGroupDetail.jsx
+++ b/awx/ui_next/src/screens/Inventory/InventoryGroupDetail/InventoryGroupDetail.jsx
@@ -85,7 +85,7 @@ function InventoryGroupDetail({ i18n, inventoryGroup }) {
           {i18n._(t`Edit`)}
         </Button>
         <Button
-          variant="danger"
+          variant="secondary"
           aria-label={i18n._(t`Delete`)}
           onClick={() => setIsDeleteModalOpen(true)}
         >
diff --git a/awx/ui_next/src/screens/Inventory/InventoryGroups/InventoryGroupsList.jsx b/awx/ui_next/src/screens/Inventory/InventoryGroups/InventoryGroupsList.jsx
index 3ba0ec1fff..02daf2edb2 100644
--- a/awx/ui_next/src/screens/Inventory/InventoryGroups/InventoryGroupsList.jsx
+++ b/awx/ui_next/src/screens/Inventory/InventoryGroups/InventoryGroupsList.jsx
@@ -214,7 +214,7 @@ function InventoryGroupsList({ i18n }) {
               <Tooltip content={renderTooltip()} position="top" key="delete">
                 <div>
                   <Button
-                    variant="danger"
+                    variant="secondary"
                     aria-label={i18n._(t`Delete`)}
                     onClick={toggleModal}
                     isDisabled={
diff --git a/awx/ui_next/src/screens/Template/Survey/SurveyList.test.jsx b/awx/ui_next/src/screens/Template/Survey/SurveyList.test.jsx
index d1ee6a8e17..091b30940e 100644
--- a/awx/ui_next/src/screens/Template/Survey/SurveyList.test.jsx
+++ b/awx/ui_next/src/screens/Template/Survey/SurveyList.test.jsx
@@ -59,7 +59,7 @@ describe('<SurveyList />', () => {
     });
     wrapper.update();
 
-    expect(wrapper.find('Button[variant="danger"]').prop('isDisabled')).toBe(
+    expect(wrapper.find('Button[variant="secondary"]').prop('isDisabled')).toBe(
       true
     );
     expect(
@@ -76,11 +76,11 @@ describe('<SurveyList />', () => {
     expect(
       wrapper.find('Checkbox[aria-label="Select all"]').prop('isChecked')
     ).toBe(true);
-    expect(wrapper.find('Button[variant="danger"]').prop('isDisabled')).toBe(
+    expect(wrapper.find('Button[variant="secondary"]').prop('isDisabled')).toBe(
       false
     );
     act(() => {
-      wrapper.find('Button[variant="danger"]').invoke('onClick')();
+      wrapper.find('Button[variant="secondary"]').invoke('onClick')();
     });
 
     wrapper.update();
@@ -148,7 +148,7 @@ describe('<SurveyList />', () => {
     ).toBe(true);
     expect(wrapper.find('Switch').prop('isDisabled')).toBe(true);
     expect(wrapper.find('ToolbarAddButton').prop('isDisabled')).toBe(true);
-    expect(wrapper.find('Button[variant="danger"]').prop('isDisabled')).toBe(
+    expect(wrapper.find('Button[variant="secondary"]').prop('isDisabled')).toBe(
       true
     );
   });
diff --git a/awx/ui_next/src/screens/Template/Survey/SurveyToolbar.jsx b/awx/ui_next/src/screens/Template/Survey/SurveyToolbar.jsx
index 16eac2be70..d8c122a0e9 100644
--- a/awx/ui_next/src/screens/Template/Survey/SurveyToolbar.jsx
+++ b/awx/ui_next/src/screens/Template/Survey/SurveyToolbar.jsx
@@ -65,7 +65,7 @@ function SurveyToolbar({
           </ToolbarItem>
           <ToolbarItem>
             <Button
-              variant="danger"
+              variant="secondary"
               isDisabled={isDeleteDisabled}
               onClick={() => onToggleDeleteModal(true)}
             >
diff --git a/awx/ui_next/src/screens/Template/Survey/SurveyToolbar.test.jsx b/awx/ui_next/src/screens/Template/Survey/SurveyToolbar.test.jsx
index 2311f1d443..504385fafe 100644
--- a/awx/ui_next/src/screens/Template/Survey/SurveyToolbar.test.jsx
+++ b/awx/ui_next/src/screens/Template/Survey/SurveyToolbar.test.jsx
@@ -21,7 +21,7 @@ describe('<SurveyToolbar />', () => {
       );
     });
 
-    expect(wrapper.find('Button[variant="danger"]').prop('isDisabled')).toBe(
+    expect(wrapper.find('Button[variant="secondary"]').prop('isDisabled')).toBe(
       true
     );
   });
@@ -43,7 +43,7 @@ describe('<SurveyToolbar />', () => {
     expect(
       wrapper.find('Checkbox[aria-label="Select all"]').prop('isChecked')
     ).toBe(true);
-    expect(wrapper.find('Button[variant="danger"]').prop('isDisabled')).toBe(
+    expect(wrapper.find('Button[variant="secondary"]').prop('isDisabled')).toBe(
       false
     );
   });
@@ -108,7 +108,7 @@ describe('<SurveyToolbar />', () => {
     ).toBe(true);
     expect(wrapper.find('Switch').prop('isDisabled')).toBe(true);
     expect(wrapper.find('ToolbarAddButton').prop('isDisabled')).toBe(true);
-    expect(wrapper.find('Button[variant="danger"]').prop('isDisabled')).toBe(
+    expect(wrapper.find('Button[variant="secondary"]').prop('isDisabled')).toBe(
       true
     );
   });