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

[BUG] Popup Dialogs are hidden/covered by calling Page when called after returning from MAUI FilePicker #3164

Closed
askariya opened this issue Jun 13, 2024 · 4 comments

Comments

@askariya
Copy link

askariya commented Jun 13, 2024

Description

If you are on Page A and call and await the FilePicker PickAsync function, and then open a Popup dialog, the Popup dialog will be covered by Page A. This is a big issue because while the popup is active in the background, we cannot interact with the app.
This does not seem to occur with the MAUI Community Toolkit Popups, so I think this must be an issue within Mopups or the Prism Dialog Popups.

You can find a sample app with a button that reproduces the issue here: https://github.com/askariya/MauiPrismMopupsSampleApp

Steps to Reproduce

  1. In a ViewModel, add a call to PickAsync() and follow it with a call to open a Popup Dialog:
    var fileData = await FilePicker.PickAsync(); var popupResult = await _dialogService.ShowDialogAsync(nameof(PopupNotificationDialog));
  2. Trigger the above call (ex: with a button press).
  3. Select a file or back out such that the File Picker closes.
  4. You will see that the popup does not appear and instead the Page you called from does.
  5. If you repeat this in debug mode with breakpoints during the Popup initialization, you can very briefly see that the Popup does appear but is covered by the calling Page immediately.

Video:
https://github.com/PrismLibrary/Prism/assets/15271126/591d8f62-56b9-4211-91d6-6f87021b1111

Platform with bug

.NET MAUI

Affected platforms

Android

Did you find any workaround?

No workaround, only solution I could find for this is to use Maui Community Toolkit popups instead.

Relevant log output

No response

@dansiegel
Copy link
Member

Thanks for creating a reproduction. This would seem to be an issue with Mopups though since we just use their API under the covers to launch the Dialog

@askariya
Copy link
Author

Thanks for creating a reproduction. This would seem to be an issue with Mopups though since we just use their API under the covers to launch the Dialog

You're right, removing Prism from the equation results in the error still happening. I will open a Bug report on the Mopups repo.
Will you guys be able to merge in the changes to Prism Popups once it's fixed on the Mopups side?

@dansiegel
Copy link
Member

I periodically update the dependencies like the Mopups plugin... but you can always take it as a top level dependency in your app if they push a fix and you're waiting on the Prism Plugin update

@askariya
Copy link
Author

Good to know, thanks!

@stale stale bot removed wontfix labels Jun 14, 2024
@dansiegel dansiegel closed this as not planned Won't fix, can't repro, duplicate, stale Jun 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants