From 66f989c27bb737ea5bc273318aa1b8a91712029b Mon Sep 17 00:00:00 2001 From: dotCooCoo Date: Sat, 14 Nov 2026 10:09:26 -0800 Subject: [PATCH] Effects: the member limit's doc comment says a compound path with fill or stroke effects becomes a group of pieces --- crates/effects/src/clip.rs | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/crates/effects/src/clip.rs b/crates/effects/src/clip.rs index d04a60b70a..c5e11c648f 100644 --- a/crates/effects/src/clip.rs +++ b/crates/effects/src/clip.rs @@ -37,9 +37,11 @@ pub(crate) fn with_clip_effects_applied(n: &Node) -> Option { /// [`with_clip_effects_applied`] that calls `take` before it visits each member of a group, layer /// or compound path in `n`, at every depth: once `take` returns `false`, that member and the /// members after it are not visited. The members of a group or layer that are not visited stay as -/// they are, with their effects. A compound path with geometry effects of its own, or of its fills -/// and strokes, still becomes a path of its reshaped geometry: the paths in it that are not visited -/// go into that geometry as they are, and their own effects are not applied. +/// they are, with their effects. A compound path with geometry effects of its own still becomes a +/// path of its reshaped geometry. In a clipping path that does not paint, one whose fills or +/// strokes have geometry effects still becomes a group of one path per fill and stroke. In both +/// cases the paths in it that are not visited go +/// into that geometry as they are, and their own effects are not applied. pub(crate) fn with_clip_effects_applied_while(n: &Node, take: fn() -> bool) -> Option { let paint = n.clip_paint(); clip_effects_applied(n, paint.fill.is_none() && paint.stroke.is_none(), take)