Fix Issue 8207 | [Bug] Shell Flyout Items on UWP aren't showing the Title

[felipebaltazar]

Description of Change

I've create an override to SelectTemplateCore(object item, DependencyObject container)

Issues Resolved

• fixes [Bug] Shell Flyout Items on UWP aren't showing the Title #8207

API Changes

Added to Xamarin.Forms.Platform.UWP.ShellFlyoutTemplateSelector:

• Windows.UI.Xaml.DataTemplate SelectTemplateCore(object item, DependencyObject container)

Platforms Affected

• UWP

Behavioral/Visual Changes

FlyoutItems will show title and icons when it setted

Before/After Screenshots

• Before

• After

Testing Procedure

1. Create Xamarin Forms Shell Template
2. Add some flyout items
3. Add and run UWP platform

Or just run the Issue8207 test from Control Gallery

PR Checklist

• Targets the correct branch
• Tests are passing (or failures are unrelated)

[felipebaltazar]

Hello!

How can I create, from C# code, a layout like this:

I'm trying to write some tests for these scenarios, but I founded only Xaml examples

[jfversluis]

Is there any specific reason you want to do it in code @felipebaltazar ? You should be able to use XAML in your issue page. See other examples like Issue4684.

Of course you should also be able to do it in code. You can find some examples of that in other issue pages as well.

[felipebaltazar]

Is there any specific reason you want to do it in code @felipebaltazar ? You should be able to use XAML in your issue page. See other examples like Issue4684.

Of course you should also be able to do it in code. You can find some examples of that in other issue pages as well.

@jfversluis This is true! I went through some problems the first time I tried to create a Xaml-based UITest ..
But in other PRs that I created (in this repository), I managed to solve them ... I will recreate this problem test in Xaml.

[felipebaltazar]

I've updated the issue class and added a UI test

[felipebaltazar]

@rmarinho @samhouts Any position about this PR? Failing tests seems like not related

[felipebaltazar]

I've updated this branch, can we merge this?

[felipebaltazar]

@rmarinho @samhouts Any position about this PR ?