Skip to content

Commit b9c61fa

Browse files
authored
[SecurityUI] Don't bind what seems like a binding mistake from Apple. (#22493)
1 parent 948cbc7 commit b9c61fa

File tree

2 files changed

+7
-1
lines changed

2 files changed

+7
-1
lines changed

src/securityui.cs

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,8 +27,13 @@ interface SFCertificatePresentation {
2727
void PresentSheet (NSWindow window, [NullAllowed] Action dismissHandler);
2828
#endif
2929

30-
#if !__MACOS__
30+
#if !__MACOS__ && !__MACCATALYST__
3131
[NoMac]
32+
// Apple seems to have gotten this wrong for Mac Catalyst: the headers say 'presentSheetInViewController:dismissHandler:'
33+
// is available on Mac Catalyst, but at runtime, 'presentSheetInWindow:dismissHandler:' is. Note that
34+
// 'presentSheetInWindow:dismissHandler:' is not possible on Mac Catalyst, because it uses 'NSWindow',
35+
// which is not available on Mac Catalyst. So until further notice, just don't bind PresentSheet at all.
36+
[NoMacCatalyst]
3237
[Export ("presentSheetInViewController:dismissHandler:")]
3338
void PresentSheet (UIViewController viewController, [NullAllowed] Action dismissHandler);
3439
#endif
Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
!missing-selector! SFCertificatePresentation::presentSheetInViewController:dismissHandler: not bound

0 commit comments

Comments
 (0)