ed-mathml2tex 0.0.8 → 0.1.0

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);
@@ -789,6 +791,7 @@ function renderChildren(children) {
       if(Brackets.contains(op)){
         let stretchy = NodeTool.getAttr(node, 'stretchy', 'true');
         stretchy = ['', 'true'].indexOf(stretchy) > -1;
+        
         if(Brackets.isRight(op)){
           const nearLeft = lefts[lefts.length - 1];
           if(nearLeft){
@@ -796,16 +799,10 @@ function renderChildren(children) {
               parts.push(Brackets.parseRight(op, stretchy));
               lefts.pop();
             } else {
-              if(Brackets.isLeft(op)) {
-                parts.push(Brackets.parseLeft(op, stretchy));
-                lefts.push(op);
-              }
-            }
-          }else {
-            if(Brackets.isLeft(op)) {
-              parts.push(Brackets.parseLeft(op, stretchy));
-              lefts.push(op);
+              parts.push(`\\right${op}`);
             }
+          } else {
+            parts.push(`\\right${op}`);
           }
         } else {
           parts.push(Brackets.parseLeft(op, stretchy));
@@ -818,10 +815,9 @@ function renderChildren(children) {
       parts.push(parse(node));
     }
   });
-  if(lefts.length > 0){
-    for(let i=0; i < lefts.length; i++){
-      parts.push("\\right.");
-    }
+  // Only add \right. if there are unmatched left brackets
+  if(lefts.length > 0 && !parts.some(p => p.includes('\\right'))) {
+    parts.push("\\right.");
   }
   lefts = undefined;
   return parts;
@@ -833,7 +829,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,10 +912,32 @@ 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', '('),
@@ -909,22 +948,25 @@ function renderMfenced(node, children) {
     const parts = renderChildren(children);
     const content = parts.join(separatorsStr === '|' ? ',' : separatorsStr).trim();
     
-    // Handle empty open/close brackets
+    // Handle nested absolute value cases
+    if (open === '|' && close === '') {
+        return `\\left|${content}`;
+    }
     if (open === '' && close === '|') {
-        return `|${content}|`;
+        return `${content}\\right|`;
     }
     
-    // Handle absolute value notation
-    if (open === '|' && (close === '|' || close === '')) {
-        return `\\left|${content}\\right|`;
+    // Handle system of equations with curly brace
+    if (open === '{') {
+        return `\\left\\{${content}${close ? '\\right\\}' : '\\right.'}`;
     }
-
-    // Regular parentheses
-    if (open === '(' && close === ')') {
-        return `\\left(${content}\\right)`;
+    
+    // Handle regular absolute value notation
+    if (open === '|' && close === '|') {
+        return `\\left|${content}\\right|`;
     }
 
-    // Default case
+    // Regular cases with proper closing
     return `\\left${open}${content}\\right${close || '.'}`;
 }
 
@@ -1003,62 +1045,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);
@@ -787,6 +789,7 @@ function renderChildren(children) {
       if(Brackets.contains(op)){
         let stretchy = NodeTool.getAttr(node, 'stretchy', 'true');
         stretchy = ['', 'true'].indexOf(stretchy) > -1;
+        
         if(Brackets.isRight(op)){
           const nearLeft = lefts[lefts.length - 1];
           if(nearLeft){
@@ -794,16 +797,10 @@ function renderChildren(children) {
               parts.push(Brackets.parseRight(op, stretchy));
               lefts.pop();
             } else {
-              if(Brackets.isLeft(op)) {
-                parts.push(Brackets.parseLeft(op, stretchy));
-                lefts.push(op);
-              }
-            }
-          }else {
-            if(Brackets.isLeft(op)) {
-              parts.push(Brackets.parseLeft(op, stretchy));
-              lefts.push(op);
+              parts.push(`\\right${op}`);
             }
+          } else {
+            parts.push(`\\right${op}`);
           }
         } else {
           parts.push(Brackets.parseLeft(op, stretchy));
@@ -816,10 +813,9 @@ function renderChildren(children) {
       parts.push(parse(node));
     }
   });
-  if(lefts.length > 0){
-    for(let i=0; i < lefts.length; i++){
-      parts.push("\\right.");
-    }
+  // Only add \right. if there are unmatched left brackets
+  if(lefts.length > 0 && !parts.some(p => p.includes('\\right'))) {
+    parts.push("\\right.");
   }
   lefts = undefined;
   return parts;
@@ -831,7 +827,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,10 +910,32 @@ 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', '('),
@@ -907,22 +946,25 @@ function renderMfenced(node, children) {
     const parts = renderChildren(children);
     const content = parts.join(separatorsStr === '|' ? ',' : separatorsStr).trim();
     
-    // Handle empty open/close brackets
+    // Handle nested absolute value cases
+    if (open === '|' && close === '') {
+        return `\\left|${content}`;
+    }
     if (open === '' && close === '|') {
-        return `|${content}|`;
+        return `${content}\\right|`;
     }
     
-    // Handle absolute value notation
-    if (open === '|' && (close === '|' || close === '')) {
-        return `\\left|${content}\\right|`;
+    // Handle system of equations with curly brace
+    if (open === '{') {
+        return `\\left\\{${content}${close ? '\\right\\}' : '\\right.'}`;
     }
-
-    // Regular parentheses
-    if (open === '(' && close === ')') {
-        return `\\left(${content}\\right)`;
+    
+    // Handle regular absolute value notation
+    if (open === '|' && close === '|') {
+        return `\\left|${content}\\right|`;
     }
 
-    // Default case
+    // Regular cases with proper closing
     return `\\left${open}${content}\\right${close || '.'}`;
 }
 
@@ -1001,62 +1043,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);
@@ -793,6 +795,7 @@
         if(Brackets.contains(op)){
           let stretchy = NodeTool.getAttr(node, 'stretchy', 'true');
           stretchy = ['', 'true'].indexOf(stretchy) > -1;
+          
           if(Brackets.isRight(op)){
             const nearLeft = lefts[lefts.length - 1];
             if(nearLeft){
@@ -800,16 +803,10 @@
                 parts.push(Brackets.parseRight(op, stretchy));
                 lefts.pop();
               } else {
-                if(Brackets.isLeft(op)) {
-                  parts.push(Brackets.parseLeft(op, stretchy));
-                  lefts.push(op);
-                }
-              }
-            }else {
-              if(Brackets.isLeft(op)) {
-                parts.push(Brackets.parseLeft(op, stretchy));
-                lefts.push(op);
+                parts.push(`\\right${op}`);
               }
+            } else {
+              parts.push(`\\right${op}`);
             }
           } else {
             parts.push(Brackets.parseLeft(op, stretchy));
@@ -822,10 +819,9 @@
         parts.push(parse(node));
       }
     });
-    if(lefts.length > 0){
-      for(let i=0; i < lefts.length; i++){
-        parts.push("\\right.");
-      }
+    // Only add \right. if there are unmatched left brackets
+    if(lefts.length > 0 && !parts.some(p => p.includes('\\right'))) {
+      parts.push("\\right.");
     }
     lefts = undefined;
     return parts;
@@ -837,7 +833,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,10 +916,32 @@
 
   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', '('),
@@ -913,22 +952,25 @@
       const parts = renderChildren(children);
       const content = parts.join(separatorsStr === '|' ? ',' : separatorsStr).trim();
       
-      // Handle empty open/close brackets
+      // Handle nested absolute value cases
+      if (open === '|' && close === '') {
+          return `\\left|${content}`;
+      }
       if (open === '' && close === '|') {
-          return `|${content}|`;
+          return `${content}\\right|`;
       }
       
-      // Handle absolute value notation
-      if (open === '|' && (close === '|' || close === '')) {
-          return `\\left|${content}\\right|`;
+      // Handle system of equations with curly brace
+      if (open === '{') {
+          return `\\left\\{${content}${close ? '\\right\\}' : '\\right.'}`;
       }
-
-      // Regular parentheses
-      if (open === '(' && close === ')') {
-          return `\\left(${content}\\right)`;
+      
+      // Handle regular absolute value notation
+      if (open === '|' && close === '|') {
+          return `\\left|${content}\\right|`;
       }
 
-      // Default case
+      // Regular cases with proper closing
       return `\\left${open}${content}\\right${close || '.'}`;
   }
 
@@ -1007,62 +1049,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);
@@ -789,6 +791,7 @@ function renderChildren(children) {
       if(Brackets.contains(op)){
         let stretchy = NodeTool.getAttr(node, 'stretchy', 'true');
         stretchy = ['', 'true'].indexOf(stretchy) > -1;
+        
         if(Brackets.isRight(op)){
           const nearLeft = lefts[lefts.length - 1];
           if(nearLeft){
@@ -796,16 +799,10 @@ function renderChildren(children) {
               parts.push(Brackets.parseRight(op, stretchy));
               lefts.pop();
             } else {
-              if(Brackets.isLeft(op)) {
-                parts.push(Brackets.parseLeft(op, stretchy));
-                lefts.push(op);
-              }
-            }
-          }else {
-            if(Brackets.isLeft(op)) {
-              parts.push(Brackets.parseLeft(op, stretchy));
-              lefts.push(op);
+              parts.push(`\\right${op}`);
             }
+          } else {
+            parts.push(`\\right${op}`);
           }
         } else {
           parts.push(Brackets.parseLeft(op, stretchy));
@@ -818,10 +815,9 @@ function renderChildren(children) {
       parts.push(parse(node));
     }
   });
-  if(lefts.length > 0){
-    for(let i=0; i < lefts.length; i++){
-      parts.push("\\right.");
-    }
+  // Only add \right. if there are unmatched left brackets
+  if(lefts.length > 0 && !parts.some(p => p.includes('\\right'))) {
+    parts.push("\\right.");
   }
   lefts = undefined;
   return parts;
@@ -833,7 +829,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,10 +912,32 @@ 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', '('),
@@ -909,22 +948,25 @@ function renderMfenced(node, children) {
     const parts = renderChildren(children);
     const content = parts.join(separatorsStr === '|' ? ',' : separatorsStr).trim();
     
-    // Handle empty open/close brackets
+    // Handle nested absolute value cases
+    if (open === '|' && close === '') {
+        return `\\left|${content}`;
+    }
     if (open === '' && close === '|') {
-        return `|${content}|`;
+        return `${content}\\right|`;
     }
     
-    // Handle absolute value notation
-    if (open === '|' && (close === '|' || close === '')) {
-        return `\\left|${content}\\right|`;
+    // Handle system of equations with curly brace
+    if (open === '{') {
+        return `\\left\\{${content}${close ? '\\right\\}' : '\\right.'}`;
     }
-
-    // Regular parentheses
-    if (open === '(' && close === ')') {
-        return `\\left(${content}\\right)`;
+    
+    // Handle regular absolute value notation
+    if (open === '|' && close === '|') {
+        return `\\left|${content}\\right|`;
     }
 
-    // Default case
+    // Regular cases with proper closing
     return `\\left${open}${content}\\right${close || '.'}`;
 }
 
@@ -1003,62 +1045,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);
@@ -787,6 +789,7 @@ function renderChildren(children) {
       if(Brackets.contains(op)){
         let stretchy = NodeTool.getAttr(node, 'stretchy', 'true');
         stretchy = ['', 'true'].indexOf(stretchy) > -1;
+        
         if(Brackets.isRight(op)){
           const nearLeft = lefts[lefts.length - 1];
           if(nearLeft){
@@ -794,16 +797,10 @@ function renderChildren(children) {
               parts.push(Brackets.parseRight(op, stretchy));
               lefts.pop();
             } else {
-              if(Brackets.isLeft(op)) {
-                parts.push(Brackets.parseLeft(op, stretchy));
-                lefts.push(op);
-              }
-            }
-          }else {
-            if(Brackets.isLeft(op)) {
-              parts.push(Brackets.parseLeft(op, stretchy));
-              lefts.push(op);
+              parts.push(`\\right${op}`);
             }
+          } else {
+            parts.push(`\\right${op}`);
           }
         } else {
           parts.push(Brackets.parseLeft(op, stretchy));
@@ -816,10 +813,9 @@ function renderChildren(children) {
       parts.push(parse(node));
     }
   });
-  if(lefts.length > 0){
-    for(let i=0; i < lefts.length; i++){
-      parts.push("\\right.");
-    }
+  // Only add \right. if there are unmatched left brackets
+  if(lefts.length > 0 && !parts.some(p => p.includes('\\right'))) {
+    parts.push("\\right.");
   }
   lefts = undefined;
   return parts;
@@ -831,7 +827,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,10 +910,32 @@ 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', '('),
@@ -907,22 +946,25 @@ function renderMfenced(node, children) {
     const parts = renderChildren(children);
     const content = parts.join(separatorsStr === '|' ? ',' : separatorsStr).trim();
     
-    // Handle empty open/close brackets
+    // Handle nested absolute value cases
+    if (open === '|' && close === '') {
+        return `\\left|${content}`;
+    }
     if (open === '' && close === '|') {
-        return `|${content}|`;
+        return `${content}\\right|`;
     }
     
-    // Handle absolute value notation
-    if (open === '|' && (close === '|' || close === '')) {
-        return `\\left|${content}\\right|`;
+    // Handle system of equations with curly brace
+    if (open === '{') {
+        return `\\left\\{${content}${close ? '\\right\\}' : '\\right.'}`;
     }
-
-    // Regular parentheses
-    if (open === '(' && close === ')') {
-        return `\\left(${content}\\right)`;
+    
+    // Handle regular absolute value notation
+    if (open === '|' && close === '|') {
+        return `\\left|${content}\\right|`;
     }
 
-    // Default case
+    // Regular cases with proper closing
     return `\\left${open}${content}\\right${close || '.'}`;
 }
 
@@ -1001,62 +1043,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);
@@ -793,6 +795,7 @@
         if(Brackets.contains(op)){
           let stretchy = NodeTool.getAttr(node, 'stretchy', 'true');
           stretchy = ['', 'true'].indexOf(stretchy) > -1;
+          
           if(Brackets.isRight(op)){
             const nearLeft = lefts[lefts.length - 1];
             if(nearLeft){
@@ -800,16 +803,10 @@
                 parts.push(Brackets.parseRight(op, stretchy));
                 lefts.pop();
               } else {
-                if(Brackets.isLeft(op)) {
-                  parts.push(Brackets.parseLeft(op, stretchy));
-                  lefts.push(op);
-                }
-              }
-            }else {
-              if(Brackets.isLeft(op)) {
-                parts.push(Brackets.parseLeft(op, stretchy));
-                lefts.push(op);
+                parts.push(`\\right${op}`);
               }
+            } else {
+              parts.push(`\\right${op}`);
             }
           } else {
             parts.push(Brackets.parseLeft(op, stretchy));
@@ -822,10 +819,9 @@
         parts.push(parse(node));
       }
     });
-    if(lefts.length > 0){
-      for(let i=0; i < lefts.length; i++){
-        parts.push("\\right.");
-      }
+    // Only add \right. if there are unmatched left brackets
+    if(lefts.length > 0 && !parts.some(p => p.includes('\\right'))) {
+      parts.push("\\right.");
     }
     lefts = undefined;
     return parts;
@@ -837,7 +833,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,10 +916,32 @@
 
   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', '('),
@@ -913,22 +952,25 @@
       const parts = renderChildren(children);
       const content = parts.join(separatorsStr === '|' ? ',' : separatorsStr).trim();
       
-      // Handle empty open/close brackets
+      // Handle nested absolute value cases
+      if (open === '|' && close === '') {
+          return `\\left|${content}`;
+      }
       if (open === '' && close === '|') {
-          return `|${content}|`;
+          return `${content}\\right|`;
       }
       
-      // Handle absolute value notation
-      if (open === '|' && (close === '|' || close === '')) {
-          return `\\left|${content}\\right|`;
+      // Handle system of equations with curly brace
+      if (open === '{') {
+          return `\\left\\{${content}${close ? '\\right\\}' : '\\right.'}`;
       }
-
-      // Regular parentheses
-      if (open === '(' && close === ')') {
-          return `\\left(${content}\\right)`;
+      
+      // Handle regular absolute value notation
+      if (open === '|' && close === '|') {
+          return `\\left|${content}\\right|`;
       }
 
-      // Default case
+      // Regular cases with proper closing
       return `\\left${open}${content}\\right${close || '.'}`;
   }
 
@@ -1007,62 +1049,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.8",
+  "version": "0.1.0",
   "description": "Convert mathml to latex.",
   "author": "Mika",
   "license": "MIT",