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

Symbols based on groups return incorrect hitTestAll results #1680

Closed
nikkwong opened this issue Jul 2, 2019 · 0 comments · Fixed by #1685
Closed

Symbols based on groups return incorrect hitTestAll results #1680

nikkwong opened this issue Jul 2, 2019 · 0 comments · Fixed by #1685
Labels
cat: hit-testing status: PR proposed an active pull request should fix this issue type: bug

Comments

@nikkwong
Copy link

nikkwong commented Jul 2, 2019

http://sketch.paperjs.org/#V/0.12.3/S/TVLBbsIwDP2VqKd0oKrTtEvZDhNI0w6TJsENOITgQkYaV6mBIcS/z0kpbaTEif3es53kmjhVQVIk8wOQ3ifjROM2nFd+5U7KC228tiDehYOz+FG0z6bRI+MZjSP5/JqPBS8pr3k6CcyWlZXG2ila9MyPNvPKbbGSLSro7zwe67v8Z9jLZUtepwESpgUSWyiNM2TQ3bG1qsFn80u1QTt7BGWUi8ySs8pANczIJ2zexAub0SiGBY+g0yp8EVSyzzEW1w4Txl/BjeVixBpPgpsdxi4FezrH7VFz1x4h2nvFC97KCAjOTGkyJ0UwcKH7xmMDMzyHLsuj07EnOIGjtC+psxpdQ2JvaAENfdiQp72V2uMvaMr6UKuR1eG90iEf+ZUs7mQPTYdpbvwhNh7UITKbpFiub/8=

I would expect to get the symbol returned as a result of the hitTestAll call, but I get it twice, as well as the original path it looks like.
sasensi added a commit to sasensi/paper.js that referenced this issue Jul 5, 2019
@sasensi
Fix SymbolItem#hitTestAll
6af7a14 
Closes paperjs#1680
@sasensi sasensi mentioned this issue Jul 5, 2019
Merged
2 tasks
@sasensi sasensi added cat: hit-testing status: PR proposed an active pull request should fix this issue type: bug labels Jul 5, 2019
lehni pushed a commit that referenced this issue Jul 12, 2019
@sasensi @lehni
Fix SymbolItem#hitTestAll()
aec1c2c 
Closes #1680
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
cat: hit-testing status: PR proposed an active pull request should fix this issue type: bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix SymbolItem#hitTestAll sasensi/paper.js
2 participants
@nikkwong @sasensi