Update svg output to properly handle token elements with multiple chi…

…ld nodes. (mathjax/MathJax#2836)
mathjax · Apr 20, 2022 · 503a28b · 503a28b
1 parent d671f3e
commit 503a28b
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 3 deletions.
diff --git a/ts/output/chtml/Wrappers/TextNode.ts b/ts/output/chtml/Wrappers/TextNode.ts
@@ -69,6 +69,7 @@ CommonTextNodeMixin<CHTMLConstructor<any, any, any>>(CHTMLWrapper) {
     const adaptor = this.adaptor;
     const variant = this.parent.variant;
     const text = (this.node as TextNode).getText();
+    if (text.length === 0) return;
     if (variant === '-explicitFont') {
       adaptor.append(parent, this.jax.unknownText(text, variant, this.getBBox().w));
     } else {

diff --git a/ts/output/svg/Wrappers/TextNode.ts b/ts/output/svg/Wrappers/TextNode.ts
@@ -58,16 +58,19 @@ CommonTextNodeMixin<SVGConstructor<any, any, any>>(SVGWrapper) {
   public toSVG(parent: N) {
     const text = (this.node as TextNode).getText();
     const variant = this.parent.variant;
+    if (text.length === 0) return;
     if (variant === '-explicitFont') {
-      this.adaptor.append(parent, this.jax.unknownText(text, variant));
+      this.element = this.adaptor.append(parent, this.jax.unknownText(text, variant));
     } else {
       const chars = this.remappedText(text, variant);
+      if (this.parent.childNodes.length > 1) {
+        parent = this.element = this.adaptor.append(parent, this.svg('g', {'data-mml-node': 'text'}));
+      }
       let x = 0;
       for (const n of chars) {
         x += this.placeChar(n, x, 0, parent, variant);
       }
     }
-    this.element = this.adaptor.lastChild(parent);
   }
 
 }
diff --git a/ts/output/svg/Wrappers/mtext.ts b/ts/output/svg/Wrappers/mtext.ts
@@ -38,7 +38,7 @@ export class SVGmtext<N, T, D> extends
 CommonMtextMixin<SVGConstructor<any, any, any>>(SVGWrapper) {
 
   /**
-   * The mtet wrapper
+   * The mtext wrapper
    */
   public static kind = MmlMtext.prototype.kind;