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

🐛 NavigationAppBar's automaticallyImplyLeading behavior is wrong #1051

Open
mateusfccp opened this issue Apr 5, 2024 · 2 comments
Open
Labels
enhancement New feature or request

Comments

@mateusfccp
Copy link

Describe the bug
Currently, automaticallyImplyLeading just puts a back arrow or remove it. Even when there is no route below the current one (example, if you use it in a root page), an back arrow will appear.

To Reproduce
Use an NavigationAppBar with automaticallyImplyLeading: true, which is the default. Note that the back arrow button will be the leading even if there's no route below.

Expected behavior
The back arrow should appears if, and only if, there's a route below, which the behavior of popping the current route.

The same behavior as Flutter's Material AppBar. It even uses the same documentation.

@bdlukaa
Copy link
Owner

bdlukaa commented Apr 6, 2024

In some Windows apps, the arrow is displayed even when there is no route to pop. This behavior is usually true when the app consider the navigation view as a route history.

WinUI 3 Gallery
WinUI 3 Gallery

On other apps, this behavior is not respected:

Microsoft Store Home
Microsoft Store Games

Microsoft Store

We could, indeed, add an option to customize this behavior, but it is really hard to follow a pattern when there is not a pattern.

We follow the guidance provided in the NavigationView documentation:

Any major changes to NavigationView should be made after #919 is addressed.

@mateusfccp
Copy link
Author

You are right, I just read the documentation and there are few instructions about the back button.

image image

Considering this documentation, I think automaticallyImplyLeading is a misleading name, specailly because it is the same name and uses the same documentation as the Material counterpart.

Instead, then, we should use the same name as in the documentation, that is, isBackButtonVisible.

@bdlukaa bdlukaa added the enhancement New feature or request label Apr 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants