From bd15040bf0661b7afe42aa2b590ba25f9146aff5 Mon Sep 17 00:00:00 2001
From: Donkoko <nbonev@duck.com>
Date: Mon, 28 Aug 2023 16:31:52 +0300
Subject: [PATCH] fixing issue that doesn't allow to delete assets due to
 conflict with dropdown closing

---
 app/components/assets/actions-dropdown.tsx | 13 +++++--------
 app/components/assets/delete-asset.tsx     |  2 +-
 2 files changed, 6 insertions(+), 9 deletions(-)

diff --git a/app/components/assets/actions-dropdown.tsx b/app/components/assets/actions-dropdown.tsx
index 13860bcfd..8c1dd7366 100644
--- a/app/components/assets/actions-dropdown.tsx
+++ b/app/components/assets/actions-dropdown.tsx
@@ -61,7 +61,7 @@ const ConditionalActionsDropdown = ({ asset }: Props) => {
         className="order actions-dropdown static w-screen rounded-lg bg-white p-0 text-right md:static md:w-[180px]"
       >
         <div className="order fixed bottom-0 left-0 w-screen rounded-lg bg-white p-0 text-right md:static md:w-[180px]">
-          <DropdownMenuItem className="border-b px-6 py-3">
+          <DropdownMenuItem className="border-b p-3">
             {!assetIsAvailable ? (
               <Button
                 to="release-custody"
@@ -95,7 +95,7 @@ const ConditionalActionsDropdown = ({ asset }: Props) => {
               </Button>
             )}
           </DropdownMenuItem>
-          <DropdownMenuItem className="px-6 py-3">
+          <DropdownMenuItem className="p-3">
             <Button
               to="edit"
               role="link"
@@ -103,18 +103,15 @@ const ConditionalActionsDropdown = ({ asset }: Props) => {
               className="justify-start text-gray-700 hover:text-gray-700"
               width="full"
             >
-              <span
-                className="flex items-center gap-2"
-                onClick={() => setOpen(false)}
-              >
+              <span className="flex items-center gap-2">
                 <PenIcon /> Edit
               </span>
             </Button>
           </DropdownMenuItem>
-          <div onClick={() => setOpen(false)}>
+          <div>
             <DeleteAsset asset={asset} />
           </div>
-          <DropdownMenuItem className="border-t px-6 py-3 md:hidden">
+          <DropdownMenuItem className="border-t p-3 md:hidden">
             <Button
               role="button"
               variant="secondary"
diff --git a/app/components/assets/delete-asset.tsx b/app/components/assets/delete-asset.tsx
index d98249479..8e31a810e 100644
--- a/app/components/assets/delete-asset.tsx
+++ b/app/components/assets/delete-asset.tsx
@@ -28,7 +28,7 @@ export const DeleteAsset = ({
         variant="link"
         data-test-id="deleteAssetButton"
         icon="trash"
-        className="justify-start rounded-sm px-6 py-3 text-sm font-semibold text-gray-700 outline-none data-[disabled]:pointer-events-none data-[disabled]:opacity-50 hover:bg-slate-100 hover:text-gray-700"
+        className="justify-start rounded-sm p-3 text-sm font-semibold text-gray-700 outline-none data-[disabled]:pointer-events-none data-[disabled]:opacity-50 hover:bg-slate-100 hover:text-gray-700"
         width="full"
       >
         Delete