Merge pull request #690 from mathjax/issue2674

dpvc · web-flow · commit b602619ad06f · 2021-04-23T12:57:06.000-04:00
Fix problem with not closing empty tags in HTML serialization (mathjax/MathJax#2674)
diff --git a/ts/adaptors/lite/Parser.ts b/ts/adaptors/lite/Parser.ts
@@ -1,3 +1,4 @@
+
 /*************************************************************
  *
  *  Copyright (c) 2018 The MathJax Consortium
@@ -358,7 +359,7 @@ export class LiteParser implements MinDOMParser<LiteDocument> {
     const content = this.serializeInner(adaptor, node, xml);
     const html =
       '<' + tag + (attributes ? ' ' + attributes : '')
-          + (content && !SELF_CLOSING[tag] ? `>${content}</${tag}>` : xml ? '/>' : '>');
+          + ((!xml || content) && !SELF_CLOSING[tag] ? `>${content}</${tag}>` : xml ? '/>' : '>');
     return html;
   }
 

Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,4 @@`
	`1`	`+`
`1`	`2`	`/*************************************************************`
`2`	`3`	`*`
`3`	`4`	`* Copyright (c) 2018 The MathJax Consortium`
`@@ -358,7 +359,7 @@ export class LiteParser implements MinDOMParser<LiteDocument> {`
`358`	`359`	`const content = this.serializeInner(adaptor, node, xml);`
`359`	`360`	`const html =`
`360`	`361`	`'<' + tag + (attributes ? ' ' + attributes : '')`
`361`		- + (content && !SELF_CLOSING[tag] ? `>${content}</${tag}>` : xml ? '/>' : '>');
	`362`	+ + ((!xml \|\| content) && !SELF_CLOSING[tag] ? `>${content}</${tag}>` : xml ? '/>' : '>');
`362`	`363`	`return html;`
`363`	`364`	`}`
`364`	`365`