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.
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
Pass the target element to the events emitted by the cursor (fixes #1… #1081
Conversation
As requested by @mayognaise. Is this what you need? |
looks good. initial raycaster tests would be great |
Hi! Is it possible to put this into |
@ngokevin Can we cherry pick this commit and make a 0.1.3 release? |
@mayognaise I can provide you a custom build of 0.1.2 with this cherry-picked. We might be able to release next week, and we might clear the blockers preventing you to upgrade as well. |
f87a0da
to
5f7ec4f
Compare
var entityFactory = require('../helpers').entityFactory; | ||
|
||
suite('raycaster', function () { | ||
'use strict'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
don't think we need the strict since we have a linter
r+wc we'll do stronger tests later when we improve the raycaster component |
Notice that the raycaster test is a bonus. This patch doesn't touch the raycaster but the cursor |
looks good |
Thanks for the bonus. |
Pass the target element to the events emitted by the cursor (fixes #1…
…067)