]> cat aescling's git repositories - mastodon.git/commitdiff
[Glitch] Fix nodes order being sometimes mangled when rewriting emoji
authorClaire <claire.github-309c@sitedethib.com>
Mon, 14 Nov 2022 21:14:55 +0000 (22:14 +0100)
committeraescling <aescling+gitlab@cat.family>
Tue, 15 Nov 2022 05:31:55 +0000 (00:31 -0500)
Port ccbca50a2502108884f7f3c9ea24df8b2129731d to glitch-soc

Signed-off-by: Claire <claire.github-309c@sitedethib.com>
app/javascript/flavours/glitch/util/emoji/index.js

index 83e2d9f202a6601e877d4f1c3ee1579edf2a45c5..6b7de213c693510582438614a7c4bb0158a30a4f 100644 (file)
@@ -19,10 +19,13 @@ const emojiFilename = (filename) => {
   return borderedEmoji.includes(filename) ? (filename + '_border') : filename;
 };
 
+const domParser = new DOMParser();
+
 const emojifyTextNode = (node, customEmojis) => {
-  const parentElement = node.parentElement;
   let str = node.textContent;
 
+  const fragment = new DocumentFragment();
+
   for (;;) {
     let match, i = 0;
 
@@ -64,12 +67,16 @@ const emojifyTextNode = (node, customEmojis) => {
       }
     }
 
+    fragment.append(document.createTextNode(str.slice(0, i)));
+    if (replacement) {
+      fragment.append(domParser.parseFromString(replacement, 'text/html').documentElement.getElementsByTagName('img')[0]);
+    }
     node.textContent = str.slice(0, i);
-    parentElement.insertAdjacentHTML('beforeend', replacement);
     str = str.slice(rend);
-    node = document.createTextNode(str);
-    parentElement.append(node);
   }
+
+  fragment.append(document.createTextNode(str));
+  node.parentElement.replaceChild(fragment, node);
 };
 
 const emojifyNode = (node, customEmojis) => {