From dad737021f11f2e56d5d769e7161de55de9cc1fe Mon Sep 17 00:00:00 2001
From: Suhavi Sandhu <beautyspell5@gmail.com>
Date: Wed, 13 Jan 2021 08:46:45 -0500
Subject: [PATCH] =?UTF-8?q?Adds=20character=20count=20to=20reply=20box=20?=
 =?UTF-8?q?=F0=9F=98=B3=20(#1634)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

* Added characters remaining to reply box

* Changed warning text for character length

* Decreased font size of message
---
 .../widgets/WootWriter/ReplyTopPanel.vue      | 35 +++++++++++++++++--
 .../widgets/WootWriter/constants.js           |  5 +++
 .../widgets/conversation/ReplyBox.vue         | 13 ++++---
 3 files changed, 47 insertions(+), 6 deletions(-)

diff --git a/app/javascript/dashboard/components/widgets/WootWriter/ReplyTopPanel.vue b/app/javascript/dashboard/components/widgets/WootWriter/ReplyTopPanel.vue
index 390bf1160..775087719 100644
--- a/app/javascript/dashboard/components/widgets/WootWriter/ReplyTopPanel.vue
+++ b/app/javascript/dashboard/components/widgets/WootWriter/ReplyTopPanel.vue
@@ -19,12 +19,18 @@
         {{ $t('CONVERSATION.REPLYBOX.PRIVATE_NOTE') }}
       </button>
     </div>
-    <div class="action-wrap"></div>
+    <div class="action-wrap">
+      <div v-if="isMessageLengthReachingThreshold" class="tabs-title">
+        <span :class="charLengthClass">
+          {{ characterLengthWarning }}
+        </span>
+      </div>
+    </div>
   </div>
 </template>
 
 <script>
-import { REPLY_EDITOR_MODES } from './constants';
+import { REPLY_EDITOR_MODES, CHAR_LENGTH_WARNING } from './constants';
 import EmojiOrIcon from 'shared/components/EmojiOrIcon';
 export default {
   name: 'ReplyTopPanel',
@@ -40,6 +46,14 @@ export default {
       type: Function,
       default: () => {},
     },
+    isMessageLengthReachingThreshold: {
+      type: Boolean,
+      default: () => false,
+    },
+    charactersRemaining: {
+      type: Number,
+      default: () => 0,
+    },
   },
   computed: {
     replyButtonClass() {
@@ -52,6 +66,14 @@ export default {
         'is-active': this.mode === REPLY_EDITOR_MODES.NOTE,
       };
     },
+    charLengthClass() {
+      return this.charactersRemaining < 0 ? 'message-error' : 'message-length';
+    },
+    characterLengthWarning() {
+      return this.charactersRemaining < 0
+        ? `${-this.charactersRemaining} ${CHAR_LENGTH_WARNING.NEGATIVE}`
+        : `${this.charactersRemaining} ${CHAR_LENGTH_WARNING.UNDER_50}`;
+    },
   },
   methods: {
     handleReplyClick() {
@@ -113,5 +135,14 @@ export default {
 .action-wrap {
   display: flex;
   align-items: center;
+  margin: 0 var(--space-normal);
+  font-size: var(--font-size-mini);
+
+  .message-error {
+    color: var(--r-600);
+  }
+  .message-length {
+    color: var(--s-600);
+  }
 }
 </style>
diff --git a/app/javascript/dashboard/components/widgets/WootWriter/constants.js b/app/javascript/dashboard/components/widgets/WootWriter/constants.js
index ef494d5ec..f3fae44f1 100644
--- a/app/javascript/dashboard/components/widgets/WootWriter/constants.js
+++ b/app/javascript/dashboard/components/widgets/WootWriter/constants.js
@@ -2,3 +2,8 @@ export const REPLY_EDITOR_MODES = {
   REPLY: 'REPLY',
   NOTE: 'NOTE',
 };
+
+export const CHAR_LENGTH_WARNING = {
+  UNDER_50: 'characters remaining',
+  NEGATIVE: 'characters over',
+};
diff --git a/app/javascript/dashboard/components/widgets/conversation/ReplyBox.vue b/app/javascript/dashboard/components/widgets/conversation/ReplyBox.vue
index 2c3e166c1..16010fbe6 100644
--- a/app/javascript/dashboard/components/widgets/conversation/ReplyBox.vue
+++ b/app/javascript/dashboard/components/widgets/conversation/ReplyBox.vue
@@ -1,6 +1,11 @@
 <template>
   <div class="reply-box" :class="replyBoxClass">
-    <reply-top-panel :mode="replyType" :set-reply-mode="setReplyMode" />
+    <reply-top-panel
+      :mode="replyType"
+      :set-reply-mode="setReplyMode"
+      :is-message-length-reaching-threshold="isMessageLengthReachingThreshold"
+      :characters-remaining="charactersRemaining"
+    />
     <div class="reply-box__top">
       <canned-response
         v-if="showCannedResponsesList"
@@ -111,10 +116,10 @@ export default {
         : this.$t('CONVERSATION.FOOTER.MSG_INPUT');
     },
     isMessageLengthReachingThreshold() {
-      return this.message.length > this.maxLength - 40;
+      return this.message.length > this.maxLength - 50;
     },
-    characterCountIndicator() {
-      return `${this.message.length} / ${this.maxLength}`;
+    charactersRemaining() {
+      return this.maxLength - this.message.length;
     },
     isReplyButtonDisabled() {
       const isMessageEmpty = !this.message.trim().replace(/\n/g, '').length;