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

[iOS] MenuItem Command not executed with FlyoutHeader set #19149

Closed
owl-guy opened this issue Dec 1, 2023 · 5 comments
Closed

[iOS] MenuItem Command not executed with FlyoutHeader set #19149

owl-guy opened this issue Dec 1, 2023 · 5 comments
Labels
area-controls-menubar Desktop MenuBarItems migration-compatibility Xamarin.Forms to .NET MAUI Migration, Upgrade Assistant, Try-Convert platform/iOS 🍎 s/triaged Issue has been reviewed s/try-latest-version Please try to reproduce the potential issue on the latest public version s/verified Verified / Reproducible Issue ready for Engineering Triage t/bug Something isn't working

Comments

@owl-guy
Copy link

owl-guy commented Dec 1, 2023

Description

I have a .NET MAUI App with a Shell and several MenuItems. Without a FlyoutHeader, every Command bound to the MenuItems is executed. With a defined FlyoutHeader however, the first (i.e. topmost) MenuItem-Command does not get executed (same for FlyoutItems)

Steps to Reproduce

  1. Create a new .NET MAUI App
  2. Add a Shell with Flyout- and MenuItems
  3. Add A FlyoutHeader (e.g. a Label)
  4. The command bound to the first item (i.e. topmost item, be it a MenuItem or a FlyoutItem) will not get executed on iOS. On Android, it works fine.

Link to public reproduction project repository

https://github.com/owl-guy/FlyoutWithHeaderExample

Version with bug

8.0.3

Is this a regression from previous behavior?

Yes, this used to work in Xamarin.Forms

Last version that worked well

Unknown/Other

Affected platforms

iOS

Affected platform versions

iOS 17

Did you find any workaround?

No

Relevant log output

No response

@owl-guy owl-guy added the t/bug Something isn't working label Dec 1, 2023
@jsuarezruiz jsuarezruiz added this to the Backlog milestone Dec 4, 2023
@ghost
Copy link

ghost commented Dec 4, 2023

We've added this issue to our backlog, and we will work to address it as time and resources allow. If you have any additional information or questions about this issue, please leave a comment. For additional info about issue management, please read our Triage Process.

@samhouts samhouts added the migration-compatibility Xamarin.Forms to .NET MAUI Migration, Upgrade Assistant, Try-Convert label Dec 6, 2023
@XamlTest XamlTest added s/verified Verified / Reproducible Issue ready for Engineering Triage s/triaged Issue has been reviewed labels Jan 4, 2024
@XamlTest
Copy link

XamlTest commented Jan 4, 2024

Verified this on Visual Studio Enterprise 17.9.0 Preview 2(8.0.3). Repro on iOS 17.0 and MacCatalyst, not repro on Android 14.0-API34 and Windows 11 with below Project:
FlyoutWithHeaderExample.zip

@owl-guy
Copy link
Author

owl-guy commented Mar 28, 2024

Will someone work on this?

@kubaflo
Copy link
Contributor

kubaflo commented Nov 18, 2024

It seems to be working fine in NET9

Simulator.Screen.Recording.-.iPhone.16.Pro.Max.-.2024-11-18.at.02.04.49.mp4

@kubaflo kubaflo added the s/try-latest-version Please try to reproduce the potential issue on the latest public version label Dec 21, 2024
Copy link
Contributor

Hi @owl-guy. We have added the "s/try-latest-version" label to this issue, which indicates that we'd like you to try and reproduce this issue on the latest available public version. This can happen because we think that this issue was fixed in a version that has just been released, or the information provided by you indicates that you might be working with an older version.

You can install the latest version by installing the latest Visual Studio (Preview) with the .NET MAUI workload installed. If the issue still persists, please let us know with any additional details and ideally a reproduction project provided through a GitHub repository.

This issue will be closed automatically in 7 days if we do not hear back from you by then - please feel free to re-open it if you come back to this issue after that time.

@dotnet-policy-service dotnet-policy-service bot removed this from the Backlog milestone Dec 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-controls-menubar Desktop MenuBarItems migration-compatibility Xamarin.Forms to .NET MAUI Migration, Upgrade Assistant, Try-Convert platform/iOS 🍎 s/triaged Issue has been reviewed s/try-latest-version Please try to reproduce the potential issue on the latest public version s/verified Verified / Reproducible Issue ready for Engineering Triage t/bug Something isn't working
Projects
None yet
Development

No branches or pull requests

5 participants