Include padding when setting node sizes in top-down layout (#1046)

eclipse · Jul 4, 2024 · e8f6b30 · e8f6b30
1 parent 830b539
commit e8f6b30
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/plugins/org.eclipse.elk.core/src/org/eclipse/elk/core/RecursiveGraphLayoutEngine.java b/plugins/org.eclipse.elk.core/src/org/eclipse/elk/core/RecursiveGraphLayoutEngine.java
@@ -252,8 +252,9 @@ protected List<ElkEdge> layoutRecursively(final ElkNode layoutNode, final TestCo
                                 TopdownSizeApproximator approximator = 
                                         childNode.getProperty(CoreOptions.TOPDOWN_SIZE_APPROXIMATOR);
                                 KVector size = approximator.getSize(childNode);
-                                childNode.setDimensions(Math.max(childNode.getWidth(), size.x),
-                                        Math.max(childNode.getHeight(), size.y));
+                                ElkPadding padding = childNode.getProperty(CoreOptions.PADDING);
+                                childNode.setDimensions(Math.max(childNode.getWidth(), size.x + padding.left + padding.right),
+                                        Math.max(childNode.getHeight(), size.y + padding.top + padding.bottom));
                             }
                         }
                     }