-
Notifications
You must be signed in to change notification settings - Fork 39
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 multitarget support for the Windows App SDK #17
Conversation
CommunityToolkit.Labs.WinAppSdk/CommunityToolkit - Backup.Labs.WinAppSdk.csproj
Outdated
Show resolved
Hide resolved
Labs/CanvasLayout/samples/CanvasLayout.WinAppSdk/Package.appxmanifest
Outdated
Show resolved
Hide resolved
Labs/CanvasLayout/samples/CanvasLayout.WinAppSdk/Package.appxmanifest
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Couple of small things but looks good. To clarify, with this setup, our library packages in Labs, we build with out of the box support for:
- UWP
- WinUI 3/WinAppSDK
- Uno/WASM w/ UWP
But not the Uno for WinUI3? (TBH, I'm still a bit fuzzy on how Uno splits between UWP and WinUI 3)
Oh, also the generated files aren't part of the |
…nappsdk Added multitarget support for the Windows App SDK
Added multitarget support for the Windows App SDK. Closes #14
This PR:
CommunityToolkit.Labs.WinAppSdk
- for the multi-sample app.CanvasLayout.WinAppSdk
- for the single-sample app.Labs.UnoLib.props
- Is nowLabs.MultiTarget.props
and adds WinAppSDK alongside UWP and Uno.net6.0-windows10.0.19041.0
- as a TargetFramework for libraries and projects.WINAPPSDK
- as a compilation conditional.Before testing, you MUST do a full clean. Visual Studio may fail to find some assemblies otherwise, and you'll get a "No sample pages were found :(" message.