Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added Windows API Contract Checker #1308

Merged
merged 10 commits into from Nov 10, 2017

Conversation

StephenHodgson
Copy link
Contributor

@StephenHodgson StephenHodgson commented Nov 7, 2017

Fixes #1289
Fixes #1325

  • Made the 16299 SDK the default target when building with the Build And Deploy Window.
  • Made msbuild 15.0 the default build version number.
  • Update readme documentation with editor usage for Hololens/IMHD.
    • After this PR is merged we should be able to use one editor version for all WMR apps.

Moved InteractionSourceExtentions back into Utilities/Extensions
Updated Build Window to only support 16299 SDK build target
@@ -89,19 +99,22 @@ public static void StopHaptics(this InteractionSource interactionSource)
public static IAsyncOperation<IRandomAccessStreamWithContentType> TryGetRenderableModelAsync(this InteractionSource interactionSource)
{
IAsyncOperation<IRandomAccessStreamWithContentType> returnValue = null;

UnityEngine.WSA.Application.InvokeOnUIThread(() =>

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: extra whitespace

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What's wrong with the whitespace here?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nvm I see it, thanks!

[SerializeField]
private Text text;

public void CheckV5Api()
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pressing the buttons with a motion controller doesn't seem to call this code (voice "select" works). Is that something that'll be fixed with your uGUI fixes?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That is correct. Will be fixed by the uGUI raycast PR

- component: {fileID: 1598773014}
- component: {fileID: 1598773019}
m_Layer: 5
m_Name: Canvas
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The buttons are in the floor on an immersive HMD, so we should probably put a SceneContentAdjuster on them.
Also, I tried that, and the SceneContentAdjuster only seems to work in Play mode. When I built and deployed to an immersive HMD, the buttons were still in the floor. Was that tested to work when deployed?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

SceneContentAdjuster also overwrites the X and Z values, which doesn't seem ideal.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll double check this in a different PR. Open an issue for me please?

Should be working when deployed (Worked for me in MRTP3, but they did change the way that stuff works so idk)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@keveleigh I'll address this in #1309 Removed OptimizeSceneForDeviceType

@StephenHodgson StephenHodgson merged commit 842c4a1 into microsoft:master Nov 10, 2017
@StephenHodgson StephenHodgson deleted the MRTK-APIChecker branch November 10, 2017 19:47
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.

None yet

2 participants