Fix Android flyout button (burger) visibility not reevaluated when orientation changes

[guyvaio]

The condition to show the flyout button (burger) or not is set in method ShouldShowToolbarButton of the FlyoutPage. In Android, this method is not hit when orientation changes. This method is only called once during initialization.

This modification triggers the update of this button when orientation changes.

#24468

[rmarinho]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 3 pipeline(s).

[Brosten]

+1

[guyvaio]

Temporary fix https://gist.github.com/guyvaio/5ca11a6aa373c1fe486cc9e25137aeec
Hope this helps others.

[dhindrik]

We have tried the temporary fix, but it doesn't fix the problem for us.

[guyvaio]

We have tried the temporary fix, but it doesn't fix the problem for us.

Line 212 should be hit when tablet is rotated.
toolbarUpdateUpdateBackButtonMethod?.Invoke(toolbar, null);

[dhindrik]

We have tried the temporary fix, but it doesn't fix the problem for us.

Line 212 should be hit when tablet is rotated. toolbarUpdateUpdateBackButtonMethod?.Invoke(toolbar, null);

That line is hit.

[guyvaio]

We have tried the temporary fix, but it doesn't fix the problem for us.

Line 212 should be hit when tablet is rotated. toolbarUpdateUpdateBackButtonMethod?.Invoke(toolbar, null);

That line is hit.

The method invoked by reflection is visible at line 158 of https://github.com/dotnet/maui/blob/main/src/Controls/src/Core/NavigationPage/NavigationPageToolbar.cs
Please check the conditions to see if you hit lines 171 and 173 (_drawerToggleVisible = ...)

Have also a check on the ShouldShowToolbarButton method (if not overriden in your code) : line 181 of https://github.com/dotnet/maui/blob/main/src/Controls/src/Core/FlyoutPage/FlyoutPage.cs

[dhindrik]

We did a simple fix for this. Maybe not the nicest and cleanest solution, but it make it work and the menu button is always visible when it should.

 Flyout = new ContentPage();
 Flyout.SizeChanged += Flyout_SizeChanged;

public override bool ShouldShowToolbarButton()
{
#if ANDROID
      return true;
#else
      return base.ShouldShowToolbarButton();
#endif
 }