From 6601812022f051d4f1b11608bdcb6dffdabaf174 Mon Sep 17 00:00:00 2001
From: Anmol Singh Bhatia <anmolsinghbhatia@plane.so>
Date: Tue, 24 Sep 2024 20:55:39 +0530
Subject: [PATCH] fix: issue quick action

---
 .../issues/relations/issue-list-item.tsx      | 13 ++++++++---
 .../issues/sub-issues/issue-list-item.tsx     | 22 +++++++++++++------
 2 files changed, 25 insertions(+), 10 deletions(-)

diff --git a/web/core/components/issues/relations/issue-list-item.tsx b/web/core/components/issues/relations/issue-list-item.tsx
index 4f24679cf5b..acb8187dc56 100644
--- a/web/core/components/issues/relations/issue-list-item.tsx
+++ b/web/core/components/issues/relations/issue-list-item.tsx
@@ -63,18 +63,25 @@ export const RelationIssueListItem: FC<Props> = observer((props) => {
   // handlers
   const handleIssuePeekOverview = (issue: TIssue) => handleRedirection(workspaceSlug, issue, isMobile);
 
-  const handleEditIssue = () => {
+  const handleEditIssue = (e: React.MouseEvent<HTMLButtonElement, MouseEvent>) => {
+    e.stopPropagation();
+    e.preventDefault();
     handleIssueCrudState("update", relationIssueId, { ...issue });
     toggleCreateIssueModal(true);
   };
 
-  const handleDeleteIssue = () => {
+  const handleDeleteIssue = (e: React.MouseEvent<HTMLButtonElement, MouseEvent>) => {
+    e.stopPropagation();
+    e.preventDefault();
     handleIssueCrudState("delete", relationIssueId, issue);
     toggleDeleteIssueModal(relationIssueId);
   };
 
-  const handleCopyIssueLink = () =>
+  const handleCopyIssueLink = (e: React.MouseEvent<HTMLButtonElement, MouseEvent>) => {
+    e.stopPropagation();
+    e.preventDefault();
     issueOperations.copyText(`${workspaceSlug}/projects/${issue.project_id}/issues/${issue.id}`);
+  };
 
   const handleRemoveRelation = (e: React.MouseEvent<HTMLButtonElement, MouseEvent>) => {
     e.preventDefault();
diff --git a/web/core/components/issues/sub-issues/issue-list-item.tsx b/web/core/components/issues/sub-issues/issue-list-item.tsx
index e2ad9c5cdae..78241633ed2 100644
--- a/web/core/components/issues/sub-issues/issue-list-item.tsx
+++ b/web/core/components/issues/sub-issues/issue-list-item.tsx
@@ -162,7 +162,9 @@ export const IssueListItem: React.FC<ISubIssues> = observer((props) => {
               <CustomMenu placement="bottom-end" ellipsis>
                 {disabled && (
                   <CustomMenu.MenuItem
-                    onClick={() => {
+                    onClick={(e) => {
+                      e.preventDefault();
+                      e.stopPropagation();
                       handleIssueCrudState("update", parentIssueId, { ...issue });
                       toggleCreateIssueModal(true);
                     }}
@@ -175,9 +177,11 @@ export const IssueListItem: React.FC<ISubIssues> = observer((props) => {
                 )}
 
                 <CustomMenu.MenuItem
-                  onClick={() =>
-                    subIssueOperations.copyText(`${workspaceSlug}/projects/${issue.project_id}/issues/${issue.id}`)
-                  }
+                  onClick={(e) => {
+                    e.stopPropagation();
+                    e.preventDefault();
+                    subIssueOperations.copyText(`${workspaceSlug}/projects/${issue.project_id}/issues/${issue.id}`);
+                  }}
                 >
                   <div className="flex items-center gap-2">
                     <LinkIcon className="h-3.5 w-3.5" strokeWidth={2} />
@@ -187,8 +191,10 @@ export const IssueListItem: React.FC<ISubIssues> = observer((props) => {
 
                 {disabled && (
                   <CustomMenu.MenuItem
-                    onClick={() => {
-                      issue.project_id &&
+                    onClick={(e) => {
+                      e.stopPropagation();
+                      e.preventDefault();
+                      if (issue.project_id)
                         subIssueOperations.removeSubIssue(workspaceSlug, issue.project_id, parentIssueId, issue.id);
                     }}
                   >
@@ -201,7 +207,9 @@ export const IssueListItem: React.FC<ISubIssues> = observer((props) => {
 
                 {disabled && (
                   <CustomMenu.MenuItem
-                    onClick={() => {
+                    onClick={(e) => {
+                      e.stopPropagation();
+                      e.preventDefault();
                       handleIssueCrudState("delete", parentIssueId, issue);
                       toggleDeleteIssueModal(issue.id);
                     }}