ed-mathml2tex 0.0.7 → 0.0.9

package/lib/mathml2latex.browser.cjs.js

@@ -147,7 +147,9 @@ const NodeTool = {
     return node.children;
   },
   getNodeName: function(node) {
-    return node.tagName.toLowerCase();
+    if (!node) return '';
+    const localName = node.localName || node.nodeName;
+    return localName ? localName.toLowerCase() : '';
   },
   getNodeText: function(node) {
     return node.textContent;
@@ -704,9 +706,9 @@ function parseLeaf(node) {
 function parseOperator(node) {
   let it = NodeTool.getNodeText(node).trim();
   
-  // Special case for arrow (→)
+  // Special case for arrow (→) with proper spacing
   if (it === "→") {
-    return "\\rightarrow";
+    return " \\rightarrow ";  // Add spaces around arrow
   }
   
   it = MathSymbol.parseOperator(it);
@@ -833,7 +835,28 @@ function getRender(node) {
   const nodeName = NodeTool.getNodeName(node);
   switch(nodeName){
     case 'msub':
-      render = getRender_default("@1_{@2}");
+      render = function(node, children) {
+        if (!children || children.length < 2) return '';
+        
+        const base = parse(children[0]);
+        if (!base) return '';
+        
+        const sub = children[1];
+        if (!sub) return base;
+
+        // Handle nested subscript with empty mrow
+        if (NodeTool.getNodeName(sub) === 'msub' && 
+            sub.firstChild && 
+            NodeTool.getNodeName(sub.firstChild) === 'mrow' &&
+            (!NodeTool.getNodeText(sub.firstChild) || NodeTool.getNodeText(sub.firstChild).trim() === '')) {
+          const lastChild = sub.lastChild;
+          if (!lastChild) return base;
+          return `${base}_${parse(lastChild)}`;
+        }
+        
+        // Regular subscript
+        return `${base}_{${parse(sub)}}`;
+      };
       break;
     case 'msup':
       render = getRender_default("@1^{@2}");
@@ -895,25 +918,55 @@ function getRender(node) {
 
 function renderTable(node, children) {
   const template = "\\begin{array}{l}@content\\end{array}";
-  const render = getRender_joinSeparator(template, "\\\\");
+  // Remove extra backslash and add proper spacing
+  const render = getRender_joinSeparator(template, " \\\\ ");
   return render(node, children);
 }
 
+function renderMover(node, children) {
+    // Check if children exists and has enough elements
+    if (!children || children.length < 2) return '';
+
+    const baseNode = children[0];
+    const overNode = children[1];
+    
+    // Check if nodes exist before accessing
+    if (!baseNode || !overNode) return '';
+    
+    const overText = NodeTool.getNodeText(overNode)?.trim() || '';
+    const isAccent = NodeTool.getAttr(node, "accent", "false") === "true";
+
+    // Handle vector notation
+    if (overText === "→" && isAccent) {
+        return `\\vec{${parse(baseNode)}}`;
+    }
+
+    return parse(baseNode);
+}
+
 function renderMfenced(node, children) {
     const [open, close, separatorsStr] = [
-        NodeTool.getAttr(node, 'open', '{'),
-        NodeTool.getAttr(node, 'close', '}'),
-        NodeTool.getAttr(node, 'separators', '|')
+        NodeTool.getAttr(node, 'open', '('),
+        NodeTool.getAttr(node, 'close', ')'),
+        NodeTool.getAttr(node, 'separators', ',')
     ];
 
     const parts = renderChildren(children);
-    const content = parts.join(separatorsStr).trim();
+    const content = parts.join(separatorsStr === '|' ? ',' : separatorsStr).trim();
     
-    if (open === '{' && close === '') {
-        return `\\left\\${open}${content}\\right.`;
+    // Handle system of equations with curly brace
+    if (open === '{') {
+        // If no closing brace specified, use \right.
+        return `\\left\\{${content}${close ? '\\right\\}' : '\\right.'}`;
     }
     
-    return `\\left\\${open}${content}\\right\\${close}`;
+    // Handle absolute value notation
+    if (open === '|' && (close === '|' || close === '')) {
+        return `\\left|${content}\\right|`;
+    }
+
+    // Regular cases with proper closing
+    return `\\left${open}${content}\\right${close || '.'}`;
 }
 
 function renderMfrac(node, children){
@@ -933,9 +986,9 @@ function renderMfrac(node, children){
     if((prevNode && NodeTool.getNodeText(prevNode).trim() === '(') &&
        (nextNode && NodeTool.getNodeText(nextNode).trim() === ')')
     ) {
-      render = getRender_default("\\DELETE_BRACKET_L\\binom{@1}{@2}\\DELETE_BRACKET_R");
+      render = getRender_default("\\binom{@1}{@2}");
     } else {
-      render = getRender_default("{}_{@2}^{@1}");
+      render = getRender_default("\\frac{@1}{@2}");
     }
   } else {
     render = getRender_default("\\frac{@1}{@2}");
@@ -991,62 +1044,6 @@ function renderMmultiscripts(node, children) {
   return [renderScripts(prevScripts), base, renderScripts(backScripts)].join('');
 }
 
-function renderMover(node, children) {
-    const nodes = flattenNodeTreeByNodeName(node, 'mover');
-    let result = undefined;
-
-    // Get the base node and check if it's a subscript or mrow
-    const baseNode = children[0];
-    const overNode = children[1];
-    const nodeName = NodeTool.getNodeName(baseNode);
-    const isSubscript = nodeName === 'msub';
-    const isMrow = nodeName === 'mrow';
-
-    // Handle case where the base is an arrow and mrow is above
-    const baseText = NodeTool.getNodeText(baseNode).trim();
-    if (baseText === "→" && NodeTool.getNodeName(overNode) === "mrow") {
-        return "\\rightarrow";
-    }
-
-    if (isSubscript) {
-        // Handle case like n₂ with arrow
-        const base = parse(baseNode);
-        return `\\overrightarrow{${base}}`;
-    }
-
-    if (isMrow) {
-        // Handle case like 0 with arrow
-        const base = parse(baseNode);
-        const overText = NodeTool.getNodeText(overNode).trim();
-        const isAccent = NodeTool.getAttr(node, "accent", "false") === "true";
-
-        if (overText === "→" && isAccent) {
-            return `\\overrightarrow{${base}}`;
-        }
-    }
-
-    // Handle case where there is only an arrow with no base
-    const overText = NodeTool.getNodeText(overNode).trim();
-    if (overText === "→" && NodeTool.getNodeText(baseNode).trim() === "") {
-        return `\\rightarrow`; // Return just the arrow
-    }
-
-    for (let i = 0; i < nodes.length - 1; i++) {
-        if (!result) {
-            result = parse(nodes[i]);
-        }
-
-        const _overNode = nodes[i + 1];
-        const overText = NodeTool.getNodeText(_overNode).trim();
-        const isAccent = NodeTool.getAttr(node, "accent", "false") === "true";
-
-        if (overText === "→" && isAccent) {
-            return `\\overrightarrow{${result}}`;
-        }
-    }
-    return result;
-}
-
 function renderMunder(node, children){
   const nodes = flattenNodeTreeByNodeName(node, 'munder');
   let result = undefined;

package/lib/mathml2latex.browser.es.js

@@ -145,7 +145,9 @@ const NodeTool = {
     return node.children;
   },
   getNodeName: function(node) {
-    return node.tagName.toLowerCase();
+    if (!node) return '';
+    const localName = node.localName || node.nodeName;
+    return localName ? localName.toLowerCase() : '';
   },
   getNodeText: function(node) {
     return node.textContent;
@@ -702,9 +704,9 @@ function parseLeaf(node) {
 function parseOperator(node) {
   let it = NodeTool.getNodeText(node).trim();
   
-  // Special case for arrow (→)
+  // Special case for arrow (→) with proper spacing
   if (it === "→") {
-    return "\\rightarrow";
+    return " \\rightarrow ";  // Add spaces around arrow
   }
   
   it = MathSymbol.parseOperator(it);
@@ -831,7 +833,28 @@ function getRender(node) {
   const nodeName = NodeTool.getNodeName(node);
   switch(nodeName){
     case 'msub':
-      render = getRender_default("@1_{@2}");
+      render = function(node, children) {
+        if (!children || children.length < 2) return '';
+        
+        const base = parse(children[0]);
+        if (!base) return '';
+        
+        const sub = children[1];
+        if (!sub) return base;
+
+        // Handle nested subscript with empty mrow
+        if (NodeTool.getNodeName(sub) === 'msub' && 
+            sub.firstChild && 
+            NodeTool.getNodeName(sub.firstChild) === 'mrow' &&
+            (!NodeTool.getNodeText(sub.firstChild) || NodeTool.getNodeText(sub.firstChild).trim() === '')) {
+          const lastChild = sub.lastChild;
+          if (!lastChild) return base;
+          return `${base}_${parse(lastChild)}`;
+        }
+        
+        // Regular subscript
+        return `${base}_{${parse(sub)}}`;
+      };
       break;
     case 'msup':
       render = getRender_default("@1^{@2}");
@@ -893,25 +916,55 @@ function getRender(node) {
 
 function renderTable(node, children) {
   const template = "\\begin{array}{l}@content\\end{array}";
-  const render = getRender_joinSeparator(template, "\\\\");
+  // Remove extra backslash and add proper spacing
+  const render = getRender_joinSeparator(template, " \\\\ ");
   return render(node, children);
 }
 
+function renderMover(node, children) {
+    // Check if children exists and has enough elements
+    if (!children || children.length < 2) return '';
+
+    const baseNode = children[0];
+    const overNode = children[1];
+    
+    // Check if nodes exist before accessing
+    if (!baseNode || !overNode) return '';
+    
+    const overText = NodeTool.getNodeText(overNode)?.trim() || '';
+    const isAccent = NodeTool.getAttr(node, "accent", "false") === "true";
+
+    // Handle vector notation
+    if (overText === "→" && isAccent) {
+        return `\\vec{${parse(baseNode)}}`;
+    }
+
+    return parse(baseNode);
+}
+
 function renderMfenced(node, children) {
     const [open, close, separatorsStr] = [
-        NodeTool.getAttr(node, 'open', '{'),
-        NodeTool.getAttr(node, 'close', '}'),
-        NodeTool.getAttr(node, 'separators', '|')
+        NodeTool.getAttr(node, 'open', '('),
+        NodeTool.getAttr(node, 'close', ')'),
+        NodeTool.getAttr(node, 'separators', ',')
     ];
 
     const parts = renderChildren(children);
-    const content = parts.join(separatorsStr).trim();
+    const content = parts.join(separatorsStr === '|' ? ',' : separatorsStr).trim();
     
-    if (open === '{' && close === '') {
-        return `\\left\\${open}${content}\\right.`;
+    // Handle system of equations with curly brace
+    if (open === '{') {
+        // If no closing brace specified, use \right.
+        return `\\left\\{${content}${close ? '\\right\\}' : '\\right.'}`;
     }
     
-    return `\\left\\${open}${content}\\right\\${close}`;
+    // Handle absolute value notation
+    if (open === '|' && (close === '|' || close === '')) {
+        return `\\left|${content}\\right|`;
+    }
+
+    // Regular cases with proper closing
+    return `\\left${open}${content}\\right${close || '.'}`;
 }
 
 function renderMfrac(node, children){
@@ -931,9 +984,9 @@ function renderMfrac(node, children){
     if((prevNode && NodeTool.getNodeText(prevNode).trim() === '(') &&
        (nextNode && NodeTool.getNodeText(nextNode).trim() === ')')
     ) {
-      render = getRender_default("\\DELETE_BRACKET_L\\binom{@1}{@2}\\DELETE_BRACKET_R");
+      render = getRender_default("\\binom{@1}{@2}");
     } else {
-      render = getRender_default("{}_{@2}^{@1}");
+      render = getRender_default("\\frac{@1}{@2}");
     }
   } else {
     render = getRender_default("\\frac{@1}{@2}");
@@ -989,62 +1042,6 @@ function renderMmultiscripts(node, children) {
   return [renderScripts(prevScripts), base, renderScripts(backScripts)].join('');
 }
 
-function renderMover(node, children) {
-    const nodes = flattenNodeTreeByNodeName(node, 'mover');
-    let result = undefined;
-
-    // Get the base node and check if it's a subscript or mrow
-    const baseNode = children[0];
-    const overNode = children[1];
-    const nodeName = NodeTool.getNodeName(baseNode);
-    const isSubscript = nodeName === 'msub';
-    const isMrow = nodeName === 'mrow';
-
-    // Handle case where the base is an arrow and mrow is above
-    const baseText = NodeTool.getNodeText(baseNode).trim();
-    if (baseText === "→" && NodeTool.getNodeName(overNode) === "mrow") {
-        return "\\rightarrow";
-    }
-
-    if (isSubscript) {
-        // Handle case like n₂ with arrow
-        const base = parse(baseNode);
-        return `\\overrightarrow{${base}}`;
-    }
-
-    if (isMrow) {
-        // Handle case like 0 with arrow
-        const base = parse(baseNode);
-        const overText = NodeTool.getNodeText(overNode).trim();
-        const isAccent = NodeTool.getAttr(node, "accent", "false") === "true";
-
-        if (overText === "→" && isAccent) {
-            return `\\overrightarrow{${base}}`;
-        }
-    }
-
-    // Handle case where there is only an arrow with no base
-    const overText = NodeTool.getNodeText(overNode).trim();
-    if (overText === "→" && NodeTool.getNodeText(baseNode).trim() === "") {
-        return `\\rightarrow`; // Return just the arrow
-    }
-
-    for (let i = 0; i < nodes.length - 1; i++) {
-        if (!result) {
-            result = parse(nodes[i]);
-        }
-
-        const _overNode = nodes[i + 1];
-        const overText = NodeTool.getNodeText(_overNode).trim();
-        const isAccent = NodeTool.getAttr(node, "accent", "false") === "true";
-
-        if (overText === "→" && isAccent) {
-            return `\\overrightarrow{${result}}`;
-        }
-    }
-    return result;
-}
-
 function renderMunder(node, children){
   const nodes = flattenNodeTreeByNodeName(node, 'munder');
   let result = undefined;

package/lib/mathml2latex.browser.umd.js

@@ -151,7 +151,9 @@
       return node.children;
     },
     getNodeName: function(node) {
-      return node.tagName.toLowerCase();
+      if (!node) return '';
+      const localName = node.localName || node.nodeName;
+      return localName ? localName.toLowerCase() : '';
     },
     getNodeText: function(node) {
       return node.textContent;
@@ -708,9 +710,9 @@
   function parseOperator(node) {
     let it = NodeTool.getNodeText(node).trim();
     
-    // Special case for arrow (→)
+    // Special case for arrow (→) with proper spacing
     if (it === "→") {
-      return "\\rightarrow";
+      return " \\rightarrow ";  // Add spaces around arrow
     }
     
     it = MathSymbol.parseOperator(it);
@@ -837,7 +839,28 @@
     const nodeName = NodeTool.getNodeName(node);
     switch(nodeName){
       case 'msub':
-        render = getRender_default("@1_{@2}");
+        render = function(node, children) {
+          if (!children || children.length < 2) return '';
+          
+          const base = parse(children[0]);
+          if (!base) return '';
+          
+          const sub = children[1];
+          if (!sub) return base;
+
+          // Handle nested subscript with empty mrow
+          if (NodeTool.getNodeName(sub) === 'msub' && 
+              sub.firstChild && 
+              NodeTool.getNodeName(sub.firstChild) === 'mrow' &&
+              (!NodeTool.getNodeText(sub.firstChild) || NodeTool.getNodeText(sub.firstChild).trim() === '')) {
+            const lastChild = sub.lastChild;
+            if (!lastChild) return base;
+            return `${base}_${parse(lastChild)}`;
+          }
+          
+          // Regular subscript
+          return `${base}_{${parse(sub)}}`;
+        };
         break;
       case 'msup':
         render = getRender_default("@1^{@2}");
@@ -899,25 +922,55 @@
 
   function renderTable(node, children) {
     const template = "\\begin{array}{l}@content\\end{array}";
-    const render = getRender_joinSeparator(template, "\\\\");
+    // Remove extra backslash and add proper spacing
+    const render = getRender_joinSeparator(template, " \\\\ ");
     return render(node, children);
   }
 
+  function renderMover(node, children) {
+      // Check if children exists and has enough elements
+      if (!children || children.length < 2) return '';
+
+      const baseNode = children[0];
+      const overNode = children[1];
+      
+      // Check if nodes exist before accessing
+      if (!baseNode || !overNode) return '';
+      
+      const overText = NodeTool.getNodeText(overNode)?.trim() || '';
+      const isAccent = NodeTool.getAttr(node, "accent", "false") === "true";
+
+      // Handle vector notation
+      if (overText === "→" && isAccent) {
+          return `\\vec{${parse(baseNode)}}`;
+      }
+
+      return parse(baseNode);
+  }
+
   function renderMfenced(node, children) {
       const [open, close, separatorsStr] = [
-          NodeTool.getAttr(node, 'open', '{'),
-          NodeTool.getAttr(node, 'close', '}'),
-          NodeTool.getAttr(node, 'separators', '|')
+          NodeTool.getAttr(node, 'open', '('),
+          NodeTool.getAttr(node, 'close', ')'),
+          NodeTool.getAttr(node, 'separators', ',')
       ];
 
       const parts = renderChildren(children);
-      const content = parts.join(separatorsStr).trim();
+      const content = parts.join(separatorsStr === '|' ? ',' : separatorsStr).trim();
       
-      if (open === '{' && close === '') {
-          return `\\left\\${open}${content}\\right.`;
+      // Handle system of equations with curly brace
+      if (open === '{') {
+          // If no closing brace specified, use \right.
+          return `\\left\\{${content}${close ? '\\right\\}' : '\\right.'}`;
       }
       
-      return `\\left\\${open}${content}\\right\\${close}`;
+      // Handle absolute value notation
+      if (open === '|' && (close === '|' || close === '')) {
+          return `\\left|${content}\\right|`;
+      }
+
+      // Regular cases with proper closing
+      return `\\left${open}${content}\\right${close || '.'}`;
   }
 
   function renderMfrac(node, children){
@@ -937,9 +990,9 @@
       if((prevNode && NodeTool.getNodeText(prevNode).trim() === '(') &&
          (nextNode && NodeTool.getNodeText(nextNode).trim() === ')')
       ) {
-        render = getRender_default("\\DELETE_BRACKET_L\\binom{@1}{@2}\\DELETE_BRACKET_R");
+        render = getRender_default("\\binom{@1}{@2}");
       } else {
-        render = getRender_default("{}_{@2}^{@1}");
+        render = getRender_default("\\frac{@1}{@2}");
       }
     } else {
       render = getRender_default("\\frac{@1}{@2}");
@@ -995,62 +1048,6 @@
     return [renderScripts(prevScripts), base, renderScripts(backScripts)].join('');
   }
 
-  function renderMover(node, children) {
-      const nodes = flattenNodeTreeByNodeName(node, 'mover');
-      let result = undefined;
-
-      // Get the base node and check if it's a subscript or mrow
-      const baseNode = children[0];
-      const overNode = children[1];
-      const nodeName = NodeTool.getNodeName(baseNode);
-      const isSubscript = nodeName === 'msub';
-      const isMrow = nodeName === 'mrow';
-
-      // Handle case where the base is an arrow and mrow is above
-      const baseText = NodeTool.getNodeText(baseNode).trim();
-      if (baseText === "→" && NodeTool.getNodeName(overNode) === "mrow") {
-          return "\\rightarrow";
-      }
-
-      if (isSubscript) {
-          // Handle case like n₂ with arrow
-          const base = parse(baseNode);
-          return `\\overrightarrow{${base}}`;
-      }
-
-      if (isMrow) {
-          // Handle case like 0 with arrow
-          const base = parse(baseNode);
-          const overText = NodeTool.getNodeText(overNode).trim();
-          const isAccent = NodeTool.getAttr(node, "accent", "false") === "true";
-
-          if (overText === "→" && isAccent) {
-              return `\\overrightarrow{${base}}`;
-          }
-      }
-
-      // Handle case where there is only an arrow with no base
-      const overText = NodeTool.getNodeText(overNode).trim();
-      if (overText === "→" && NodeTool.getNodeText(baseNode).trim() === "") {
-          return `\\rightarrow`; // Return just the arrow
-      }
-
-      for (let i = 0; i < nodes.length - 1; i++) {
-          if (!result) {
-              result = parse(nodes[i]);
-          }
-
-          const _overNode = nodes[i + 1];
-          const overText = NodeTool.getNodeText(_overNode).trim();
-          const isAccent = NodeTool.getAttr(node, "accent", "false") === "true";
-
-          if (overText === "→" && isAccent) {
-              return `\\overrightarrow{${result}}`;
-          }
-      }
-      return result;
-  }
-
   function renderMunder(node, children){
     const nodes = flattenNodeTreeByNodeName(node, 'munder');
     let result = undefined;

package/lib/mathml2latex.cjs.js

@@ -147,7 +147,9 @@ const NodeTool = {
     return node.children;
   },
   getNodeName: function(node) {
-    return node.tagName.toLowerCase();
+    if (!node) return '';
+    const localName = node.localName || node.nodeName;
+    return localName ? localName.toLowerCase() : '';
   },
   getNodeText: function(node) {
     return node.textContent;
@@ -704,9 +706,9 @@ function parseLeaf(node) {
 function parseOperator(node) {
   let it = NodeTool.getNodeText(node).trim();
   
-  // Special case for arrow (→)
+  // Special case for arrow (→) with proper spacing
   if (it === "→") {
-    return "\\rightarrow";
+    return " \\rightarrow ";  // Add spaces around arrow
   }
   
   it = MathSymbol.parseOperator(it);
@@ -833,7 +835,28 @@ function getRender(node) {
   const nodeName = NodeTool.getNodeName(node);
   switch(nodeName){
     case 'msub':
-      render = getRender_default("@1_{@2}");
+      render = function(node, children) {
+        if (!children || children.length < 2) return '';
+        
+        const base = parse(children[0]);
+        if (!base) return '';
+        
+        const sub = children[1];
+        if (!sub) return base;
+
+        // Handle nested subscript with empty mrow
+        if (NodeTool.getNodeName(sub) === 'msub' && 
+            sub.firstChild && 
+            NodeTool.getNodeName(sub.firstChild) === 'mrow' &&
+            (!NodeTool.getNodeText(sub.firstChild) || NodeTool.getNodeText(sub.firstChild).trim() === '')) {
+          const lastChild = sub.lastChild;
+          if (!lastChild) return base;
+          return `${base}_${parse(lastChild)}`;
+        }
+        
+        // Regular subscript
+        return `${base}_{${parse(sub)}}`;
+      };
       break;
     case 'msup':
       render = getRender_default("@1^{@2}");
@@ -895,25 +918,55 @@ function getRender(node) {
 
 function renderTable(node, children) {
   const template = "\\begin{array}{l}@content\\end{array}";
-  const render = getRender_joinSeparator(template, "\\\\");
+  // Remove extra backslash and add proper spacing
+  const render = getRender_joinSeparator(template, " \\\\ ");
   return render(node, children);
 }
 
+function renderMover(node, children) {
+    // Check if children exists and has enough elements
+    if (!children || children.length < 2) return '';
+
+    const baseNode = children[0];
+    const overNode = children[1];
+    
+    // Check if nodes exist before accessing
+    if (!baseNode || !overNode) return '';
+    
+    const overText = NodeTool.getNodeText(overNode)?.trim() || '';
+    const isAccent = NodeTool.getAttr(node, "accent", "false") === "true";
+
+    // Handle vector notation
+    if (overText === "→" && isAccent) {
+        return `\\vec{${parse(baseNode)}}`;
+    }
+
+    return parse(baseNode);
+}
+
 function renderMfenced(node, children) {
     const [open, close, separatorsStr] = [
-        NodeTool.getAttr(node, 'open', '{'),
-        NodeTool.getAttr(node, 'close', '}'),
-        NodeTool.getAttr(node, 'separators', '|')
+        NodeTool.getAttr(node, 'open', '('),
+        NodeTool.getAttr(node, 'close', ')'),
+        NodeTool.getAttr(node, 'separators', ',')
     ];
 
     const parts = renderChildren(children);
-    const content = parts.join(separatorsStr).trim();
+    const content = parts.join(separatorsStr === '|' ? ',' : separatorsStr).trim();
     
-    if (open === '{' && close === '') {
-        return `\\left\\${open}${content}\\right.`;
+    // Handle system of equations with curly brace
+    if (open === '{') {
+        // If no closing brace specified, use \right.
+        return `\\left\\{${content}${close ? '\\right\\}' : '\\right.'}`;
     }
     
-    return `\\left\\${open}${content}\\right\\${close}`;
+    // Handle absolute value notation
+    if (open === '|' && (close === '|' || close === '')) {
+        return `\\left|${content}\\right|`;
+    }
+
+    // Regular cases with proper closing
+    return `\\left${open}${content}\\right${close || '.'}`;
 }
 
 function renderMfrac(node, children){
@@ -933,9 +986,9 @@ function renderMfrac(node, children){
     if((prevNode && NodeTool.getNodeText(prevNode).trim() === '(') &&
        (nextNode && NodeTool.getNodeText(nextNode).trim() === ')')
     ) {
-      render = getRender_default("\\DELETE_BRACKET_L\\binom{@1}{@2}\\DELETE_BRACKET_R");
+      render = getRender_default("\\binom{@1}{@2}");
     } else {
-      render = getRender_default("{}_{@2}^{@1}");
+      render = getRender_default("\\frac{@1}{@2}");
     }
   } else {
     render = getRender_default("\\frac{@1}{@2}");
@@ -991,62 +1044,6 @@ function renderMmultiscripts(node, children) {
   return [renderScripts(prevScripts), base, renderScripts(backScripts)].join('');
 }
 
-function renderMover(node, children) {
-    const nodes = flattenNodeTreeByNodeName(node, 'mover');
-    let result = undefined;
-
-    // Get the base node and check if it's a subscript or mrow
-    const baseNode = children[0];
-    const overNode = children[1];
-    const nodeName = NodeTool.getNodeName(baseNode);
-    const isSubscript = nodeName === 'msub';
-    const isMrow = nodeName === 'mrow';
-
-    // Handle case where the base is an arrow and mrow is above
-    const baseText = NodeTool.getNodeText(baseNode).trim();
-    if (baseText === "→" && NodeTool.getNodeName(overNode) === "mrow") {
-        return "\\rightarrow";
-    }
-
-    if (isSubscript) {
-        // Handle case like n₂ with arrow
-        const base = parse(baseNode);
-        return `\\overrightarrow{${base}}`;
-    }
-
-    if (isMrow) {
-        // Handle case like 0 with arrow
-        const base = parse(baseNode);
-        const overText = NodeTool.getNodeText(overNode).trim();
-        const isAccent = NodeTool.getAttr(node, "accent", "false") === "true";
-
-        if (overText === "→" && isAccent) {
-            return `\\overrightarrow{${base}}`;
-        }
-    }
-
-    // Handle case where there is only an arrow with no base
-    const overText = NodeTool.getNodeText(overNode).trim();
-    if (overText === "→" && NodeTool.getNodeText(baseNode).trim() === "") {
-        return `\\rightarrow`; // Return just the arrow
-    }
-
-    for (let i = 0; i < nodes.length - 1; i++) {
-        if (!result) {
-            result = parse(nodes[i]);
-        }
-
-        const _overNode = nodes[i + 1];
-        const overText = NodeTool.getNodeText(_overNode).trim();
-        const isAccent = NodeTool.getAttr(node, "accent", "false") === "true";
-
-        if (overText === "→" && isAccent) {
-            return `\\overrightarrow{${result}}`;
-        }
-    }
-    return result;
-}
-
 function renderMunder(node, children){
   const nodes = flattenNodeTreeByNodeName(node, 'munder');
   let result = undefined;

package/lib/mathml2latex.es.js

@@ -145,7 +145,9 @@ const NodeTool = {
     return node.children;
   },
   getNodeName: function(node) {
-    return node.tagName.toLowerCase();
+    if (!node) return '';
+    const localName = node.localName || node.nodeName;
+    return localName ? localName.toLowerCase() : '';
   },
   getNodeText: function(node) {
     return node.textContent;
@@ -702,9 +704,9 @@ function parseLeaf(node) {
 function parseOperator(node) {
   let it = NodeTool.getNodeText(node).trim();
   
-  // Special case for arrow (→)
+  // Special case for arrow (→) with proper spacing
   if (it === "→") {
-    return "\\rightarrow";
+    return " \\rightarrow ";  // Add spaces around arrow
   }
   
   it = MathSymbol.parseOperator(it);
@@ -831,7 +833,28 @@ function getRender(node) {
   const nodeName = NodeTool.getNodeName(node);
   switch(nodeName){
     case 'msub':
-      render = getRender_default("@1_{@2}");
+      render = function(node, children) {
+        if (!children || children.length < 2) return '';
+        
+        const base = parse(children[0]);
+        if (!base) return '';
+        
+        const sub = children[1];
+        if (!sub) return base;
+
+        // Handle nested subscript with empty mrow
+        if (NodeTool.getNodeName(sub) === 'msub' && 
+            sub.firstChild && 
+            NodeTool.getNodeName(sub.firstChild) === 'mrow' &&
+            (!NodeTool.getNodeText(sub.firstChild) || NodeTool.getNodeText(sub.firstChild).trim() === '')) {
+          const lastChild = sub.lastChild;
+          if (!lastChild) return base;
+          return `${base}_${parse(lastChild)}`;
+        }
+        
+        // Regular subscript
+        return `${base}_{${parse(sub)}}`;
+      };
       break;
     case 'msup':
       render = getRender_default("@1^{@2}");
@@ -893,25 +916,55 @@ function getRender(node) {
 
 function renderTable(node, children) {
   const template = "\\begin{array}{l}@content\\end{array}";
-  const render = getRender_joinSeparator(template, "\\\\");
+  // Remove extra backslash and add proper spacing
+  const render = getRender_joinSeparator(template, " \\\\ ");
   return render(node, children);
 }
 
+function renderMover(node, children) {
+    // Check if children exists and has enough elements
+    if (!children || children.length < 2) return '';
+
+    const baseNode = children[0];
+    const overNode = children[1];
+    
+    // Check if nodes exist before accessing
+    if (!baseNode || !overNode) return '';
+    
+    const overText = NodeTool.getNodeText(overNode)?.trim() || '';
+    const isAccent = NodeTool.getAttr(node, "accent", "false") === "true";
+
+    // Handle vector notation
+    if (overText === "→" && isAccent) {
+        return `\\vec{${parse(baseNode)}}`;
+    }
+
+    return parse(baseNode);
+}
+
 function renderMfenced(node, children) {
     const [open, close, separatorsStr] = [
-        NodeTool.getAttr(node, 'open', '{'),
-        NodeTool.getAttr(node, 'close', '}'),
-        NodeTool.getAttr(node, 'separators', '|')
+        NodeTool.getAttr(node, 'open', '('),
+        NodeTool.getAttr(node, 'close', ')'),
+        NodeTool.getAttr(node, 'separators', ',')
     ];
 
     const parts = renderChildren(children);
-    const content = parts.join(separatorsStr).trim();
+    const content = parts.join(separatorsStr === '|' ? ',' : separatorsStr).trim();
     
-    if (open === '{' && close === '') {
-        return `\\left\\${open}${content}\\right.`;
+    // Handle system of equations with curly brace
+    if (open === '{') {
+        // If no closing brace specified, use \right.
+        return `\\left\\{${content}${close ? '\\right\\}' : '\\right.'}`;
     }
     
-    return `\\left\\${open}${content}\\right\\${close}`;
+    // Handle absolute value notation
+    if (open === '|' && (close === '|' || close === '')) {
+        return `\\left|${content}\\right|`;
+    }
+
+    // Regular cases with proper closing
+    return `\\left${open}${content}\\right${close || '.'}`;
 }
 
 function renderMfrac(node, children){
@@ -931,9 +984,9 @@ function renderMfrac(node, children){
     if((prevNode && NodeTool.getNodeText(prevNode).trim() === '(') &&
        (nextNode && NodeTool.getNodeText(nextNode).trim() === ')')
     ) {
-      render = getRender_default("\\DELETE_BRACKET_L\\binom{@1}{@2}\\DELETE_BRACKET_R");
+      render = getRender_default("\\binom{@1}{@2}");
     } else {
-      render = getRender_default("{}_{@2}^{@1}");
+      render = getRender_default("\\frac{@1}{@2}");
     }
   } else {
     render = getRender_default("\\frac{@1}{@2}");
@@ -989,62 +1042,6 @@ function renderMmultiscripts(node, children) {
   return [renderScripts(prevScripts), base, renderScripts(backScripts)].join('');
 }
 
-function renderMover(node, children) {
-    const nodes = flattenNodeTreeByNodeName(node, 'mover');
-    let result = undefined;
-
-    // Get the base node and check if it's a subscript or mrow
-    const baseNode = children[0];
-    const overNode = children[1];
-    const nodeName = NodeTool.getNodeName(baseNode);
-    const isSubscript = nodeName === 'msub';
-    const isMrow = nodeName === 'mrow';
-
-    // Handle case where the base is an arrow and mrow is above
-    const baseText = NodeTool.getNodeText(baseNode).trim();
-    if (baseText === "→" && NodeTool.getNodeName(overNode) === "mrow") {
-        return "\\rightarrow";
-    }
-
-    if (isSubscript) {
-        // Handle case like n₂ with arrow
-        const base = parse(baseNode);
-        return `\\overrightarrow{${base}}`;
-    }
-
-    if (isMrow) {
-        // Handle case like 0 with arrow
-        const base = parse(baseNode);
-        const overText = NodeTool.getNodeText(overNode).trim();
-        const isAccent = NodeTool.getAttr(node, "accent", "false") === "true";
-
-        if (overText === "→" && isAccent) {
-            return `\\overrightarrow{${base}}`;
-        }
-    }
-
-    // Handle case where there is only an arrow with no base
-    const overText = NodeTool.getNodeText(overNode).trim();
-    if (overText === "→" && NodeTool.getNodeText(baseNode).trim() === "") {
-        return `\\rightarrow`; // Return just the arrow
-    }
-
-    for (let i = 0; i < nodes.length - 1; i++) {
-        if (!result) {
-            result = parse(nodes[i]);
-        }
-
-        const _overNode = nodes[i + 1];
-        const overText = NodeTool.getNodeText(_overNode).trim();
-        const isAccent = NodeTool.getAttr(node, "accent", "false") === "true";
-
-        if (overText === "→" && isAccent) {
-            return `\\overrightarrow{${result}}`;
-        }
-    }
-    return result;
-}
-
 function renderMunder(node, children){
   const nodes = flattenNodeTreeByNodeName(node, 'munder');
   let result = undefined;

package/lib/mathml2latex.umd.js

@@ -151,7 +151,9 @@
       return node.children;
     },
     getNodeName: function(node) {
-      return node.tagName.toLowerCase();
+      if (!node) return '';
+      const localName = node.localName || node.nodeName;
+      return localName ? localName.toLowerCase() : '';
     },
     getNodeText: function(node) {
       return node.textContent;
@@ -708,9 +710,9 @@
   function parseOperator(node) {
     let it = NodeTool.getNodeText(node).trim();
     
-    // Special case for arrow (→)
+    // Special case for arrow (→) with proper spacing
     if (it === "→") {
-      return "\\rightarrow";
+      return " \\rightarrow ";  // Add spaces around arrow
     }
     
     it = MathSymbol.parseOperator(it);
@@ -837,7 +839,28 @@
     const nodeName = NodeTool.getNodeName(node);
     switch(nodeName){
       case 'msub':
-        render = getRender_default("@1_{@2}");
+        render = function(node, children) {
+          if (!children || children.length < 2) return '';
+          
+          const base = parse(children[0]);
+          if (!base) return '';
+          
+          const sub = children[1];
+          if (!sub) return base;
+
+          // Handle nested subscript with empty mrow
+          if (NodeTool.getNodeName(sub) === 'msub' && 
+              sub.firstChild && 
+              NodeTool.getNodeName(sub.firstChild) === 'mrow' &&
+              (!NodeTool.getNodeText(sub.firstChild) || NodeTool.getNodeText(sub.firstChild).trim() === '')) {
+            const lastChild = sub.lastChild;
+            if (!lastChild) return base;
+            return `${base}_${parse(lastChild)}`;
+          }
+          
+          // Regular subscript
+          return `${base}_{${parse(sub)}}`;
+        };
         break;
       case 'msup':
         render = getRender_default("@1^{@2}");
@@ -899,25 +922,55 @@
 
   function renderTable(node, children) {
     const template = "\\begin{array}{l}@content\\end{array}";
-    const render = getRender_joinSeparator(template, "\\\\");
+    // Remove extra backslash and add proper spacing
+    const render = getRender_joinSeparator(template, " \\\\ ");
     return render(node, children);
   }
 
+  function renderMover(node, children) {
+      // Check if children exists and has enough elements
+      if (!children || children.length < 2) return '';
+
+      const baseNode = children[0];
+      const overNode = children[1];
+      
+      // Check if nodes exist before accessing
+      if (!baseNode || !overNode) return '';
+      
+      const overText = NodeTool.getNodeText(overNode)?.trim() || '';
+      const isAccent = NodeTool.getAttr(node, "accent", "false") === "true";
+
+      // Handle vector notation
+      if (overText === "→" && isAccent) {
+          return `\\vec{${parse(baseNode)}}`;
+      }
+
+      return parse(baseNode);
+  }
+
   function renderMfenced(node, children) {
       const [open, close, separatorsStr] = [
-          NodeTool.getAttr(node, 'open', '{'),
-          NodeTool.getAttr(node, 'close', '}'),
-          NodeTool.getAttr(node, 'separators', '|')
+          NodeTool.getAttr(node, 'open', '('),
+          NodeTool.getAttr(node, 'close', ')'),
+          NodeTool.getAttr(node, 'separators', ',')
       ];
 
       const parts = renderChildren(children);
-      const content = parts.join(separatorsStr).trim();
+      const content = parts.join(separatorsStr === '|' ? ',' : separatorsStr).trim();
       
-      if (open === '{' && close === '') {
-          return `\\left\\${open}${content}\\right.`;
+      // Handle system of equations with curly brace
+      if (open === '{') {
+          // If no closing brace specified, use \right.
+          return `\\left\\{${content}${close ? '\\right\\}' : '\\right.'}`;
       }
       
-      return `\\left\\${open}${content}\\right\\${close}`;
+      // Handle absolute value notation
+      if (open === '|' && (close === '|' || close === '')) {
+          return `\\left|${content}\\right|`;
+      }
+
+      // Regular cases with proper closing
+      return `\\left${open}${content}\\right${close || '.'}`;
   }
 
   function renderMfrac(node, children){
@@ -937,9 +990,9 @@
       if((prevNode && NodeTool.getNodeText(prevNode).trim() === '(') &&
          (nextNode && NodeTool.getNodeText(nextNode).trim() === ')')
       ) {
-        render = getRender_default("\\DELETE_BRACKET_L\\binom{@1}{@2}\\DELETE_BRACKET_R");
+        render = getRender_default("\\binom{@1}{@2}");
       } else {
-        render = getRender_default("{}_{@2}^{@1}");
+        render = getRender_default("\\frac{@1}{@2}");
       }
     } else {
       render = getRender_default("\\frac{@1}{@2}");
@@ -995,62 +1048,6 @@
     return [renderScripts(prevScripts), base, renderScripts(backScripts)].join('');
   }
 
-  function renderMover(node, children) {
-      const nodes = flattenNodeTreeByNodeName(node, 'mover');
-      let result = undefined;
-
-      // Get the base node and check if it's a subscript or mrow
-      const baseNode = children[0];
-      const overNode = children[1];
-      const nodeName = NodeTool.getNodeName(baseNode);
-      const isSubscript = nodeName === 'msub';
-      const isMrow = nodeName === 'mrow';
-
-      // Handle case where the base is an arrow and mrow is above
-      const baseText = NodeTool.getNodeText(baseNode).trim();
-      if (baseText === "→" && NodeTool.getNodeName(overNode) === "mrow") {
-          return "\\rightarrow";
-      }
-
-      if (isSubscript) {
-          // Handle case like n₂ with arrow
-          const base = parse(baseNode);
-          return `\\overrightarrow{${base}}`;
-      }
-
-      if (isMrow) {
-          // Handle case like 0 with arrow
-          const base = parse(baseNode);
-          const overText = NodeTool.getNodeText(overNode).trim();
-          const isAccent = NodeTool.getAttr(node, "accent", "false") === "true";
-
-          if (overText === "→" && isAccent) {
-              return `\\overrightarrow{${base}}`;
-          }
-      }
-
-      // Handle case where there is only an arrow with no base
-      const overText = NodeTool.getNodeText(overNode).trim();
-      if (overText === "→" && NodeTool.getNodeText(baseNode).trim() === "") {
-          return `\\rightarrow`; // Return just the arrow
-      }
-
-      for (let i = 0; i < nodes.length - 1; i++) {
-          if (!result) {
-              result = parse(nodes[i]);
-          }
-
-          const _overNode = nodes[i + 1];
-          const overText = NodeTool.getNodeText(_overNode).trim();
-          const isAccent = NodeTool.getAttr(node, "accent", "false") === "true";
-
-          if (overText === "→" && isAccent) {
-              return `\\overrightarrow{${result}}`;
-          }
-      }
-      return result;
-  }
-
   function renderMunder(node, children){
     const nodes = flattenNodeTreeByNodeName(node, 'munder');
     let result = undefined;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ed-mathml2tex",
-  "version": "0.0.7",
+  "version": "0.0.9",
   "description": "Convert mathml to latex.",
   "author": "Mika",
   "license": "MIT",