L'emoji est inséré au mauvais endroit

Bug
10

Ah, je vois, ça résout le problème :

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))
-        ) {
+        // Pour l'autocomplétion d'emojis (key === ':'), utiliser une vérification plus permissive qui inclut
+        // la ponctuation courante comme la virgule, le point, etc. qui peuvent apparaître avant les emojis
+        let isAllowed;
+        if (this.options.key === ":") {
+          // Correspond aux mêmes caractères autorisés dans l'expression régulière onKeyUp de l'autocomplétion d'emojis
+          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 Je ne suis pas très familier avec l’évolution de ce code, peut-être que certaines de ces choses te rappellent quelque chose de tes récents portages ?

2 « J'aime »