Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Hit test on Path results in uncaught TypeError #617

Closed
dasboe opened this issue Jan 8, 2015 · 1 comment
Closed

Hit test on Path results in uncaught TypeError #617

dasboe opened this issue Jan 8, 2015 · 1 comment
Assignees
@lehni

Comments

@dasboe
Copy link

dasboe commented Jan 8, 2015

Setting a Paths horizontal and/or vertical scale factor to 0 and its applyMatrix property to false results in an uncaught TypeError in paper.js when performing a hit test on the Path:

see the console log in this sketch:
http://sketch.paperjs.org/#S/q1bKS8xNVbJSCs5OLUnOUNJRSs5PAfHLEosUkjOLknNSFWwV8lLLFQISSzL0nMEiGmB+fmZeiYahgYGOApDQhJAxeRA9eokFBTmVvoklRZkVQP1piTnFqXC54uREoBkgfSD1+XnF+UDBnPx0Dah8RmZJSGpxCTZbNDWBLkwqSk3MLgBJFCtZRcfWAgA=
@lehni lehni self-assigned this Jun 23, 2015
@lehni
Copy link
Member

lehni commented Jun 23, 2015

This happens with non-reversible matrices, due to the call of matrix._inverseTransform(point);:

https://github.com/paperjs/paper.js/blob/develop/src/item/Item.js#L1765

@lehni lehni mentioned this issue Dec 27, 2015
Closed
@lehni lehni closed this as completed in 77a6fda Dec 27, 2015
lehni added a commit that referenced this issue Dec 28, 2015
@lehni
Handle non-reversible matrix in Item#hitTest()
44850ba 
Closes #617
lehni added a commit that referenced this issue Dec 30, 2015
@lehni
Handle non-reversible matrix in Item#hitTest()
b724a59 
Closes #617
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lehni lehni

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@dasboe @lehni