Gisellameloni / svg-edit

Automatically exported from code.google.com/p/svg-edit
MIT License
0 stars 0 forks source link

Snap to Grid option prevents editor working #1205

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
Using http://svg-edit.googlecode.com/svn/branches/stable/editor/svg-editor.html

What steps will reproduce the problem?
1. Turn on snap to grid using Editor Options

What is the expected output? What do you see instead?
- Expected: Can draw & select elements, but boxes will snap to grid.
- Actual: I don't seem to be able to select anything or draw anything, and get
the following javascript error:

TypeError: c.getSnappingStep is not a function

In what browser did you experience this problem? (ALL, Firefox, Opera, etc)

Firefox 28.0 (Linux Mint) 

In what version of SVG-edit does the problem occur? (Latest trunk, 2.5.1,
etc)

Latest Stable which I believe to be 2.7.1

Please provide any additional information below.

Original issue reported on code.google.com by stuart.m...@spektrix.com on 28 Apr 2014 at 12:14

GoogleCodeExporter commented 8 years ago
I believe the attached patch fixes this.

This is my first submission to this project - please let me know if this isn't 
the method you'd prefer.

Original comment by stuart.m...@spektrix.com on 28 Apr 2014 at 1:51

Attachments:

GoogleCodeExporter commented 8 years ago
Thanks for providing the patch.  I ran into the same issue.

Original comment by acab...@gmail.com on 7 May 2014 at 10:54

GoogleCodeExporter commented 8 years ago
It looks good as far as I can tell (and I searched the code to confirm 
getSnappingStep was indeed not used elsewhere and that curConfig.stepSize was 
not defined elsewhere). Applied to trunk.

Original comment by bret...@gmail.com on 8 May 2014 at 4:10

GoogleCodeExporter commented 8 years ago
Meant to add thanks too!

Original comment by bret...@gmail.com on 8 May 2014 at 4:10

GoogleCodeExporter commented 8 years ago
In the compiled js file of the demo it is still missing.

Original comment by advan...@gmail.com on 28 Dec 2014 at 2:13