handle unwrapped textnodes

issue #11
outsideris · Mar 10, 2018 · 5779f1d · 5779f1d
1 parent 351669b
commit 5779f1d
Showing 1 changed file with 12 additions and 1 deletion.
diff --git a/src/translate.js b/src/translate.js
@@ -191,7 +191,18 @@ export function enableTranslation(API_KEY, LANGUAGE) {
   document.querySelector('.js-discussion').addEventListener('click', (event) => {
     if (isTranslateButton(event.target) || isTranslateButton(event.target.parentNode)) {
       const commentBody = findCommentBody(event.target);
-      const commentParts = commentBody.parentElement.querySelectorAll(markdownTagSelector());
+      const commentParentClone = commentBody.parentElement.cloneNode(true)
+      // sometimes, there is textNode which isn't in any HTML tag.
+      // So, wrap it with p tag to translate it.
+      commentParentClone.querySelector('td').childNodes.forEach((n) => {
+        if (n.nodeType === Node.TEXT_NODE && n.nodeValue.trim() !== '' ) {
+          const wrap = document.createElement('p');
+          wrap.innerHTML = n.nodeValue.trim();
+          commentParentClone.querySelector('td').replaceChild(wrap, n);
+        }
+      });
+
+      const commentParts = commentParentClone.querySelectorAll(markdownTagSelector());
       const commentWrapper = commentBody.parentElement.parentElement;
 
       // prevent to translate twice