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

Updating TabView to consume tear-out APIs #1574

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

llongley
Copy link
Member

These changes update WinUI Gallery to consume the new TabView tear-out APIs in WASDK experimental 1, instead of using the WinRT drag-and-drop APIs. This makes it so that tab tear-out immediately creates a new window that the user starts to drag, rather than waiting for a drop to create the new window.

While I was here, I also added a change to enable running WinUI Gallery as an unpackaged app, which appeared to previously not be properly functional.

@karkarl
Copy link
Collaborator

karkarl commented Jul 18, 2024

Is it intended to merge into Scott's 1.6 Experimental1 branch?

Copy link
Member

@Scottj1s Scottj1s left a comment

Choose a reason for hiding this comment

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

per Karen, target branch looks like it needs updating

@llongley llongley force-pushed the user/llongley/ConsumingTabTearOut branch from 2377d8e to 5458f50 Compare August 13, 2024 23:49
@llongley llongley changed the base branch from user/sjones/1_6_exp1_sync to main August 13, 2024 23:49
@llongley
Copy link
Member Author

Updated to target the main branch instead.

@llongley llongley requested a review from Scottj1s August 13, 2024 23:50
@@ -1,7 +1,8 @@
<Project>
<PropertyGroup>
<!-- The NuGet versions of dependencies to build against. -->
<WindowsAppSdkPackageVersion>1.5.240607001</WindowsAppSdkPackageVersion>
<WindowsSdkPackageVersion>10.0.22621.35-preview</WindowsSdkPackageVersion>
Copy link
Member

Choose a reason for hiding this comment

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

can use non-preview 10.0.22621.42

@@ -311,5 +311,10 @@
<CopyToPublishDirectory>PreserveNewest</CopyToPublishDirectory>
</_NoneWithTargetPath>
</ItemGroup>
</Target>
</Target>
<!-- Needed in order for running unpackaged to work properly. -->
Copy link
Member

Choose a reason for hiding this comment

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

shouldn't need this - what assets aren't being copied?

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

3 participants