Skip to content

Conversation

@coderofsalvation
Copy link
Contributor

NOTE: I need to lint this PR (but before bothering, I would love a pre-blessing of this PR)

This adds the controllermodelready event to hand-tracking-controls, so it behaves like the others controllermodels.
I've also added a examples/showcase/multi-input-anchors demo, to see it in action:

simplescreenrecorder-2025-03-06_18.39.18.mp4
simplescreenrecorder-2025-03-06_18.38.19.mp4

NOTE: Quest 2 and Webxr-emulator was used for testing (other controllers can be added in similar fashion)

Its wearable-component gives others a huge headstart, without having to worry about:

  • controller-models (not) wiping/re-attaching their children after a controller-switch
  • anchoring objects to specific legacy.future controllers
  • the tricky when & why behind model-loaded, controllersupdated-, controllermodelready-, and controllerconnected-events.

@@ -0,0 +1,41 @@
<!DOCTYPE html>
<html>
<head>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a bit too specific to be part of the showcase that should be more like self-contained mini-apps

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@coderofsalvation coderofsalvation force-pushed the fix/controller-specific-anchors branch from ade7e76 to b6d62bf Compare March 12, 2025 11:47
@coderofsalvation coderofsalvation changed the title added 'controllermodelready'-event + controller-specific-anchors demo added 'controllermodelready'-event Mar 12, 2025
@coderofsalvation
Copy link
Contributor Author

coderofsalvation commented Mar 12, 2025

I'm separating concerns by postponing the demo/showcase.
So this PR is now just adressing the missing event of hand-tracking-controls.

The hand-attached-menu showcase (as discussed on discord) needs some ripening (stopwatch e.g.).
I have a hunch that the wearable-component might be working around a timing-bug (which might be better to fix in AFRAME itself).
There's no hurry..slow-cooked demonstrations are better than overcooked fastfood.

@dmarcos
Copy link
Member

dmarcos commented Mar 12, 2025

Thanks!

@dmarcos dmarcos merged commit 8feb200 into aframevr:master Mar 12, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants