Fix Alert not showing in an app using UIScene

Summary: In an app using `UIScene`, `Alert` no longer pops up because the window that gets created are not attached to a scene. [iOS] [Fixed] - Fix `Alert` not showing in an app using `UIScene`
facebook · Sep 1, 2022 · 98e9a9c · 98e9a9c
1 parent 163171c
commit 98e9a9c
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 33 deletions.
diff --git a/React/CoreModules/RCTAlertController.h b/React/CoreModules/RCTAlertController.h
@@ -10,6 +10,5 @@
 @interface RCTAlertController : UIAlertController
 
 - (void)show:(BOOL)animated completion:(void (^)(void))completion;
-- (void)hide;
 
 @end
diff --git a/React/CoreModules/RCTAlertController.m b/React/CoreModules/RCTAlertController.m
@@ -9,39 +9,11 @@
 
 #import <React/RCTAlertController.h>
 
-@interface RCTAlertController ()
-
-@property (nonatomic, strong) UIWindow *alertWindow;
-
-@end
-
 @implementation RCTAlertController
 
-- (UIWindow *)alertWindow
-{
-  if (_alertWindow == nil) {
-    _alertWindow = [[UIWindow alloc] initWithFrame:RCTSharedApplication().keyWindow.bounds];
-    _alertWindow.rootViewController = [UIViewController new];
-    _alertWindow.windowLevel = UIWindowLevelAlert + 1;
-  }
-  return _alertWindow;
-}
-
 - (void)show:(BOOL)animated completion:(void (^)(void))completion
 {
-  [self.alertWindow makeKeyAndVisible];
-  [self.alertWindow.rootViewController presentViewController:self animated:animated completion:completion];
-}
-
-- (void)hide
-{
-  [_alertWindow setHidden:YES];
-
-  if (@available(iOS 13, *)) {
-    _alertWindow.windowScene = nil;
-  }
-
-  _alertWindow = nil;
+  [[RCTKeyWindow() rootViewController] presentViewController:self animated:animated completion:completion];
 }
 
 @end
diff --git a/React/CoreModules/RCTAlertManager.mm b/React/CoreModules/RCTAlertManager.mm
@@ -185,20 +185,17 @@ - (void)invalidate
                                    case RCTAlertViewStylePlainTextInput:
                                    case RCTAlertViewStyleSecureTextInput:
                                      callback(@[ buttonKey, [weakAlertController.textFields.firstObject text] ]);
-                                     [weakAlertController hide];
                                      break;
                                    case RCTAlertViewStyleLoginAndPasswordInput: {
                                      NSDictionary<NSString *, NSString *> *loginCredentials = @{
                                        @"login" : [weakAlertController.textFields.firstObject text],
                                        @"password" : [weakAlertController.textFields.lastObject text]
                                      };
                                      callback(@[ buttonKey, loginCredentials ]);
-                                     [weakAlertController hide];
                                      break;
                                    }
                                    case RCTAlertViewStyleDefault:
                                      callback(@[ buttonKey ]);
-                                     [weakAlertController hide];
                                      break;
                                  }
                                }];