@adobe/helix-markdown-support 3.1.4 → 3.1.5

package/CHANGELOG.md

@@ -1,3 +1,10 @@
+## [3.1.5](https://github.com/adobe/helix-markdown-support/compare/v3.1.4...v3.1.5) (2022-05-17)
+
+
+### Bug Fixes
+
+* (re)move breaks in links and headings ([621c85b](https://github.com/adobe/helix-markdown-support/commit/621c85b664621e56111e00e9cbb9cd7d648471d6)), closes [#106](https://github.com/adobe/helix-markdown-support/issues/106)
+
 ## [3.1.4](https://github.com/adobe/helix-markdown-support/compare/v3.1.3...v3.1.4) (2022-05-10)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adobe/helix-markdown-support",
-  "version": "3.1.4",
+  "version": "3.1.5",
   "description": "Helix Markdown Support",
   "type": "module",
   "scripts": {

package/src/mdast-sanitize-heading.js

@@ -14,6 +14,7 @@ import { visit } from 'unist-util-visit';
 /**
  * Sanitizes headings:
  * - (re)move images ('before', 'both', 'after')
+ * - converts BREAKs inside headings to <br>.
  *
  * @param {object} tree
  * @param {object} [opts] options
@@ -47,6 +48,9 @@ export default function sanitizeHeading(tree, opts = {}) {
             siblings.splice(after, 0, para);
             after += 1;
           }
+        } else if (child.type === 'break') {
+          child.type = 'html';
+          child.value = '<br>';
         }
       }
       // remove empty headings

package/src/mdast-sanitize-links.js

@@ -15,6 +15,8 @@ import find from 'unist-util-find';
 /**
  * Sanitizes links:
  * - unwraps formatting in links if possible. eg: [_text_](...) -> _[test](...)_
+ * - moves leading, trailing BREAKs out of link
+ * - converts BREAKs inside link to <br>.
  *
  * @param {object} tree
  * @returns {object} The modified (original) tree.
@@ -60,6 +62,31 @@ export default function sanitizeLinks(tree) {
           }
         }
       }
+    } else if (node.type === 'link' && children.length > 1) {
+      // move leading breaks outside of link
+      while (children[0]?.type === 'break') {
+        const brk = children.shift();
+        parent.children.splice(index, 0, brk);
+        // eslint-disable-next-line no-param-reassign
+        index += 1;
+      }
+      // move trailing breaks after link
+      let last = children.length - 1;
+      while (children[last]?.type === 'break') {
+        const brk = children.pop();
+        parent.children.splice(index + 1, 0, brk);
+        last -= 1;
+      }
+      // convert inline breaks to <br>
+      for (let i = 0; i < children.length; i += 1) {
+        if (children[i].type === 'break') {
+          children[i] = {
+            type: 'html',
+            value: '<br>',
+          };
+        }
+      }
+      return index + 1;
     }
     return CONTINUE;
   });