Incorrect visual bbox for compound paths with mid-markers (regression)

Bug #1051131 reported by su_v on 2012-09-15
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Inkscape
Medium
Alvin Penner

Bug Description

The extents of the visual bounding box of paths with two or more sub-paths and mid-markers are incorrect: an imaginary mid-marker positioned at the SVG origin (using the original path data of the marker, without transforms) is always included in the visual bounding box, independent of the current position of the object.

Steps to reproduce:
1) mv ~/.config/inkscape{,-bkp}
2) launch inkscape
3) create a path with several sub-paths
   (e.g. subtract two shapes, or combine several lines)
4) assign a mid-maker (doesn't matter which one)
--> the visual bounding box now extends to include an imaginary mid-marker at the SVG origin (upper left corner of the page)
5) drag the object around the canvas
--> the visual bounding box extends to include the "imaginary" mid-marker independent of the position of the object.

Not reproduced with stable Inkscape 0.48.x.
Reproduced with Inkscape 0.48+devel r11668

Tested on: Mac OS X 10.5.8 and OS X 10.7.4.

Tests with archived builds of trunk indicate that the regression was introduced with the removal of libnr in revision 10589:
r10588: ok
r10591, r10612, r10614: phantom mid-marker at desktop origin (lower left corner of the page)
r10618, r10620, r10795: phantom mid-marker at SVG origin (upper left corner of the page)

su_v (suv-lp) wrote :

sample SVG file

Alvin Penner (apenner) on 2012-09-15
Changed in inkscape:
status: New → Confirmed
Alvin Penner (apenner) wrote :

fixed typo in marker bbox calculation.
committed to rev 11670

Changed in inkscape:
status: Confirmed → Fix Committed
su_v (suv-lp) on 2012-09-18
Changed in inkscape:
assignee: nobody → Alvin Penner (apenner)
importance: Undecided → Medium
milestone: none → 0.49
milestone: 0.49 → none
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers