Emoji inserido no lugar errado

Bug
10

Ah, entendi, isso corrige:

diff --git a/frontend/discourse/app/modifiers/d-autocomplete.js b/frontend/discourse/app/modifiers/d-autocomplete.js
index e49fc0ec53..33fb51a1f4 100644
--- a/frontend/discourse/app/modifiers/d-autocomplete.js
+++ b/frontend/discourse/app/modifiers/d-autocomplete.js
@@ -599,10 +599,19 @@ export default class DAutocompleteModifier extends Modifier {
         prev = this.getValue()[caretPos - 1];
         const shouldTrigger = await this.shouldTrigger({ backSpace });
 
-        if (
-          shouldTrigger &&
-          (prev === undefined || this.ALLOWED_LETTERS_REGEXP.test(prev))
-        ) {
+        // Para autocompletar emoji (key === ':'), use uma verificação mais permissiva que inclua
+        // pontuação comum como vírgula, ponto, etc., que podem aparecer antes de emojis
+        let isAllowed;
+        if (this.options.key === ":") {
+          // Corresponde aos mesmos caracteres permitidos na regex onKeyUp do autocompletar de emoji
+          isAllowed =
+            prev === undefined || /[\\s.?,@/#!%\u0026*;:\\[\\]{}=\\-_()+]/.test(prev);
+        } else {
+          isAllowed =
+            prev === undefined || this.ALLOWED_LETTERS_REGEXP.test(prev);
+        }
+
+        if (shouldTrigger && isAllowed) {
           start = caretPos;
           term = this.getValue().substring(caretPos + 1, initialCaretPos);
           end = caretPos + term.length;

@kelv Não estou muito familiarizado com a evolução deste código, talvez algo disso soe familiar com alguma das portabilidades que você fez recentemente?

2 curtidas