Miter join style not taken into account when calculating object size

Bug #787568 reported by Ricardo Graça
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Inkscape
Fix Released
Low
Johan Engelen

Bug Description

When using a miter join style for a path, the size of the join is not taken into account when calculating the object size. This can lead to the drawing being clipped in the export process if the user does a "resize page to content" as demonstrated in the attached screenshot.

Revision history for this message
Ricardo Graça (devius) wrote :
Revision history for this message
su_v (suv-lp) wrote :

Please provide information about your OS/platform and Inkscape version and attach the sample SVG file used for the screenshot. with a list of 'steps to reproduced'.

Note that Inkscape 0.48 uses a more precise calculation method for export to bitmap as well as when resizing the page to selection, but not for rendering the selection bounding box on-canvas because the calculation is expensive and would slowly down interactive usage far too much (as currently happens when keeping the export dialog opened, as reported in bug #719993).

See also <http://thread.gmane.org/gmane.comp.graphics.inkscape.devel/33883/focus=33886>

Related reports (fixed in 0.48):
Bug #166659 “Bounding box does not include long miters”
Bug #479964 “"Fit Page to Selection" doesn't take arrow heads into account”

tags: added: selection stroke
removed: join miter size
Revision history for this message
Ricardo Graça (devius) wrote :

The test system is running Ubuntu 11.04 x86_64 with all the packages up to date. Inkscape is version 0.48 installed from the Ubuntu repositories.
This is problematic when doing a "resize page to content" mainly, as the content will be clipped when doing a page export, as is evident from the image I posted. During edit it doesn't seem to cause any harm.
I'm going to create a drawing the shows the error (I'll upload it as soon as I can). Afterwards, to reproduce this is just a matter of opening the file, going to the file menu, drawing properties and then choosing resize page to content.

This seems to be the same bug as #166659 but that one says the fix is targeted to version 0.48, and that's the version I'm using and it's still present.

Revision history for this message
su_v (suv-lp) wrote :

Reproduced with Inkscape 0.48.1 on OS X 10.5.8 (i386)
Not reproduced with Inkscape 0.48+devel r10219

In Inkscape 0.48.1, it works as expected when resizing page to selection, but fails when resizing page to drawing content (without current selection). In current trunk, both methods do include sharp miters within the page borders (o clipping).

Revision history for this message
su_v (suv-lp) wrote :
Changed in inkscape:
importance: Undecided → Low
status: New → Fix Committed
Revision history for this message
su_v (suv-lp) wrote :

Proposing to backport the fix to the 0.48.x branch.

Changed in inkscape:
milestone: none → 0.49
tags: added: backport-proposed
Revision history for this message
jazzynico (jazzynico) wrote :

Commit 10097 can't be backported alone (‘const struct SPItem’ has no member named ‘i2d_affine’ in 0.48.x), and I'm not sure it's an easy an safe thing to do now.
@Johan - What's your take on it?

Changed in inkscape:
assignee: nobody → Johan Engelen (johanengelen)
Revision history for this message
Alex Valavanis (valavanisalex) wrote :

Re comment #7 : I'll drop the backport proposal for now

tags: removed: backport-proposed
Changed in inkscape:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.