-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Create a specific XAML page for the ShellPageWrapper used for Shell Navigation #11766
Conversation
@@ -19,6 +19,8 @@ public static IApp ConfigureApp() | |||
{ | |||
if (Session == null) | |||
Session = CreateWindowsDriver(); | |||
else | |||
Reset(); |
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.
If user gets here but the app is already running then they want it reset to the first page. This is how android/ios work
@@ -573,7 +581,19 @@ public CoreRootPage(Page rootPage, NavigationBehavior navigationBehavior = Navig | |||
Command = new Command(async () => | |||
{ | |||
if (!string.IsNullOrEmpty(searchBar.Text)) | |||
await corePageView.PushPage(searchBar.Text); | |||
{ | |||
if(!(await corePageView.PushPage(searchBar.Text))) |
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.
All the gallery code added is to enable navigating to store shell from the root page of the gallery
@@ -193,12 +193,12 @@ steps: | |||
TargetFolder: ${{ parameters.artifactsTargetFolder }} | |||
|
|||
- script: build.cmd -Target cg-uwp-build-tests -ScriptArgs '-BUILD_CONFIGURATION="$(BuildConfiguration)"' | |||
condition: eq(variables['BuildConfiguration'], 'Debug') | |||
condition: and(eq(variables['BuildConfiguration'], 'Release'), ne('${{ parameters.includePages }}', true)) |
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.
none of these steps are useful to the pages build lane
.WithRestore() | ||
); | ||
|
||
MSBuild("Xamarin.Forms.Core.Windows.UITests\\Xamarin.Forms.Core.Windows.UITests.csproj", | ||
GetMSBuildSettings().WithRestore()); | ||
GetMSBuildSettings(buildConfiguration:"Debug").WithRestore()); |
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.
There's no reason to build the UI Tests themselves with a release configuration
Test failures unrelated |
iOS and Android failures unrelated |
Description of Change
Issues Resolved
Platforms Affected
Testing Procedure
PR Checklist