Skip to content
This repository has been archived by the owner on May 1, 2024. It is now read-only.

[Bug] [iOS] Xamarin Shell Flyout navigates incorrectly with more than 4 items #12563

Open
dan-matthews opened this issue Oct 22, 2020 · 7 comments

Comments

@dan-matthews
Copy link

Description

If you use a Xamarin Shell flyout menu with more than 4 items on an iOS device, then selecting the 5th item or below will slide up the 'More' menu, requiring a second selection. This is not ideal behaviour.

On Android navigating to any item in the flyout navigates directly to it.

Steps to Reproduce

  1. Create a Xamarin Forms project (I used Xaminals)
  2. Create a flyout menu
  3. Copy-paste a tab so that you have at least 6 tabs
  4. Run the project on an iOS simulator/phone
  5. Select the 5th option in the Flyout menu

Expected Behavior

App will navigate directly to route specified on 5th item.

Actual Behavior

App shows the More popup, requiring a second selection of the item (will now be top item in the 'More' list).

Basic Information

  • Version with issue: 4.8.0.1269, 4.0.8.0.1560
  • Last known good version: Unknown
  • IDE: VS 2019
  • Platform Target Frameworks:
    • iOS: 11, 12
    • Android: n/a
    • UWP: n/a
  • Android Support Library Version: n/a
  • Nuget Packages: n/a
  • Affected Devices: All iOS
@dan-matthews dan-matthews added s/unverified New report that has yet to be verified t/bug 🐛 labels Oct 22, 2020
@samhouts samhouts added this to New in Triage Oct 22, 2020
@gitkrm
Copy link

gitkrm commented Oct 23, 2020

don't subscribe, I'm getting over 100 emails a day from this subscription

@PureWeen
Copy link
Contributor

@dan-matthews can you try out 5.0 pre3 and if you still see the issue attach a repro?

I'm not quite seeing what you're talking about here..

I'm seeing the same behavior on Android and iOS for this one

@PureWeen PureWeen added s/needs-info ❓ A question has been asked that requires an answer before work can continue on this issue. s/needs-repro ❔ This reported issue doesn't include a sample project reproducing the issue. Please provide one. labels Oct 31, 2020
@PureWeen PureWeen moved this from New to Ready For Work in Triage Oct 31, 2020
@PureWeen PureWeen moved this from Ready For Work to Needs Info in Triage Oct 31, 2020
@pictos
Copy link
Contributor

pictos commented Nov 2, 2020

@PureWeen this bug still happens. :/
I'm doing some investigation, but the cause is the ViewController for all items inside the More tab are the MoreNavigationController.

@pictos
Copy link
Contributor

pictos commented Nov 2, 2020

I think this one is related to:
#12564
#7594
#11446

@Redth Redth removed s/needs-info ❓ A question has been asked that requires an answer before work can continue on this issue. s/needs-repro ❔ This reported issue doesn't include a sample project reproducing the issue. Please provide one. labels Nov 10, 2020
@Redth Redth moved this from Needs Info to New in Triage Nov 17, 2020
@jsuarezruiz
Copy link
Contributor

Seems to be the same (or related) with #11446

@jsuarezruiz jsuarezruiz added a/navigation p/iOS 🍎 and removed s/unverified New report that has yet to be verified labels Nov 18, 2020
@jsuarezruiz jsuarezruiz moved this from New to Needs Estimate in Triage Nov 18, 2020
@jfdomitor
Copy link

A workaround seems to be to declare separate flyout items instead of using FlyoutDisplayOptions="AsMultipleItems".

This does not work

<FlyoutItem FlyoutDisplayOptions="AsMultipleItems">
   <ShellContent Title="Start"	Route="StartPage"   ContentTemplate="{DataTemplate local:StartPage}"  />
   <ShellContent Title="Check Out"	Route="CheckOutPage"   ContentTemplate="{DataTemplate local:CheckOutPage}"  />
   <ShellContent Title="Check In"	Route="CheckInPage"    ContentTemplate="{DataTemplate local:CheckInPage}" />
  <ShellContent Title="Consume"	Route="ConsumePage"  ContentTemplate="{DataTemplate local:ConsumePage}"  />
  <ShellContent Title="Physical Inventory" Route="PhysInventPage"  ContentTemplate="{DataTemplate local:PhysInvPage}"  />
  <ShellContent Title="About"	Route="AboutPage"   ContentTemplate="{DataTemplate local:AboutPage}" />
</FlyoutItem>

While this works:

<FlyoutItem Title="Start">
  <ShellContent Route="StartPage"    ContentTemplate="{DataTemplate local:StartPage}"  />
</FlyoutItem>
<FlyoutItem Title="Check Out">
  <ShellContent Route="CheckOutPage"  ContentTemplate="{DataTemplate local:CheckOutPage}" />
</FlyoutItem>
<FlyoutItem Title="Check In">
 <ShellContent	Route="CheckInPage"    ContentTemplate="{DataTemplate local:CheckInPage}"  />
</FlyoutItem>
<FlyoutItem Title="Consume">
 <ShellContent Route="ConsumePage"      ContentTemplate="{DataTemplate local:ConsumePage}" />
</FlyoutItem>
<FlyoutItem Title="Physical Inventory">
 <ShellContent  Route="PhysInventPage"  ContentTemplate="{DataTemplate local:PhysInventPage}" />
</FlyoutItem>
<FlyoutItem Title="About">
 <ShellContent 		Route="AboutPage"       ContentTemplate="{DataTemplate local:AboutPage}"  />
</FlyoutItem>

@edgiardina
Copy link

renders the 'more' menu unusable and restricts you to five tabbar options. A vote here to resolve this, please.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
Triage
  
Needs Estimate
Development

No branches or pull requests

8 participants