Fix CJK IME(like Chinese, Japanese) double input (#4702)

* fix: resolve CJK IME double input * fix: resolve UC mobile no input issue * feat: add changeset * Update .changeset/large-melons-warn.md Co-authored-by: Dylan Schiemann <dylan@dojotoolkit.org> * fix: more specific way to deal with browsers Co-authored-by: Dylan Schiemann <dylan@dojotoolkit.org>
ianstormtaylor · Dec 5, 2021 · 8bc6a46 · 8bc6a46
1 parent c020ca2
commit 8bc6a46
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 0 deletions.
diff --git a/.changeset/large-melons-warn.md b/.changeset/large-melons-warn.md
@@ -0,0 +1,5 @@
+---
+'slate-react': minor
+---
+
+Fix CJK IME (e.g. Chinese or Japanese) double input
diff --git a/packages/slate-react/src/components/editable.tsx b/packages/slate-react/src/components/editable.tsx
@@ -24,6 +24,8 @@ import {
   IS_FIREFOX_LEGACY,
   IS_QQBROWSER,
   IS_SAFARI,
+  IS_UC_MOBILE,
+  IS_WECHATBROWSER,
   CAN_USE_DOM,
 } from '../utils/environment'
 import { ReactEditor } from '..'
@@ -772,6 +774,8 @@ export const Editable = (props: EditableProps) => {
                   !IS_FIREFOX_LEGACY &&
                   !IS_IOS &&
                   !IS_QQBROWSER &&
+                  !IS_WECHATBROWSER &&
+                  !IS_UC_MOBILE &&
                   event.data
                 ) {
                   Editor.insertText(editor, event.data)

diff --git a/packages/slate-react/src/utils/environment.ts b/packages/slate-react/src/utils/environment.ts
@@ -43,6 +43,14 @@ export const IS_FIREFOX_LEGACY =
 export const IS_QQBROWSER =
   typeof navigator !== 'undefined' && /.*QQBrowser/.test(navigator.userAgent)
 
+// UC mobile browser
+export const IS_UC_MOBILE =
+  typeof navigator !== 'undefined' && /.*UCBrowser/.test(navigator.userAgent)
+
+// Wechat browser
+export const IS_WECHATBROWSER =
+  typeof navigator !== 'undefined' && /.*Wechat/.test(navigator.userAgent)
+
 // Check if DOM is available as React does internally.
 // https://github.com/facebook/react/blob/master/packages/shared/ExecutionEnvironment.js
 export const CAN_USE_DOM = !!(