Fix clipping when a box/block has a `radius` (#2338)

1 files changed, 16 insertions, 10 deletions

diff --git a/crates/typst-library/src/layout/container.rs b/crates/typst-library/src/layout/container.rs
index 28a56103..36b62864 100644
--- a/crates/typst-library/src/layout/container.rs
+++ b/crates/typst-library/src/layout/container.rs
@@ -146,15 +146,18 @@ impl Layout for BoxElem {
             frame.set_baseline(frame.baseline() - shift);
         }
 
-        // Clip the contents
-        if self.clip(styles) {
-            frame.clip();
-        }
-
         // Prepare fill and stroke.
         let fill = self.fill(styles);
         let stroke = self.stroke(styles).map(|s| s.map(Stroke::unwrap_or_default));
 
+        // Clip the contents
+        if self.clip(styles) {
+            let outset = self.outset(styles).relative_to(frame.size());
+            let size = frame.size() + outset.sum_by_axis();
+            let radius = self.radius(styles);
+            frame.clip(path_rect(size, radius, &stroke));
+        }
+
         // Add fill and/or stroke.
         if fill.is_some() || stroke.iter().any(Option::is_some) {
             let outset = self.outset(styles);
@@ -408,17 +411,20 @@ impl Layout for BlockElem {
             frames
         };
 
+        // Prepare fill and stroke.
+        let fill = self.fill(styles);
+        let stroke = self.stroke(styles).map(|s| s.map(Stroke::unwrap_or_default));
+
         // Clip the contents
         if self.clip(styles) {
             for frame in frames.iter_mut() {
-                frame.clip();
+                let outset = self.outset(styles).relative_to(frame.size());
+                let size = frame.size() + outset.sum_by_axis();
+                let radius = self.radius(styles);
+                frame.clip(path_rect(size, radius, &stroke));
             }
         }
 
-        // Prepare fill and stroke.
-        let fill = self.fill(styles);
-        let stroke = self.stroke(styles).map(|s| s.map(Stroke::unwrap_or_default));
-
         // Add fill and/or stroke.
         if fill.is_some() || stroke.iter().any(Option::is_some) {
             let mut skip = false;