RegisterForNavigationOnPlatform not Registers Default View instead of Platform View

[erikh111]

Description

I'm attempting to implement the common ViewModel platform or idiom specific View use case and am having trouble with the platform specific View rendering as it should according to the documentation.

Steps to Reproduce

1. Create ViewAPage.xaml, ViewBPage.xaml view using add new item in IDE, ViewAPageViewModel & ViewBPageViewModel generated
2. Create ViewAPageTablet.xaml and ViewBPageAndroid.xaml
3. Register Views/ViewModels in the containerRegistry
   containerRegistry.RegisterForNavigationOnIdiom<ViewAPage, ViewAPageViewModel>(tabletView: typeof(ViewAPageTablet));
containerRegistry.RegisterForNavigationOnPlatform<ViewBPage, ViewBPageViewModel>(new Platform<ViewBPageAndroid>(RuntimePlatform.Android));

Expected Behavior

Expected the ViewBPage.xaml would render when running on an android simulator just as the ViewPageTablet.xaml rendered when running an iOS Tablet simulator

Actual Behavior

The default ViewBPage.xaml renders regardless of simulator OS.

Basic Information

• Version with issue: 7.1.0.431
• Last known good version: 7.1.0.431
• Xamarin.Forms version: 3.2.0.871581
• IDE: Visual Studio Community (for Mac)

Screenshots

Reproduction Link

Archive.zip

[dansiegel]

This is confirmed. It appears that the current registration logic ends up registering the View twice resulting in the original default View type taking precedence.

[dansiegel]

@erikh111 thanks for the bug report and repo app! this has been fixed and will be available shortly in the CI feed.

[erikh111]

@dansiegel thanks for reviewing this issue and providing a fix so quickly!

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.