Skip to content
This repository has been archived by the owner on Oct 18, 2023. It is now read-only.

[Bug] Using Prism.Plugin.Popup in combination with IConfirmNavigationAsync breaks GoBack navigations #233

Open
boufran1 opened this issue Jul 26, 2021 · 1 comment

Comments

@boufran1
Copy link

Description

Hi Dan, I am using the Prism bug template to submit this bug. I took some time to debug the issue on my own. It appears that when a Dialog is show to confirm whether the user wants to navigate back or not, the go back navigation fails when the value is positive. The dialog closes but the override in DoPop in PopupPageNavigationService returns a null page instead of the current page. This leads goback to fail with the default GoBackException in Prism which asks about modal or not.

I made a sample app to reproduce the issue. It

Steps to Reproduce

  1. Load the app in android.
  2. Navigate to ViewA using the displayed button.
  3. Hit the goback button or the hardware goback button.
  4. Confirm that you want to navigate back.
  5. Observe that the backward navigation never occured.

Expected Behavior

Navigate back to the MainPage

Actual Behavior

Stays in the current page

Basic Information

  • Version with issue: Prism 8.1.97, Prism.Plugin.Popups 8.0.76
  • Last known good version: Unable to say
  • Xamarin.Forms version: 5.0.0.2083
  • IDE: VisualStudio 2019

Screenshots

Reproduction Link

TestBug.zip

@maxchu92
Copy link

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants