From f5fe904a389f1ea866c783a03fdf70df466e055f Mon Sep 17 00:00:00 2001 From: Diederik van Lierop Date: Thu, 22 Apr 2010 22:42:03 +0200 Subject: Fix snapping regression introduced by rev. #9118 Fixed bugs: - https://launchpad.net/bugs/562205 (bzr r9364) --- src/display/curve.cpp | 13 +++++++++++-- 1 file changed, 11 insertions(+), 2 deletions(-) (limited to 'src/display/curve.cpp') diff --git a/src/display/curve.cpp b/src/display/curve.cpp index 1b54c981c..73b8dc36d 100644 --- a/src/display/curve.cpp +++ b/src/display/curve.cpp @@ -44,7 +44,7 @@ SPCurve::SPCurve(Geom::PathVector const& pathv) } SPCurve * -SPCurve::new_from_rect(Geom::Rect const &rect) +SPCurve::new_from_rect(Geom::Rect const &rect, bool all_four_sides) { SPCurve *c = new SPCurve(); @@ -54,7 +54,16 @@ SPCurve::new_from_rect(Geom::Rect const &rect) for (int i=3; i>=1; i--) { c->lineto(rect.corner(i)); } - c->closepath(); + + if (all_four_sides) { + // When _constrained_ snapping to a path, the 2geom::SimpleCrosser will be invoked which doesn't consider the closing segment. + // of a path. Consequently, in case we want to snap to for example the page border, we must provide all four sides of the + // rectangle explicitly + c->lineto(rect.corner(0)); + } else { + // ... instead of just three plus a closing segment + c->closepath(); + } return c; } -- cgit v1.2.3