From 0d726fd8f050ed515b8ba7e09f2225b4b26a3a44 Mon Sep 17 00:00:00 2001
From: jaasen-livefront <jaasen@livefront.com>
Date: Fri, 19 Jul 2024 10:24:08 -0700
Subject: [PATCH] finalize send list container

---
 apps/browser/src/_locales/en/messages.json    |  3 +++
 apps/desktop/src/locales/en/messages.json     | 24 -------------------
 apps/web/src/locales/en/messages.json         |  3 +++
 .../send-list-items-container.component.html  |  8 ++++---
 .../send-list-items-container.component.ts    |  4 +++-
 5 files changed, 14 insertions(+), 28 deletions(-)

diff --git a/apps/browser/src/_locales/en/messages.json b/apps/browser/src/_locales/en/messages.json
index 5b306fdb2a7..63d5f9eb9cb 100644
--- a/apps/browser/src/_locales/en/messages.json
+++ b/apps/browser/src/_locales/en/messages.json
@@ -1996,6 +1996,9 @@
   "passwordProtected": {
     "message": "Password protected"
   },
+  "copyLink": {
+    "message": "Copy link"
+  },
   "copySendLink": {
     "message": "Copy Send link",
     "description": "'Send' is a noun and the name of a feature called 'Bitwarden Send'. It should not be translated."
diff --git a/apps/desktop/src/locales/en/messages.json b/apps/desktop/src/locales/en/messages.json
index abbdfef84a0..31de4095827 100644
--- a/apps/desktop/src/locales/en/messages.json
+++ b/apps/desktop/src/locales/en/messages.json
@@ -1937,30 +1937,6 @@
   "copyLink": {
     "message": "Copy link"
   },
-  "copySendTitle": {
-    "message": "Copy link - $NAME$",
-    "placeholders": {
-      "name": {
-        "content": "$1"
-      }
-    }
-  },
-  "deleteSend": {
-    "message": "Delete - $NAME$",
-    "placeholders": {
-      "name": {
-        "content": "$1"
-      }
-    }
-  },
-  "editSendTitle": {
-    "message": "Edit - $NAME$",
-    "placeholders": {
-      "name": {
-        "content": "$1"
-      }
-    }
-  },
   "disabled": {
     "message": "Disabled"
   },
diff --git a/apps/web/src/locales/en/messages.json b/apps/web/src/locales/en/messages.json
index 73396c39c16..560803c87e0 100644
--- a/apps/web/src/locales/en/messages.json
+++ b/apps/web/src/locales/en/messages.json
@@ -4350,6 +4350,9 @@
     "message": "Send link",
     "description": "'Send' is a noun and the name of a feature called 'Bitwarden Send'. It should not be translated."
   },
+  "copyLink": {
+    "message": "Copy link"
+  },
   "copySendLink": {
     "message": "Copy Send link",
     "description": "'Send' is a noun and the name of a feature called 'Bitwarden Send'. It should not be translated."
diff --git a/libs/tools/send/send-ui/src/send-list-items-container/send-list-items-container.component.html b/libs/tools/send/send-ui/src/send-list-items-container/send-list-items-container.component.html
index feb87eda39e..6a4c6a308ed 100644
--- a/libs/tools/send/send-ui/src/send-list-items-container/send-list-items-container.component.html
+++ b/libs/tools/send/send-ui/src/send-list-items-container/send-list-items-container.component.html
@@ -9,7 +9,7 @@
     <bit-item *ngFor="let send of sends">
       <button
         bit-item-content
-        [appA11yTitle]="'editSendTitle' | i18n: (send.name | i18n)"
+        appA11yTitle="{{ 'edit' | i18n }} - {{ send.name }}"
         appStopClick
         type="button"
         class="tw-pb-1"
@@ -32,7 +32,8 @@
           <bit-item-action>
             <button
               type="button"
-              appA11yTitle="{{ 'copySendTitle' | i18n }}: {{ send.name | i18n }}"
+              (click)="copySendLink(send)"
+              appA11yTitle="{{ 'copyLink' | i18n }} - {{ send.name }}"
             >
               <i class="bwi tw-text-lg bwi-clone"></i>
             </button>
@@ -40,7 +41,8 @@
           <bit-item-action>
             <button
               type="button"
-              appA11yTitle="{{ 'deleteSendTitle' | i18n }}: {{ send.name | i18n }}"
+              (click)="deleteSend(send)"
+              appA11yTitle="{{ 'delete' | i18n }} - {{ send.name }}"
             >
               <i class="bwi tw-text-lg bwi-trash"></i>
             </button>
diff --git a/libs/tools/send/send-ui/src/send-list-items-container/send-list-items-container.component.ts b/libs/tools/send/send-ui/src/send-list-items-container/send-list-items-container.component.ts
index 8a3078489fc..ef7232e97a0 100644
--- a/libs/tools/send/send-ui/src/send-list-items-container/send-list-items-container.component.ts
+++ b/libs/tools/send/send-ui/src/send-list-items-container/send-list-items-container.component.ts
@@ -69,6 +69,8 @@ export class SendListItemsContainerComponent {
       return false;
     }
 
+    await this.sendApiService.delete(s.id);
+
     try {
       this.toastService.showToast({
         variant: "success",
@@ -80,7 +82,7 @@ export class SendListItemsContainerComponent {
     }
   }
 
-  async copy(s: SendView) {
+  async copySendLink(s: SendView) {
     const env = await firstValueFrom(this.environmentService.environment$);
     const link = env.getSendUrl() + s.accessId + "/" + s.urlB64Key;
     this.platformUtilsService.copyToClipboard(link);