Merge pull request #82 from Financial-Times/stronger-whitespace-check

adgad · web-flow · commit 6bda5b87abfa · 2025-05-23T09:58:19.000+01:00
More rigourous removal of weird whitespace at the end of paragraph
diff --git a/libraries/from-bodyxml/index.js b/libraries/from-bodyxml/index.js
@@ -444,7 +444,7 @@ export function fromXast(bodyxast, transformers = defaultTransformers) {
  * @returns {string}
  */
 function removeWhitespaceBeforeBodyTag(xml) {
-  return xml.replace("</p> </body>", "</p></body>");
+  return xml.replace(/<\/p>\s+<\/body>/gi, '</p></body>');
 }
 
 /** @param {string} bodyxml */

Original file line number	Diff line number	Diff line change
`@@ -444,7 +444,7 @@ export function fromXast(bodyxast, transformers = defaultTransformers) {`
`444`	`444`	`* @returns {string}`
`445`	`445`	`*/`
`446`	`446`	`function removeWhitespaceBeforeBodyTag(xml) {`
`447`		`- return xml.replace("</p> </body>", "</p></body>");`
	`447`	`+ return xml.replace(/<\/p>\s+<\/body>/gi, '</p></body>');`
`448`	`448`	`}`
`449`	`449`
`450`	`450`	`/** @param {string} bodyxml */`