L'emoji è inserito nel posto sbagliato

Bug
10

Capisco, questo risolve il problema:

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))
-        ) {
+        // Per l'autocompletamento emoji (key === ':'), usa un controllo più permissivo che include
+        // punteggiatura comune come virgola, punto, ecc. che possono apparire prima delle emoji
+        let isAllowed;
+        if (this.options.key === ":") {
+          // Corrisponde agli stessi caratteri consentiti nell'espressione regolare onKeyUp dell'autocompletamento 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 Non ho molta familiarità con l’evoluzione di questo codice, forse qualcosa ti ricorda riguardo al porting che hai fatto di recente?

2 Mi Piace