Merge pull request #1838 from basecamp/fix-crash-when-inserting-an-emoji-in-a-maxlength-reaction

Fix crash when inserting an emoji into a max length reaction
2025-12-03 10:36:38 +01:00
parent 62e43c76fa f8eda42187
commit 297f727199
1 changed files with 6 additions and 1 deletions
@@ -6,7 +6,12 @@ export default class extends Controller {
  insertEmoji(event) {
    const emojiChar = event.target.getAttribute("data-emoji")
    const value = this.inputTarget.value
-    this.inputTarget.value = `${value}${emojiChar}`
+    const newValue = `${value}${emojiChar}`
+
+    if (this.inputTarget.maxLength > 0 && newValue.length <= this.inputTarget.maxLength) {
+      this.inputTarget.value = newValue
+    }
+
    this.inputTarget.focus()
  }
 }