8000 Fix endless configure toolkit dialog issue by FejZa · Pull Request #593 · XRTK/com.xrtk.core · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
This repository was archived by the owner on Aug 11, 2024. It is now read-only.

Fix endless configure toolkit dialog issue #593

Merged
merged 2 commits into from
May 18, 2020

Conversation

FejZa
Copy link
Contributor
@FejZa FejZa commented May 18, 2020

XRTK - Mixed Reality Toolkit Pull Request

Overview

Fixes an issue where the "There is no active Mixed Reality Toolkit in your scene! Would you like to create one now?" dialog endlessly blocks the editor when pressing "Later".

To observe the error in dev open a scene without the Toolkit in it, e.g. some scene you want to load in additively when running. While in that scene select the MixedRealityToolkitRootProfile asset. The dialog shows up. Press later, press later, press later, press later...

Changes

Clear selection for one frame if the user answers "Later". That will make sure Unity does not trigger the OnEnable event right away again after selectin "Later".

@FejZa FejZa requested a review from StephenHodgson May 18, 2020 14:50
@FejZa FejZa self-assigned this May 18, 2020
@FejZa FejZa added Ready for review PR finished primary development, open for review Bug Something isn't working In Progress PR currently still being developed and removed Ready for review PR finished primary development, open for review labels May 18, 2020
@FejZa
Copy link
Contributor Author
FejZa commented May 18, 2020

It's not fixed it seems. Let me take another look. My first test passed but now it's back sometimes.

@FejZa FejZa added Ready for review PR finished primary development, open for review and removed In Progress PR currently still being developed labels May 18, 2020
@FejZa
Copy link
Contributor Author
FejZa commented May 18, 2020

Now it's ready

@FejZa FejZa merged commit 5c17920 into development May 18, 2020
@FejZa FejZa deleted the fix/configure-toolkit-later branch May 18, 2020 16:55
XRTK-Build-Bot pushed a commit that referenced this pull request Dec 25, 2020
* Fix endless configure toolkit dialog issue

* Fix prompt
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Bug Something isn't working 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
0