Skip to content

Fixed issue where Controller Visuals weren't initialized before OnSourceDetected#10625

Merged
RogPodge merged 5 commits into
microsoft:releases/2.8.1from
RogPodge:ControllerOnSourceStateFix
Jun 1, 2022
Merged

Fixed issue where Controller Visuals weren't initialized before OnSourceDetected#10625
RogPodge merged 5 commits into
microsoft:releases/2.8.1from
RogPodge:ControllerOnSourceStateFix

Conversation

@RogPodge

@RogPodge RogPodge commented Jun 1, 2022

Copy link
Copy Markdown
Contributor

Overview

The Oculus Controller's Visuals property would not be initialized prior to OnSourceDetected getting raised, despite it behaving differently on other platforms. This change realigns the behavior to match other platforms

Changes

@RogPodge RogPodge changed the base branch from main to releases/2.8.1 June 1, 2022 00:49
@RogPodge

RogPodge commented Jun 1, 2022

Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

Comment thread Assets/MRTK/Providers/Oculus/XRSDK/Controllers/OculusXRSDKTouchController.cs Outdated
RogPodge and others added 2 commits June 1, 2022 10:20
…hController.cs

Co-authored-by: Kurtis <kurtie@microsoft.com>
@RogPodge

RogPodge commented Jun 1, 2022

Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@RogPodge

RogPodge commented Jun 1, 2022

Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@RogPodge

RogPodge commented Jun 1, 2022

Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@RogPodge RogPodge merged commit 0edcb0b into microsoft:releases/2.8.1 Jun 1, 2022
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.

Inactive controller visualizer after update to 2.7.0

2 participants