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] Media Element Popup restore from Full Screen CTD #2252

Open
2 tasks done
ne0rrmatrix opened this issue Oct 3, 2024 · 1 comment
Open
2 tasks done

[BUG] Media Element Popup restore from Full Screen CTD #2252

ne0rrmatrix opened this issue Oct 3, 2024 · 1 comment
Labels
bug Something isn't working 📽️ MediaElement Issue/PR that has to do with MediaElement

Comments

@ne0rrmatrix
Copy link
Contributor

ne0rrmatrix commented Oct 3, 2024

Is there an existing issue for this?

  • I have searched the existing issues

Did you read the "Reporting a bug" section on Contributing file?

Current Behavior

When restoring media element from full screen in a popup the app crashes.

Expected Behavior

The app should not crash and media element in popup should close without error.

Steps To Reproduce

  1. Open sample app
  2. Navigate to Media Element page
  3. Open a popup
  4. Set full screen
  5. Close full screen

Link to public reproduction project repository

https://github.com/CommunityToolkit/Maui

Environment

- .NET MAUI CommunityToolkit:Current Main as of October 03 2024
- OS:iOS 18.x, Windows 10/11
- .NET MAUI:8.0.71

Anything else?

9F1D8AB0-4EB8-44F1-9CFD-4D9F605DD087

@ne0rrmatrix ne0rrmatrix added bug Something isn't working unverified labels Oct 3, 2024
@ne0rrmatrix ne0rrmatrix changed the title [BUG] Media Element Popup restore from Full Screen CTD [BUG] Media Element Popup restore from Full Screen CTD IOS Oct 4, 2024
@ne0rrmatrix ne0rrmatrix changed the title [BUG] Media Element Popup restore from Full Screen CTD IOS [BUG] Media Element Popup restore from Full Screen CTD Oct 4, 2024
@ne0rrmatrix ne0rrmatrix added 📽️ MediaElement Issue/PR that has to do with MediaElement and removed unverified labels Oct 4, 2024
@ne0rrmatrix
Copy link
Contributor Author

Anyone needing to have this today and need a temporary workaround for windows can do the following. For testing purposes and not for production a quick way to test an app where this is a blocker for testing removing the handler disconnect will prevent the crash. Not ideal or workable for a production app but if you want to test other things and require this to just work for now it is quick and dirty fix. Not ideal and not recommended for anything but testing. This specifically applies to the handler disconnect inside the popup. For media element itself follow the recommended guidelines.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working 📽️ MediaElement Issue/PR that has to do with MediaElement
Projects
None yet
Development

No branches or pull requests

1 participant