FIX: insertAdjacentHTML is a better choice here

discourse · Jun 1, 2020 · b4bf9b2 · b4bf9b2
1 parent 13f14c3
commit b4bf9b2
Showing 1 changed file with 4 additions and 9 deletions.
diff --git a/assets/javascripts/initializers/add-holiday-flair.js.es6 b/assets/javascripts/initializers/add-holiday-flair.js.es6
@@ -6,17 +6,12 @@ function applyFlairOnMention(element, username) {
   if (!element) return;
 
   const href = `${Discourse.BaseUri}/u/${username}`;
-
-  const mentions = element.querySelectorAll(
-    `a.mention[href="${href}"]:not(.on-holiday)`
-  );
-
-  const parser = new DOMParser();
-  const doc = parser.parseFromString(iconHTML("calendar-alt"), "text/html");
-  const iconDocument = doc.body.firstChild;
+  const mentions = element.querySelectorAll(`a.mention[href="${href}"]`);
 
   mentions.forEach(mention => {
-    mention.appendChild(iconDocument);
+    if (!mention.querySelector(".d-icon-calendar-alt")) {
+      mention.insertAdjacentHTML("beforeend", iconHTML("calendar-alt"));
+    }
     mention.classList.add("on-holiday");
   });
 }