github/penpot
1
0
Fork 0
mirror of https://github.com/penpot/penpot.git synced 2026-06-16 04:12:03 +00:00
Code Issues Packages Projects Releases Wiki Activity

🐛 Fix stroke to path extra points

Browse Source
This commit is contained in:
Aitor Moreno 2026-06-15 16:06:24 +02:00
parent b03537fa68
commit f10a164c8d
2 changed files with 17 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
render-wasm/src/shapes/paths.rs
View File

@ -156,6 +156,7 @@ impl Path {
let mut current_point = 0;
let mut current_conic = 0;
let mut last_point = skia::Point::new(0.0, 0.0);
let mut subpath_start = skia::Point::new(0.0, 0.0);
for verb in verbs {
match verb {
@ -163,12 +164,15 @@ impl Path {
let p = points[current_point];
segments.push(Segment::MoveTo((p.x, p.y)));
last_point = p;
subpath_start = p;
current_point += 1;
}
skia::PathVerb::Line => {
let p = points[current_point];
segments.push(Segment::LineTo((p.x, p.y)));
last_point = p;
if p != last_point {
segments.push(Segment::LineTo((p.x, p.y)));
last_point = p;
}
current_point += 1;
}
skia::PathVerb::Quad => {
@ -239,6 +243,13 @@ impl Path {
current_point += 3;
}
skia::PathVerb::Close => {
if let Some(Segment::LineTo(p)) = segments.last() {
if (p.0 - subpath_start.x).abs() < 1e-5
&& (p.1 - subpath_start.y).abs() < 1e-5
{
segments.pop();
}
}
segments.push(Segment::Close);
}
}

8
render-wasm/src/shapes/stroke_paths.rs
View File

@ -60,6 +60,9 @@ pub fn stroke_to_path(
// Set EvenOdd to preserve the annular ring's inner hole,
// then as_winding() on the result fixes contour winding
// for Penpot's NonZero fill rule.
// Center strokes skip the conversion: fill_path_with_paint
// already produces correctly-wound contours, so as_winding()
// would only add unnecessary reconstruction points.
let final_path = match render_kind {
StrokeKind::Inner => {
stroke_outline.set_fill_type(skia::PathFillType::EvenOdd);
@ -75,10 +78,7 @@ pub fn stroke_to_path(
.unwrap_or(stroke_outline);
outer.as_winding().unwrap_or(outer)
}
StrokeKind::Center => {
stroke_outline.set_fill_type(skia::PathFillType::EvenOdd);
stroke_outline.as_winding().unwrap_or(stroke_outline)
}
StrokeKind::Center => stroke_outline,
};
// If there was a path_transform, invert it back to local coords