Fixes nasty bug meaning that positioned elements were being output tw…

…ice. + Does this by adding a test to see if the element already has an element as this method (BlockBox::layout) is called recursively for postioned elements. + Updates the absolute test that was broken by no longer overprinting the absolute positioned element.
danfickle · Sep 7, 2018 · 87dc1a9 · 87dc1a9
1 parent 3c20893
commit 87dc1a9
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 1 deletion.
diff --git a/openhtmltopdf-core/src/main/java/com/openhtmltopdf/render/BlockBox.java b/openhtmltopdf-core/src/main/java/com/openhtmltopdf/render/BlockBox.java
@@ -844,7 +844,7 @@ public void layout(LayoutContext c, int contentStart) {
             	}
             	c.getRootLayer().addPage(c);
             }
-        } else if (style.requiresLayer()) {
+        } else if (style.requiresLayer() && this.getLayer() == null) {
             pushedLayer = true;
             c.pushLayer(this);
         }

diff --git a/...tmltopdf-examples/src/main/resources/visualtest/expected/horiz-page-overflow-absolute.pdf b/...tmltopdf-examples/src/main/resources/visualtest/expected/horiz-page-overflow-absolute.pdf