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

Switches to UIResponder for App/Scene delegates #4391

Merged
merged 4 commits into from
Jan 29, 2022

Conversation

Redth
Copy link
Member

@Redth Redth commented Jan 28, 2022

This is the proper inheritence chain and allows us to use things like overriding the BuildMenu method.

This fixes #4390

This is the proper inheritence chain and allows us to use things like overriding the BuildMenu method.

This fixes #4390
Copy link
Member

@dalexsoto dalexsoto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me!

@Redth Redth enabled auto-merge (squash) January 28, 2022 22:47
@mattleibow
Copy link
Member

mattleibow commented Jan 29, 2022

Just a note that this is a massive breaking change for anyone that has any extension methods or parameters that took the concrete class, but this is definitely a more future-proof interface to use. Should we ever actually use the concrete type for extensions/parameters? It is an iOS protocol after all.

@Redth Redth disabled auto-merge January 29, 2022 17:28
@Redth
Copy link
Member Author

Redth commented Jan 29, 2022

Maybe this should be made virtual... https://github.com/dotnet/maui/pull/4391/files#diff-2b4bb95e12307423f7126d91b43a1511b68d00535e9f2b5f535af85d7a388333R43

Copy link
Member

@dalexsoto dalexsoto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

now for reals 👍

@Redth Redth merged commit b0ec683 into main Jan 29, 2022
@Redth Redth deleted the ios-maccatalyst-fix-app-scene-delegates branch January 29, 2022 22:16
@github-actions github-actions bot locked and limited conversation to collaborators Dec 22, 2023
@samhouts samhouts added the fixed-in-6.0.200-preview.13.2 Look for this fix in 6.0.200-preview.13.2! label Aug 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
fixed-in-6.0.200-preview.13.2 Look for this fix in 6.0.200-preview.13.2!
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

iOS and MacCatalyst MauiApplicationDelegate should derive from UIResponder
5 participants