From 2e71006f9d399fea08721316aceeced8bfc629a7 Mon Sep 17 00:00:00 2001
From: Nithin David Thomas <1277421+nithindavid@users.noreply.github.com>
Date: Tue, 29 Jun 2021 20:06:40 +0530
Subject: [PATCH] feat: Add option to delete message content (#2532)

---
 .../widgets/conversation/Message.vue          | 92 +++++++++++++++++-
 .../i18n/locale/en/conversation.json          |  6 ++
 .../components/MessageContextMenu.vue         | 93 +++++++++++++++++++
 3 files changed, 186 insertions(+), 5 deletions(-)
 create mode 100644 app/javascript/dashboard/modules/conversations/components/MessageContextMenu.vue

diff --git a/app/javascript/dashboard/components/widgets/conversation/Message.vue b/app/javascript/dashboard/components/widgets/conversation/Message.vue
index d5d6e9edf..6c319ee15 100644
--- a/app/javascript/dashboard/components/widgets/conversation/Message.vue
+++ b/app/javascript/dashboard/components/widgets/conversation/Message.vue
@@ -43,6 +43,16 @@
           :source-id="data.source_id"
         />
       </div>
+      <context-menu
+        v-if="isBubble"
+        :is-open="showContextMenu"
+        :show-copy="hasText"
+        :menu-position="contextMenuPosition"
+        @toggle="handleContextMenuClick"
+        @delete="handleDelete"
+        @copy="handleCopy"
+      />
+
       <spinner v-if="isPending" size="tiny" />
 
       <a
@@ -65,13 +75,18 @@
   </li>
 </template>
 <script>
+import copy from 'copy-text-to-clipboard';
+
 import messageFormatterMixin from 'shared/mixins/messageFormatterMixin';
 import timeMixin from '../../../mixins/time';
 import BubbleText from './bubble/Text';
 import BubbleImage from './bubble/Image';
 import BubbleFile from './bubble/File';
 import Spinner from 'shared/components/Spinner';
+import ContextMenu from 'dashboard/modules/conversations/components/MessageContextMenu';
+
 import { isEmptyObject } from 'dashboard/helper/commons';
+import alertMixin from 'shared/mixins/alertMixin';
 import contentTypeMixin from 'shared/mixins/contentTypeMixin';
 import BubbleActions from './bubble/Actions';
 import { MESSAGE_TYPE, MESSAGE_STATUS } from 'shared/constants/messages';
@@ -84,9 +99,10 @@ export default {
     BubbleText,
     BubbleImage,
     BubbleFile,
+    ContextMenu,
     Spinner,
   },
-  mixins: [timeMixin, messageFormatterMixin, contentTypeMixin],
+  mixins: [alertMixin, timeMixin, messageFormatterMixin, contentTypeMixin],
   props: {
     data: {
       type: Object,
@@ -97,6 +113,11 @@ export default {
       default: false,
     },
   },
+  data() {
+    return {
+      showContextMenu: false,
+    };
+  },
   computed: {
     message() {
       const botMessageContent = generateBotMessageContent(
@@ -147,10 +168,13 @@ export default {
     },
     alignBubble() {
       const { message_type: messageType } = this.data;
-      if (messageType === MESSAGE_TYPE.ACTIVITY) {
-        return 'center';
-      }
-      return !messageType ? 'left' : 'right';
+      const isCentered = messageType === MESSAGE_TYPE.ACTIVITY;
+      return {
+        center: isCentered,
+        left: !messageType,
+        right: !!messageType,
+        'has-context-menu': this.showContextMenu,
+      };
     },
     readableTime() {
       return this.messageStamp(
@@ -210,6 +234,33 @@ export default {
       if (this.isPending) return false;
       return !this.sender.type || this.sender.type === 'agent_bot';
     },
+    contextMenuPosition() {
+      const { message_type: messageType } = this.data;
+      return messageType ? 'right' : 'left';
+    },
+  },
+  methods: {
+    handleContextMenuClick() {
+      this.showContextMenu = !this.showContextMenu;
+    },
+    async handleDelete() {
+      const { conversation_id: conversationId, id: messageId } = this.data;
+      try {
+        await this.$store.dispatch('deleteMessage', {
+          conversationId,
+          messageId,
+        });
+        this.showAlert(this.$t('CONVERSATION.SUCCESS_DELETE_MESSAGE'));
+        this.showContextMenu = false;
+      } catch (error) {
+        this.showAlert(this.$t('CONVERSATION.FAIL_DELETE_MESSSAGE'));
+      }
+    },
+    handleCopy() {
+      copy(this.data.content);
+      this.showAlert(this.$t('CONTACT_PANEL.COPY_SUCCESSFUL'));
+      this.showContextMenu = false;
+    },
   },
 };
 </script>
@@ -294,4 +345,35 @@ export default {
     margin-left: var(--space-smaller);
   }
 }
+
+.button--delete-message {
+  visibility: hidden;
+}
+
+.wrap {
+  display: flex;
+  align-items: flex-end;
+}
+
+li.right .wrap {
+  flex-direction: row-reverse;
+}
+
+li.left,
+li.right {
+  &:hover .button--delete-message {
+    visibility: visible;
+  }
+}
+
+.has-context-menu {
+  background: var(--color-background);
+  .button--delete-message {
+    visibility: visible;
+  }
+}
+
+.context-menu {
+  position: relative;
+}
 </style>
diff --git a/app/javascript/dashboard/i18n/locale/en/conversation.json b/app/javascript/dashboard/i18n/locale/en/conversation.json
index 587530e74..133bdcaa2 100644
--- a/app/javascript/dashboard/i18n/locale/en/conversation.json
+++ b/app/javascript/dashboard/i18n/locale/en/conversation.json
@@ -27,6 +27,8 @@
     "REMOVE_SELECTION": "Remove Selection",
     "DOWNLOAD": "Download",
     "UPLOADING_ATTACHMENTS": "Uploading attachments...",
+    "SUCCESS_DELETE_MESSAGE": "Message deleted successfully",
+    "FAIL_DELETE_MESSSAGE": "Couldn't delete message! Try again",
     "NO_RESPONSE": "No response",
     "RATING_TITLE": "Rating",
     "FEEDBACK_TITLE": "Feedback",
@@ -66,6 +68,10 @@
       "SELECT_AGENT": "Select Agent",
       "REMOVE": "Remove",
       "ASSIGN": "Assign"
+    },
+    "CONTEXT_MENU": {
+      "COPY": "Copy",
+      "DELETE": "Delete"
     }
   },
   "EMAIL_TRANSCRIPT": {
diff --git a/app/javascript/dashboard/modules/conversations/components/MessageContextMenu.vue b/app/javascript/dashboard/modules/conversations/components/MessageContextMenu.vue
new file mode 100644
index 000000000..9a7cd1d4c
--- /dev/null
+++ b/app/javascript/dashboard/modules/conversations/components/MessageContextMenu.vue
@@ -0,0 +1,93 @@
+<template>
+  <div class="context-menu">
+    <woot-button
+      icon="ion-more"
+      size="large"
+      class="button--delete-message"
+      color-scheme="secondary"
+      variant="clear"
+      @click="handleContextMenuClick"
+    />
+    <div
+      v-if="isOpen"
+      v-on-clickaway="handleContextMenuClick"
+      class="dropdown-pane dropdown-pane--open"
+      :class="`dropdown-pane--${menuPosition}`"
+    >
+      <woot-dropdown-menu>
+        <woot-dropdown-item v-if="showCopy">
+          <woot-button
+            variant="clear"
+            size="small"
+            icon="ion-ios-copy-outline"
+            @click="handleCopy"
+          >
+            {{ $t('CONVERSATION.CONTEXT_MENU.COPY') }}
+          </woot-button>
+        </woot-dropdown-item>
+        <woot-dropdown-item>
+          <woot-button
+            variant="clear"
+            color-scheme="alert"
+            size="small"
+            icon="ion-trash-a"
+            @click="handleDelete"
+          >
+            {{ $t('CONVERSATION.CONTEXT_MENU.DELETE') }}
+          </woot-button>
+        </woot-dropdown-item>
+      </woot-dropdown-menu>
+    </div>
+  </div>
+</template>
+<script>
+import { mixin as clickaway } from 'vue-clickaway';
+
+import WootDropdownItem from 'shared/components/ui/dropdown/DropdownItem';
+import WootDropdownMenu from 'shared/components/ui/dropdown/DropdownMenu';
+
+export default {
+  components: {
+    WootDropdownMenu,
+    WootDropdownItem,
+  },
+  mixins: [clickaway],
+  props: {
+    isOpen: {
+      type: Boolean,
+      default: false,
+    },
+    showCopy: {
+      type: Boolean,
+      default: false,
+    },
+    menuPosition: {
+      type: String,
+      default: 'left',
+    },
+  },
+  methods: {
+    handleContextMenuClick() {
+      this.$emit('toggle', !this.isOpen);
+    },
+    handleCopy() {
+      this.$emit('copy');
+    },
+    handleDelete() {
+      this.$emit('delete');
+    },
+  },
+};
+</script>
+<style lang="scss" scoped>
+/* TDOD: Remove once MenuComponent supports postions */
+.dropdown-pane {
+  bottom: var(--space-large);
+}
+.dropdown-pane--left {
+  right: var(--space-small);
+}
+.dropdown-pane--right {
+  left: var(--space-small);
+}
+</style>