Skip to content

Commit 951087c

Browse files
RogPodgekeveleigh
andauthored
Extra cleanup on the Scene Content System (#9688)
* addressing lingering PR comments * reverting XR Device check due to API compatability * surpressed obsolete function warnings * fixing tracking origin mode check * Update Assets/MRTK/Core/Utilities/SceneContent/MixedRealitySceneContent.cs Co-authored-by: Kurtis <[email protected]> * whitespace fix * added missing edge case for 2018 * updated to only filter out warning on offending lines Co-authored-by: Kurtis <[email protected]>
1 parent d70150b commit 951087c

File tree

3 files changed

+21
-10
lines changed

3 files changed

+21
-10
lines changed

Assets/MRTK/Core/Utilities/SceneContent.meta

Lines changed: 8 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

Assets/MRTK/Core/Utilities/MixedRealitySceneContent.cs renamed to Assets/MRTK/Core/Utilities/SceneContent/MixedRealitySceneContent.cs

Lines changed: 13 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -4,12 +4,8 @@
44
using UnityEngine;
55
using Microsoft.MixedReality.Toolkit.Utilities;
66

7-
#if UNITY_2017_2_OR_NEWER
87
using System.Collections;
98
using UnityEngine.XR;
10-
#else
11-
using UnityEngine.VR;
12-
#endif
139

1410
namespace Microsoft.MixedReality.Toolkit
1511
{
@@ -73,23 +69,30 @@ public void InitializeSceneContent()
7369
return;
7470
}
7571

72+
MixedRealityExperienceSettingsProfile experienceSettingsProfile = MixedRealityToolkit.Instance.ActiveProfile.ExperienceSettingsProfile;
73+
7674
if (alignmentType == AlignmentType.AlignWithExperienceScale)
7775
{
7876
bool experienceAdjustedByXRDevice =
79-
#if UNITY_2019_3_OR_NEWER
80-
XRSubsystemHelpers.InputSubsystem != null && !XRSubsystemHelpers.InputSubsystem.GetTrackingOriginMode().HasFlag(TrackingOriginModeFlags.Unknown);
77+
#if UNITY_2020_1_OR_NEWER
78+
XRSubsystemHelpers.InputSubsystem != null && XRSubsystemHelpers.InputSubsystem.GetTrackingOriginMode().HasFlag(TrackingOriginModeFlags.Floor);
79+
#elif UNITY_2019_1_OR_NEWER
80+
#pragma warning disable 0618
81+
(XRSubsystemHelpers.InputSubsystem != null && XRSubsystemHelpers.InputSubsystem.GetTrackingOriginMode().HasFlag(TrackingOriginModeFlags.Floor)) ||
82+
(XRDevice.isPresent && XRDevice.GetTrackingSpaceType() == TrackingSpaceType.RoomScale);
83+
#pragma warning restore 0618
8184
#else
8285
XRDevice.isPresent && XRDevice.GetTrackingSpaceType() == TrackingSpaceType.RoomScale;
83-
#endif // UNITY_2019_3_OR_NEWER
86+
#endif // UNITY_2020_1_OR_NEWER
8487

8588
// The scene content will be adjusted upwards if the target experience scale is set to room or world scale
8689
// AND if we are either in editor (!XRDevicePresent) or we are on an XR device that will adjust the camera's height
87-
if ((MixedRealityToolkit.Instance.ActiveProfile.ExperienceSettingsProfile.TargetExperienceScale == ExperienceScale.Room ||
88-
MixedRealityToolkit.Instance.ActiveProfile.ExperienceSettingsProfile.TargetExperienceScale == ExperienceScale.World) &&
90+
if ((experienceSettingsProfile.TargetExperienceScale == ExperienceScale.Room ||
91+
experienceSettingsProfile.TargetExperienceScale == ExperienceScale.World) &&
8992
(!DeviceUtility.IsPresent || experienceAdjustedByXRDevice))
9093
{
9194
contentPosition.x = containerObject.position.x;
92-
contentPosition.y = containerObject.position.y + MixedRealityToolkit.Instance.ActiveProfile.ExperienceSettingsProfile.ContentOffset;
95+
contentPosition.y = containerObject.position.y + experienceSettingsProfile.ContentOffset;
9396
contentPosition.z = containerObject.position.z;
9497

9598
containerObject.position = contentPosition;

0 commit comments

Comments
 (0)