From bbf04d867ee68967373ace260a1ece4926799a55 Mon Sep 17 00:00:00 2001
From: Ahmet Kilinc <akx9@icloud.com>
Date: Wed, 6 Aug 2025 13:50:26 +0100
Subject: [PATCH] remove delete button and extract function

---
 apps/mail/components/mail/mail-list.tsx | 60 ++++++++-----------------
 1 file changed, 18 insertions(+), 42 deletions(-)

diff --git a/apps/mail/components/mail/mail-list.tsx b/apps/mail/components/mail/mail-list.tsx
index 4d114e7e1..9b7ef6f5e 100644
--- a/apps/mail/components/mail/mail-list.tsx
+++ b/apps/mail/components/mail/mail-list.tsx
@@ -1,6 +1,5 @@
 import {
   Archive2,
-  ArrowRight,
   Copy,
   ExclamationCircle,
   GroupPeople,
@@ -63,7 +62,7 @@ const Thread = memo(
     const [id, setThreadId] = useQueryState('threadId');
     const [focusedIndex, setFocusedIndex] = useAtom(focusedIndexAtom);
     const { markAsCopied, isCodeCopied } = useCopiedOtpCodes();
-    const { markAsUsed, isLinkUsed } = useMagicLinks();
+    const { isLinkUsed } = useMagicLinks();
 
     const { latestMessage, idToUse, cleanName, otpCode, magicLink } = useMemo(() => {
       const latestMessage = getThreadData?.latest;
@@ -221,6 +220,21 @@ const Thread = memo(
     const content = useMemo(() => {
       if (!latestMessage || !getThreadData) return null;
 
+      const copyCode = (e: React.MouseEvent, code: { id: string; code: string }) => {
+        e.stopPropagation();
+        e.preventDefault();
+        navigator.clipboard.writeText(code.code);
+        markAsCopied(code.id);
+        toast.success('Copied to clipboard');
+      };
+
+      const openMagicLink = (e: React.MouseEvent, url: string) => {
+        e.stopPropagation();
+        e.preventDefault();
+        window.open(url, '_blank');
+        toast.success('Opening magic link in new tab');
+      };
+
       return (
         <div
           className={cn('mx-1 select-none border-b md:my-1 md:border-none')}
@@ -553,30 +567,11 @@ const Thread = memo(
                         ? 'bg-green-500/10 hover:bg-green-500/20 dark:bg-green-500/10 dark:hover:bg-green-500/20'
                         : 'bg-black/10 hover:bg-black/20 dark:bg-white/10 dark:hover:bg-white/20',
                     )}
-                    onClick={(e) => {
-                      e.stopPropagation();
-                      e.preventDefault();
-                      navigator.clipboard.writeText(otpCode.code);
-                      markAsCopied(otpCode.id);
-                      toast.success('Copied to clipboard');
-                    }}
+                    onClick={(e) => copyCode(e, otpCode.code)}
                   >
                     <Copy className="h-2 w-2" />
                     <span className="font-mono text-xs">{otpCode.code}</span>
                   </Button>
-                  <Button
-                    variant="secondary"
-                    size="sm"
-                    className="z-10 flex h-6 flex-row gap-1 bg-red-500/10 !px-2 !py-1 text-xs hover:bg-red-500/20 dark:bg-red-500/10 dark:hover:bg-red-500/20"
-                    onClick={(e) => {
-                      e.stopPropagation();
-                      e.preventDefault();
-                      moveThreadTo('bin');
-                    }}
-                  >
-                    <Trash className="h-2 w-2 fill-red-500" />
-                    <span className="text-xs">Delete</span>
-                  </Button>
                 </div>
               </div>
             )}
@@ -593,30 +588,11 @@ const Thread = memo(
                         ? 'bg-green-500/10 hover:bg-green-500/20 dark:bg-green-500/10 dark:hover:bg-green-500/20'
                         : 'bg-black/10 hover:bg-black/20 dark:bg-white/10 dark:hover:bg-white/20',
                     )}
-                    onClick={(e) => {
-                      e.stopPropagation();
-                      e.preventDefault();
-                      window.open(magicLink.url, '_blank');
-                      markAsUsed(magicLink.id);
-                      toast.success('Opening magic link in new tab');
-                    }}
+                    onClick={(e) => openMagicLink(e, magicLink.url)}
                   >
                     <ExternalLink className="h-2 w-2" />
                     <span className="text-xs">Open Link</span>
                   </Button>
-                  <Button
-                    variant="secondary"
-                    size="sm"
-                    className="z-10 flex h-6 flex-row items-center gap-1 bg-red-500/10 !px-2 !py-1 text-xs hover:bg-red-500/20 dark:bg-red-500/10 dark:hover:bg-red-500/20"
-                    onClick={(e) => {
-                      e.stopPropagation();
-                      e.preventDefault();
-                      moveThreadTo('bin');
-                    }}
-                  >
-                    <Trash className="h-2 w-2 fill-red-500" />
-                    <span className="text-xs">Delete</span>
-                  </Button>
                 </div>
               </div>
             )}