Skip to content
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.

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

Introduce Interactable reset API and behaviour using it #144

Merged
merged 3 commits into from
Jul 25, 2024

Conversation

FejZa
Copy link
Contributor

@FejZa FejZa commented Jul 25, 2024

Reality Collective - Reality Toolkit Pull Request

Overview

It is a common scenario that we want to reset or clean up an interactable after the user is done interacting with it. When such a reset occurs, all interaction behaviours on the interactable must rest, if applicable.

  • A new IInteractable.ResetInteractable API was introduced. This API will ask the interactable to perform a full reset and will make sure all IInteractionBehaviours attached to it, will receive a notification to do so as well.

  • A new AutoResetBehaviour has been added. This interaction behaviour will monitor interactions and once ALL interaction on the interactable have ended, it will execute a reset after a configurable amount of time. It also comes with an option to auto reset the interactable to its initial pose.

  • A new "Reset" event has been exposed for third party scripts or the Unity editor to hook into when a reset happens

  • Updated all the docs help URLs to the new domain

  • Added a way to hide the target handedness setting in the inspector for behaviours that ignore that setting

@SimonDarksideJ SimonDarksideJ changed the title Introduce Interacable reset API and behaviour using it Introduce Interactable reset API and behaviour using it Jul 25, 2024
@FejZa FejZa merged commit f985d45 into development Jul 25, 2024
9 checks passed
@FejZa FejZa deleted the feature/interactable-reset branch July 25, 2024 12:06
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