wrapEmoji should ignore all non-unicode (#41)

webmuds · May 3, 2024 · 7a6ebf3 · 7a6ebf3
1 parent 87898fd
commit 7a6ebf3
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/src/constants.js b/src/constants.js
@@ -28,7 +28,7 @@ export const STRIP_RX = new RegExp('{(?:' + COLORS + '|/)}', 'gim')
 /**
  * RegExp to wrap all emoji in <span> tags.
  */
-export const EMOJI_RX = /([\p{Emoji}\p{Emoji_Presentation}]+)/gimsu
+export const EMOJI_RX = /([\p{Emoji_Presentation}]+)/gu
 
 /**
  * Template to use when replacing emoji.

diff --git a/test/wrapEmoji.js b/test/wrapEmoji.js
@@ -11,7 +11,7 @@ describe('#wrapEmoji', function () {
   })
 
   it('does not affect non-emoji', function () {
-    expect(wrapEmoji('abc😅def')).to.eq('abc<span class="wmE">😅</span>def')
+    expect(wrapEmoji('123abc😅def456#*')).to.eq('123abc<span class="wmE">😅</span>def456#*')
   })
 
   it('tags all emojis', function () {
@@ -25,4 +25,8 @@ describe('#wrapEmoji', function () {
   it('tags sequential emojis in a single tag', function () {
     expect(wrapEmoji('abc 😂💯😅 def')).to.eq('abc <span class="wmE">😂💯😅</span> def')
   })
+
+  it('breaks down sequential emojis split by multiline', function () {
+    expect(wrapEmoji('abc 😂\n💯😅 def')).to.eq('abc <span class="wmE">😂</span>\n<span class="wmE">💯😅</span> def')
+  })
 })