search

dotnet / maui

.NET MAUI is the .NET Multi-platform App UI, a framework for building native device applications spanning mobile, tablet, and desktop.
https://dot.net/maui
MIT License
22.26k stars 1.76k forks source link

Flyout Menu Icon Changes to 'Back' Icon When Flyout Appears #25809

Closed brett-macisaac closed 3 days ago

brett-macisaac commented 1 week ago

Description

When the flyout initially appears, the hamburger icon is replaced with a 'back' icon. This can be seen if the flyout menu isn't completely opaque (see below).

Image

Steps to Reproduce

  1. Click the hamburger icon.
  2. Notice that when the flyout menu appears, the hamburger icon is replaced with a 'back' icon.
  3. Interestingly, if you move the flyout menu even just slightly, the hamburger icon reappears.

Link to public reproduction project repository

https://github.com/brett-macisaac/maui_flyout_bug

Version with bug

9.0.0-rc.2.24503.2

Is this a regression from previous behavior?

Not sure, did not test other versions

Last version that worked well

No response

Affected platforms

Android, I was not able test on other platforms

Affected platform versions

Android 14

Did you find any workaround?

The issue can't be seen if the flyout menu's background colour is opaque, although this may go against your app's design.

Relevant log output

Zhanglirong-Winnie commented 1 week ago

This issue has been verified using Visual Studio 17.12 Preview 5(8.0.93 & 8.0.82 & 9.0.0-rc.2.24503.2). Can repro this issue on android platform.

kubaflo commented 5 days ago

Hi @brett-macisaac it is the desired behaviour. Currently MAUI doesn't let you place the Flyout below a navigation bar, so the animation for drawer icon has been turned off. Normally, when it is turned on it follows Android native's behaviour of transforming hamburger menu into arrow and vice versa. And because of the fact that Android shows arrow when drawer is opened the issue you pointed out is not a bug. Check this video to see how it works with the animation on:

https://github.com/user-attachments/assets/3b3d1ea4-c83d-46ec-b0af-849a96e89eb5

@jfversluis @PureWeen I think it should be marked as not planned?

brett-macisaac commented 4 days ago

Thanks @kubaflo that makes sense, I didn't realise that was the native behaviour. It does look a little strange without the animation though. It's a very minor detail though so there's definitely more important areas to focus on.

brett-macisaac commented 4 days ago

@kubaflo (or anyone else) if you find some time could you please check out the other issue that I opened?

mattleibow commented 3 days ago

Closing this as a not an issue.