ソースを参照

Merge pull request #3502 from CarlosLanderas/lande/vrhelper-refactor-onmeshselected

VRHelper onNewMeshSelected now notifies PickingInfo to subscribers instead of AbstractMesh
David Catuhe 7 年 前
コミット
105969bbe6

+ 2 - 0
dist/preview release/what's new.md

@@ -20,3 +20,5 @@
 ## Bug fixes
 
 ## Breaking changes
+
+- VRHelper onNewMeshSelected now notifies a PickingInfo to subscribers instead of an AbstractMesh ([carloslanderas](https://github.com/carloslanderas))

+ 6 - 2
src/Cameras/VR/babylon.vrExperienceHelper.ts

@@ -109,7 +109,11 @@ module BABYLON {
         private _leftLaserPointer: Nullable<Mesh>;
         private _rightLaserPointer: Nullable<Mesh>;
         private _currentMeshSelected: Nullable<AbstractMesh>;
-        public onNewMeshSelected = new Observable<AbstractMesh>();
+
+        /**
+         * Observable raised when a new mesh is selected based on meshSelectionPredicate
+         */        
+        public onNewMeshSelected = new Observable<PickingInfo>();
         private _circleEase: CircleEase;
 
         /**
@@ -1417,7 +1421,7 @@ module BABYLON {
                             this._isActionableMesh = false;
                         }
                         try {
-                            this.onNewMeshSelected.notifyObservers(this._currentMeshSelected);
+                            this.onNewMeshSelected.notifyObservers(hit);
                         }
                         catch (err) {
                             Tools.Warn("Error in your custom logic onNewMeshSelected: " + err);