remove emitting the hit event with el:null when there is no collision, the comment was also wrong there was no state to remove from current element

Author: vincentfretin

src/misc/grab.js

@@ -65,7 +65,7 @@ module.exports = AFRAME.registerComponent('grab', {
     // If the element is already grabbed (it could be grabbed by another controller).
     // If the hand is not grabbing the element does not stick.
     // If we're already grabbing something you can't grab again.
-    if (!hitEl || hitEl.is(this.GRABBED_STATE) || !this.grabbing || this.hitEl) { return; }
+    if (hitEl.is(this.GRABBED_STATE) || !this.grabbing || this.hitEl) { return; }
     hitEl.addState(this.GRABBED_STATE);
     this.hitEl = hitEl;
     this.constraint = new CANNON.LockConstraint(this.el.body, hitEl.body);

src/misc/sphere-collider.js

@@ -100,12 +100,6 @@ module.exports = AFRAME.registerComponent('sphere-collider', {
         .sort((a, b) => distanceMap.get(a) > distanceMap.get(b) ? 1 : -1)
         .forEach(this.handleHit);
 
-      // Remove collision state from current element.
-      if (collisions.length === 0) {
-        this.eventDetail.el = null;
-        el.emit('hit', this.eventDetail);
-      }
-
       // Remove collision state from other elements.
       this.collisions
         .filter((el) => !distanceMap.has(el))