Window.ModalPopping closes app when e.Cancel = true

[brianlagunas]

Description

Using a custom Window class, when hooking into the Window.ModalPopping event, setting the e.Cancel = true causes the app to close when the Android hardware button is pressed for a modal navigation stack.

The offending code can be found in the App.xaml.cs file

Steps to Reproduce

1. Clone the reproduction repo
2. Run the App on an Android enumlator with API 34
3. Click the button on ViewA to Navigate to ViewB
4. Click the button on ViewB to Navigate to ViewC
5. Hit the Android Hardware back button
6. Observe the modal view is no popped, but the App closes.

What I expect to happen is for the modal view to not be popped, and the app to stay running on screen.

For more context, the Prism Library uses a custom Window to take control of the navigation process on a go back for the modal navigation stack. We need to cancel the built-in popping that is invoked by the hardware back button, and use our pop code instead. This gives us the ability to prevent a navigation based on our IConfirmationRequest interface. If the result of our IConfirmationRequest is true we will do the pop modal again, if the result is false, we will not pop the modal and keep the user on the current view.

Link to public reproduction project repository

https://github.com/brianlagunas/WindowModalPoppingBug

Version with bug

8.0.61 SR6.1

Is this a regression from previous behavior?

Not sure, did not test other versions

Last version that worked well

Unknown/Other

Affected platforms

Android

Affected platform versions

Android API 34

Did you find any workaround?

No

Relevant log output

No response

[github-actions]

Hi I'm an AI powered bot that finds similar issues based off the issue title.

Please view the issues below to see if they solve your problem, and if the issue describes your problem please consider closing this one and thumbs upping the other issue to help us prioritize it. Thank you!

Open similar issues:

• PushModal on Windows does not show back button (#6760), similarity score: 0.71

Closed similar issues:

• MAUI Blazor App OnBackPressed not firing after launching Modal XAML Window from Blazor (#11105), similarity score: 0.75
• Android PopModalAsync still fails when popping a NavigationPage after latest review update (.NET 7) (#10656), similarity score: 0.73
• Modal Navigation does not work or throws exception prior to Window Creation/Parenting (#8157), similarity score: 0.72
• Navigation.PopModalAsync fails on Android if page is a NavigationView [.Net7] (#10513), similarity score: 0.72

Note: You can give me feedback by thumbs upping or thumbs downing this comment.

[PureWeen]

I think we'd make this work by wiring up through the OnBackPressedDispatcher thought with #22869 looking I think we'd just override OnBackPressed on the dialog? Or perhaps the expectation is still to use OnBackPressedDispatcher now

[Zhanglirong-Winnie]

This issue has been verified using Visual Studio 17.11.0 Preview 2.1(8.0.61). Can repro on Android platform.

[dotnet-policy-service]

Hi @@brianlagunas. 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.