[BUG] Popup is not closing when background is clicked

[xDaijobu]

Is there an existing issue for this?

• I have searched the existing issues

Current Behavior

Work well on IOS but I have some issues on Android

1. changing CanBeDismissedByTappingOutsideOfPopup is not working at runtime
2. Loaded is not called

sorry for the watermark in the video @-@

ScreenRecorderProject1.mp4

Expected Behavior

1. When CloseWhenBackgroundIsClicked is set to true, the popup should close when the background is clicked.
2. Loaded should be called

Steps To Reproduce

1. Open & run the solution
2. click the button & observe the bug

Link to public reproduction project repository

https://github.com/xDaijobu/ToolkitPopupAndroid/tree/master/Popup

Environment

- .NET MAUI CommunityToolkit: 1.3.0
- OS: MacOS Monterey
- .NET MAUI: 7 RC1
- Android API 30
- iOS 15.5

Anything else?

No response

[pictos]

@xDaijobu thanks for reporting the issue. About the VisualElement.Loaded events aren't getting fired part, we can't fix it here... At least I can't see a way to do that.

The Popup is an Element and doesn't have the Loaded/Unloaded events. These events are triggered by the VisualElement class, and I didn't find a way to fire that event outside.
So I don't think it's a bug, but a limitation by design.

If this feature is important to you, please open a new discussion asking for it, and I'll circle back to the .NET MAUI team and see if there's something that I'm missing.

So, I'll remove the Loaded part for this issue.

[xDaijobu]

hi @pictos,
Thank you for the quick fix!

The Popup is an Element and doesn't have the Loaded/Unloaded events

but Popup.Content is View and the View class inherits from VisualElement @.@

If this feature is important to you, please open a new discussion asking for it, and I'll circle back to the .NET MAUI team and see if there's something that I'm missing.

roger that, thanks!

btw, it looks like there is another bug in the Popup..
IconTintColorBehavior not working inside popup.. ( Android )

Should I open a new issue for it?

[pictos]

@xDaijobu For the behavior, please open a new issue

Reopening this because the PR isn't merged yet