Skip to content
This repository has been archived by the owner on Aug 11, 2024. It is now read-only.

Feature/721 teleport input action #729

Merged
merged 4 commits into from
Dec 21, 2020

Conversation

FejZa
Copy link
Contributor

@FejZa FejZa commented Dec 21, 2020

XRTK - Mixed Reality Toolkit Pull Request

Overview

This PR is the conclusion to my Teleport System series for v0.2. It implements the feature request described in #721 and fixes some other issues. Full Content:

  • Added a "Teleport Action" configuration option to the MixedRealityTeleportSystemProfile
  • Implemented another MixedRealityTeleportValidationDataProvider constraint maxHeightDistance, which allows restricting teleport distance on the y-axis
  • Fixed TeleportPointerInspector, I missed removing obsolete serialized properties in my previous PRs
  • Implemented support for Digital and SingleAxis input actions for TeleportPointer. Overall the teleport system now supports Digital | SingleAxis | DualAxis input actions for teleport

How To Test

  • Create a Digital input action or use an existing one and map it to some controller input, then assign it as the teleport action in the teleport system profile
  • Create a SingleAxis input action or use an existing one and map it to some controller input, then assign it as the teleport action in the teleport system profile
  • Use the existing "Teleport Direction" input action to test and verify dual axis still works

Changes

Submodule Changes

@FejZa FejZa merged commit fa15550 into development Dec 21, 2020
@FejZa FejZa deleted the feature/721-teleport-input-action branch December 21, 2020 12:49
@FejZa
Copy link
Contributor Author

FejZa commented Dec 21, 2020

Need to bump the SDK package manifest again once the release is out

XRTK-Build-Bot pushed a commit that referenced this pull request Dec 25, 2020
* Add teleport action to system profile

* Add TeleportAction to teleport system

* Implement additionl teleport constraint for height offset
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Ready for review PR finished primary development, open for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants