fix(pdf) headings can now contain different kinds of things

Signed-off-by: Jerome Simeon <[email protected]>
accordproject · Sep 21, 2020 · 063bdeb · 063bdeb
1 parent 15b6a10
commit 063bdeb
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/packages/markdown-pdf/src/ToPdfMakeVisitor.js b/packages/markdown-pdf/src/ToPdfMakeVisitor.js
@@ -236,11 +236,9 @@ class ToPdfMakeVisitor {
         }
             break;
         case 'Heading': {
-            const child = this.processChildNodes(thing,parameters);
-            const text = child && child[0] && child[0].text ? child[0].text : '';
             result.style = ToPdfMakeVisitor.getHeadingType(thing);
-            result.text = `\n${text}\n`;
-            result.tocItem = text.length > 0 ? true : false;
+            result.text = this.processChildNodes(thing,parameters);
+            result.tocItem = thing.nodes && thing.nodes.length > 0 ? true : false;
         }
             break;
         case 'ThematicBreak': {

diff --git a/packages/markdown-pdf/test/data/contract.md b/packages/markdown-pdf/test/data/contract.md
@@ -1,5 +1,10 @@
 # Title (Heading One)
 
+Title2
+*(Heading Two)*
+**on several lines**
+----
+
 This is normal text.
 
 This is *italic* text.