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

NavigationPage shows title bar before navigating #21134

Open
leo-mck opened this issue Mar 11, 2024 · 4 comments
Open

NavigationPage shows title bar before navigating #21134

leo-mck opened this issue Mar 11, 2024 · 4 comments
Labels
area-navigation NavigationPage delighter migration-compatibility Xamarin.Forms to .NET MAUI Migration, Upgrade Assistant, Try-Convert platform/android 🤖 s/triaged Issue has been reviewed s/verified Verified / Reproducible Issue ready for Engineering Triage t/bug Something isn't working
Milestone

Comments

@leo-mck
Copy link

leo-mck commented Mar 11, 2024

Description

I am migrating a Xamarin.Forms app to MAUI and in some cases I have a ContentPage with NavigationPage.HasNavigationBar="False" that navigates to a ContentPage with NavigationPage.HasNavigationBar="True". On XF, this works as expected but on MAUI (android) the NavigationPage title bar push the previous page content down before navigating. This is not using shell.

navpage_xf.mp4
navpage_maui.mp4

Here is a frame showing the problem. We are seeing the first page but with the second page title bar.

mpv-shot0001

Steps to Reproduce

No response

Link to public reproduction project repository

No response

Version with bug

8.0.7 SR2

Is this a regression from previous behavior?

Yes, this used to work in Xamarin.Forms

Last version that worked well

Unknown/Other

Affected platforms

Android

Affected platform versions

No response

Did you find any workaround?

No response

Relevant log output

No response

@leo-mck leo-mck added the t/bug Something isn't working label Mar 11, 2024
@jsuarezruiz jsuarezruiz added area-navigation NavigationPage s/needs-repro Attach a solution or code which reproduces the issue labels Mar 11, 2024
@PureWeen PureWeen added the migration-compatibility Xamarin.Forms to .NET MAUI Migration, Upgrade Assistant, Try-Convert label Mar 11, 2024
@leo-mck
Copy link
Author

leo-mck commented Mar 11, 2024

@PureWeen
Copy link
Member

This is on some level by design.

In XF every single page had its own toolbar so what's getting pushed is the entire screen with or without a toolbar.

in MAUI the toolbar is part of the chrome, so what we're seeing here is the toolbar hiding/showing as a new page is pushed onto the stack.

In XF you'd get this animation when you pushed where the entire page (including toolbar) would swipe onto the screen, which isn't really desirable.

Shell still does this
#20035

We could probably look at improving the animation or timing of the navbar though so it's less abrasive.

@kevinxufei
Copy link

Verified on 17.10.0 Preview 2.0, can repro on android platform with sample project.

@kevinxufei kevinxufei added s/verified Verified / Reproducible Issue ready for Engineering Triage s/triaged Issue has been reviewed labels Mar 13, 2024
@dotnet-policy-service dotnet-policy-service bot added the s/no-recent-activity Issue has had no recent activity label Mar 18, 2024
@leo-mck
Copy link
Author

leo-mck commented Mar 18, 2024

I just added a project that reproduce this issue above. Here it is again: https://github.com/leo-mck/MauiNavigationPageTestApp

@dotnet-policy-service dotnet-policy-service bot added s/needs-attention Issue has more information and needs another look and removed s/needs-repro Attach a solution or code which reproduces the issue s/no-recent-activity Issue has had no recent activity labels Mar 18, 2024
@PureWeen PureWeen added this to the Backlog milestone May 7, 2024
@PureWeen PureWeen removed the s/needs-attention Issue has more information and needs another look label May 7, 2024
@samhouts samhouts removed s/verified Verified / Reproducible Issue ready for Engineering Triage s/triaged Issue has been reviewed labels Jul 3, 2024
@samhouts samhouts added s/verified Verified / Reproducible Issue ready for Engineering Triage s/triaged Issue has been reviewed labels Jul 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-navigation NavigationPage delighter migration-compatibility Xamarin.Forms to .NET MAUI Migration, Upgrade Assistant, Try-Convert platform/android 🤖 s/triaged Issue has been reviewed 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