Skip to content

Navigation event doesn't sends if disable laser pointer while pointer above UI element #16

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
Closed
Hitomilras opened this issue Nov 25, 2017 · 1 comment
Closed

Navigation event doesn't sends if disable laser pointer while pointer above UI element #16

Hitomilras opened this issue Nov 25, 2017 · 1 comment
Labels
bug

Comments

@Hitomilras
Copy link

If laser pointer is above some UI element (and element highlighted), disabling laser pointer doesnt disable ui element highlights.
@lawwong lawwong added the bug label Nov 26, 2017
@lawwong
Copy link
Contributor

lawwong commented Nov 26, 2017

Hi, thanks for your report.
Handling the hover event when disabling the laser pointer is accidentally skipped by this line. You can simply remove the line or copy from the fixed file to resolve the bug temporarily. We will release the changes in next version asap.

@lawwong lawwong reopened this Nov 26, 2017
@lawwong lawwong closed this as completed Nov 26, 2017
lawwong pushed a commit that referenced this issue Nov 28, 2017
Update to v1.7.3
d7c3642 
* Bug fix
  - [VRModule] Fix compile error in OculusVRModule.cs #14
  - [Pointer3D] Fix **IPointerExit** not handled correctly when disabling a raycaster #16
  - [Pointer3D] Fix **IPointerExit**/**IPointerPressExit**/**IPointerUp** doesn't execute in some cases
    - Add **Pointer3DEventData.pressProcessed** flag to ensure Down/Up processed correctly

* Improvement
  - Add new struct type **HTC.UnityPlugin.Utility.RigidPose** to replace **HTC.UnityPlugin.PoseTracker.Pose**
    - Utility.RigidPose inherit all members in PoseTracker.Pose
    - Add forward, up, right property getter
  - Obsolete struct **HTC.UnityPlugin.PoseTracker.Pose** (will be removed in future version) to avoid type ambiguous with UnityEngine.Pose (new type in Unity 2017.2)
    - If you somehow want to using HTC.UnityPlugin.PoseTracker and UnityEngine.Pose in your script at the same time, please use full type name or add type alias to avoid ambiguous compile error
  ``` csharp
  using HTC.UnityPlugin.PoseTracker;
  using UnityEngine;
  using Pose = UnityEngine.Pose;

  public class MyPoseTracker : BasePoseTracker
  {
      public Pose m_pose;
      ...
  }
  ```
  - Change some recommended setting value
    - Now binding interface switch is recommended as enable only if Steam VR plugin is imported
    - Now external camera interface switch is recommended as enable only if Steam VR plugin is imported
  - Now VIU system game object will be auto generated only if nessary
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@lawwong @Hitomilras