From f6484c872ac65da40dd1a0fc6ccd8d5f877873e9 Mon Sep 17 00:00:00 2001
From: chan_gami <1789422+changami@users.noreply.github.com>
Date: Mon, 23 Mar 2020 08:35:17 -0700
Subject: [PATCH] Fixes Japanese conversion bugs on Safari and Chrome (#11917)

decided to fix using flags since KeyboardEvent.isComposing behaves differently depending on the browser
---
 website/client/src/components/tasks/taskModal.vue | 11 +++++++++--
 1 file changed, 9 insertions(+), 2 deletions(-)

diff --git a/website/client/src/components/tasks/taskModal.vue b/website/client/src/components/tasks/taskModal.vue
index 59eaa47cec..deaf783157 100644
--- a/website/client/src/components/tasks/taskModal.vue
+++ b/website/client/src/components/tasks/taskModal.vue
@@ -138,7 +138,8 @@
             class="inline-edit-input checklist-item form-control"
             type="text"
             :placeholder="$t('newChecklistItem')"
-            @keydown.enter="addChecklistItem($event)"
+            @keypress.enter="setHasPossibilityOfIMEConversion(false)"
+            @keyup.enter="addChecklistItem($event)"
           >
         </div>
         <div
@@ -1222,6 +1223,7 @@ export default {
         per: 'perception',
       },
       calendarHighlights: { dates: [new Date()] },
+      hasPossibilityOfIMEConversion: true,
     };
   },
   computed: {
@@ -1384,7 +1386,12 @@ export default {
       sorting.splice(data.newIndex, 0, movingItem);
       this.task.checklist = sorting;
     },
+    setHasPossibilityOfIMEConversion (bool) {
+      this.hasPossibilityOfIMEConversion = bool;
+    },
     addChecklistItem (e) {
+      if (e) e.preventDefault();
+      if (this.hasPossibilityOfIMEConversion) return;
       const checkListItem = {
         id: uuid.v4(),
         text: this.newChecklistItem,
@@ -1394,7 +1401,7 @@ export default {
       // @TODO: managing checklist separately to help with sorting on the UI
       this.checklist.push(checkListItem);
       this.newChecklistItem = null;
-      if (e) e.preventDefault();
+      this.setHasPossibilityOfIMEConversion(true);
     },
     removeChecklistItem (i) {
       this.task.checklist.splice(i, 1);