trunk: "imprecise" viewBox height affects node snapping to grid (rev >= 12554/77)
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Inkscape |
Fix Released
|
Medium
|
Alvin Penner |
Bug Description
Steps to reproduce:
1) launch trunk (default prefs, default new doc)
2) open 'File > Document Properties' and change
- page width: 600
- page height: 400
(unit unchanged: px)
3) Add a new layer (Shift+Ctrl+N) to be sure to work without transforms on parent container.
4) enable 'View > Grid'
5) switch to the pen tool and draw the diagonals (on the new layer), using node snapping for start and end nodes of each line.
6) open XML Editor and check the path data
Expected result:
The path data uses integer px (SVG user unit) values.
Actual results:
There are rounding errors (?) in the coordinates of both start and end nodes.
7) in the XML Editor, set the height value in the viewBox attribute to the correct integer value for the height (400)
8) activate 'Snap to cusp nodes' (snap controls bar)
9) select each of the four nodes and snap them again to the grid lines in each page corner
Result:
After correcting the viewBox height value snapping works with the expected precision.
Reproduced with r12653 on Ubuntu 13.04 (inkscape-trunk PPA) and r12654 on OS X 10.7.5
Based on tests with archived builds on OS X:
- works as expected with rev <= 12553
- setting custom page size broken between 12554-12576,
- imprecise viewBox height for custom page sizes with rev 12557 and later,
the regression seems related to the merges of the GSoC 2013 unit improvement branch.
<http://
Attached: sample SVG file created based on steps 1-5 above, revision 12654.
Related branches
description: | updated |
Changed in inkscape: | |
assignee: | nobody → Matthew Petroff (matthewpetroff) |
Changed in inkscape: | |
importance: | Undecided → Medium |
status: | New → Confirmed |
Changed in inkscape: | |
status: | Confirmed → Fix Released |
Changed in inkscape: | |
status: | Fix Released → Triaged |
Changed in inkscape: | |
status: | Fix Committed → Fix Released |
note that the OP has a 1x1px grid