fix: correct SetMargins to use proper parameters for each side

TanguyPa · web-flow · commit 5260450c04f4 · 2025-04-27T12:03:24.000+02:00
diff --git a/lib/Domain/Builders/Faceted/PagePropertyBuilder.cs b/lib/Domain/Builders/Faceted/PagePropertyBuilder.cs
@@ -152,9 +152,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;
     }
@@ -270,4 +270,4 @@ public PagePropertyBuilder UseChromeDefaults()
     }
 
     #endregion
-}
+}

Original file line number	Diff line number	Diff line change
`@@ -152,9 +152,9 @@ public PagePropertyBuilder SetMargins(`
`152`	`152`	`Dimension marginLeft)`
`153`	`153`	`{`
`154`	`154`	`this.SetMarginTop(marginTop);`
`155`		`- this.SetMarginRight(marginTop);`
`156`		`- this.SetMarginBottom(marginTop);`
`157`		`- this.SetMarginLeft(marginTop);`
	`155`	`+ this.SetMarginRight(marginRight);`
	`156`	`+ this.SetMarginBottom(marginBottom);`
	`157`	`+ this.SetMarginLeft(marginLeft);`
`158`	`158`
`159`	`159`	`return this;`
`160`	`160`	`}`
`@@ -270,4 +270,4 @@ public PagePropertyBuilder UseChromeDefaults()`
`270`	`270`	`}`
`271`	`271`
`272`	`272`	`#endregion`
`273`		`-}`
	`273`	`+}`