aframe icon indicating copy to clipboard operation
aframe copied to clipboard

Don't use the cursor component if you need to render the controllers yourself

Open AdaRoseCannon opened this issue 1 year ago • 4 comments

This is a small addition to the cursor component that I was thinking about but was a bit slow adding it to the last PR. I still need to test to make sure it behaves as expected in headsets.

Description:

This disables the cursor component when controllers need to be rendered by the user (tracked controllers on opaque displays) because in these situations the user is probably using one of AFrame's controller inputs to render a controller and should put an additional cursor directly on those inputs.

Changes proposed:

  • Early return for xr select events for tracked controllers on opaque displays

AdaRoseCannon avatar Jul 12 '22 08:07 AdaRoseCannon

This needs rebase.

dmarcos avatar Aug 18 '22 11:08 dmarcos

rebased

AdaRoseCannon avatar Aug 18 '22 14:08 AdaRoseCannon

~~This is related as well https://github.com/aframevr/aframe/pull/5076~~ Nevermind I need to look further into this, as was brought up in the related issue to that PR

AdaRoseCannon avatar Aug 18 '22 14:08 AdaRoseCannon

This issue is okay to be merged I think but I will close 5076 for now.

AdaRoseCannon avatar Sep 06 '22 23:09 AdaRoseCannon