Ensure margin is considered when positioning margin elements

dragonstyle · dragonstyle · commit ff88d499727a · 2022-06-21T15:18:34.000-04:00
- margins on previous elements weren’t being included when computing their bottom
diff --git a/src/resources/formats/html/quarto.js b/src/resources/formats/html/quarto.js
@@ -427,7 +427,10 @@ window.document.addEventListener("DOMContentLoaded", function (_event) {
       const margin = lastBottom - top;
       marginChild.style.marginTop = `${margin}px`;
     }
-    lastBottom = top + marginChild.getBoundingClientRect().height;
+    const styles = window.getComputedStyle(marginChild);
+    const marginTop = parseFloat(styles["marginTop"]);
+
+    lastBottom = top + marginChild.getBoundingClientRect().height + marginTop;
   }
 
   // Manage the visibility of the toc and the sidebar

Original file line number	Diff line number	Diff line change
`@@ -427,7 +427,10 @@ window.document.addEventListener("DOMContentLoaded", function (_event) {`
`427`	`427`	`const margin = lastBottom - top;`
`428`	`428`	marginChild.style.marginTop = `${margin}px`;
`429`	`429`	`}`
`430`		`- lastBottom = top + marginChild.getBoundingClientRect().height;`
	`430`	`+ const styles = window.getComputedStyle(marginChild);`
	`431`	`+ const marginTop = parseFloat(styles["marginTop"]);`
	`432`	`+`
	`433`	`+ lastBottom = top + marginChild.getBoundingClientRect().height + marginTop;`
`431`	`434`	`}`
`432`	`435`
`433`	`436`	`// Manage the visibility of the toc and the sidebar`