fix: margin-break:discard not working properly

- fix #1123
vivliostyle · Feb 8, 2023 · 660eefd · 660eefd
1 parent 910bedc
commit 660eefd
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/packages/core/src/vivliostyle/vgen.ts b/packages/core/src/vivliostyle/vgen.ts
@@ -1424,7 +1424,8 @@ export class ViewFactory
           } else {
             const marginBreak = computedStyle["margin-break"];
             if (
-              marginBreak === Css.ident.discard ||
+              (marginBreak === Css.ident.discard &&
+                (atUnforcedBreak || this.isAtForcedBreak())) ||
               (marginBreak !== Css.ident.keep &&
                 atUnforcedBreak &&
                 !this.isAtForcedBreak())
@@ -1503,7 +1504,12 @@ export class ViewFactory
       nodeContext && !nodeContext.after;
       nodeContext = nodeContext.parent
     ) {
-      if (Break.isForcedBreakValue(nodeContext.breakBefore)) {
+      if (
+        Break.isForcedBreakValue(nodeContext.breakBefore) ||
+        nodeContext.sourceNode === nodeContext.sourceNode.ownerDocument.body ||
+        nodeContext.sourceNode ===
+          nodeContext.sourceNode.ownerDocument.documentElement
+      ) {
         return true;
       }
       if (