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

Fix auto scene switcher dialog appearing when toolkit is already active and configured #754

Merged
merged 2 commits into from
Jan 14, 2021

Conversation

FejZa
Copy link
Contributor

@FejZa FejZa commented Jan 13, 2021

XRTK - Mixed Reality Toolkit Pull Request

Overview

This PR fixes the AutoSceneSwitcher showing up when the toolkit is actually already configured in the scene(s).
Also this fixes the AutoSceneSwitcher forcing the XRTK base scene to be the active scene. This breaks multi-scene editing behaviour expected from the editor. As only the active scene's lightmaps will be used and people might want another scene to be the active scene in editor. The XRTK does not require the base scene to be the active scene, it just needs to be loaded.

@FejZa FejZa self-assigned this Jan 13, 2021
@FejZa FejZa added the Ready for review PR finished primary development, open for review label Jan 13, 2021
@FejZa FejZa mentioned this pull request Jan 13, 2021
@FejZa FejZa requested review from a team and SimonDarksideJ and removed request for a team January 13, 2021 15:07
@StephenHodgson
Copy link
Contributor

/azp run

@azure-pipelines
Copy link
Contributor

Azure Pipelines successfully started running 1 pipeline(s).

@StephenHodgson StephenHodgson merged commit 64b8232 into development Jan 14, 2021
@StephenHodgson StephenHodgson deleted the fix/auto-scene-switcher-dialog branch January 14, 2021 18:49
XRTK-Build-Bot pushed a commit that referenced this pull request Jan 15, 2021
Co-authored-by: Stephen Hodgson <hodgson.designs@gmail.com>
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.

3 participants