Merge pull request #56 from TanguyPa/margins-fix

Jaben · web-flow · commit a0762c4ff2d5 · 2025-09-25T13:03:41.000-04:00
Great -- thanks for your PR!
diff --git a/lib/Domain/Builders/Faceted/PagePropertyBuilder.cs b/lib/Domain/Builders/Faceted/PagePropertyBuilder.cs
@@ -158,9 +158,9 @@ public PagePropertyBuilder SetMargins(
         Dimension marginLeft)
     {
         this.SetMarginTop(marginTop);
-        this.SetMarginRight(marginTop);
-        this.SetMarginBottom(marginTop);
-        this.SetMarginLeft(marginTop);
+        this.SetMarginRight(marginRight);
+        this.SetMarginBottom(marginBottom);
+        this.SetMarginLeft(marginLeft);
 
         return this;
     }
@@ -276,4 +276,4 @@ public PagePropertyBuilder UseChromeDefaults()
     }
 
     #endregion
-}
+}

Original file line number	Diff line number	Diff line change
`@@ -158,9 +158,9 @@ public PagePropertyBuilder SetMargins(`
`158`	`158`	`Dimension marginLeft)`
`159`	`159`	`{`
`160`	`160`	`this.SetMarginTop(marginTop);`
`161`		`- this.SetMarginRight(marginTop);`
`162`		`- this.SetMarginBottom(marginTop);`
`163`		`- this.SetMarginLeft(marginTop);`
	`161`	`+ this.SetMarginRight(marginRight);`
	`162`	`+ this.SetMarginBottom(marginBottom);`
	`163`	`+ this.SetMarginLeft(marginLeft);`
`164`	`164`
`165`	`165`	`return this;`
`166`	`166`	`}`
`@@ -276,4 +276,4 @@ public PagePropertyBuilder UseChromeDefaults()`
`276`	`276`	`}`
`277`	`277`
`278`	`278`	`#endregion`
`279`		`-}`
	`279`	`+}`