Orintentation problems with iOS16

Question

Orintentation problems with iOS16

Opened this issue 2 years ago · 25 comments

Girum commented 2 years ago

Hi! After update to iOS16, some functionallities dont work sometimes. The package will be updated?

bishalfan commented 2 years ago

+1

Answer 1 · 2022-09-24T22:47:18.000Z

Up,same issue

Answer 2 · 2022-09-26T18:26:10.000Z

I changed to react-native-orientation-locker and it works fine! But now android has a problem: I'm using react-native-video lib. When I change the orientation, or even change something, the video reloads

Answer 3 · 2022-09-28T05:03:37.000Z

I changed to react-native-orientation-locker and it works fine! But now android has a problem: I'm using react-native-video lib. When I change the orientation, or even change something, the video reloads
Hi @Girum , I am using react-native-orientation-locker, but the callback function is not called in addOrientationListener. So I can not check the orientation. Any solution? Thank you!

Answer 4 · 2022-09-28T11:08:14.000Z

Hi @minhpham1203 . HEre works perfectly.

Orientation.addOrientationListener(orientationChange)
        
return () => {
  Orientation.removeOrientationListener(orientationChange)
};

const orientationChange = (orientation) => {
        if(orientation == 'LANDSCAPE-RIGHT' || orientation ==  'LANDSCAPE-LEFT') {
           ...

            
        } else if (orientation == 'PORTRAIT') {
            ...
        }
    }

The only change I made is that dont exists the enum "LANDASCAPE". I had to set the conditions to "LANDASCAPE_LEFT" and "LANDASCAPE_RIGHT"

@minhpham1203 afeter another test, it not works too... =(. We have to ask in the react-native-orientation-locker github

It fires the if(orientation == 'LANDSCAPE-RIGHT' || orientation == 'LANDSCAPE-LEFT'), but not work in another functions inside of this if

Answer 5 · 2022-09-30T08:34:11.000Z

Hey guys, here's a PR with a fix for iOS16 orientation :)

Answer 6 · 2022-10-01T01:44:44.000Z

if you are still building with iOS 15 sdk, could try going into Orientation.m, and add

[UIViewController attemptRotationToDeviceOrientation];

below every

[[UIDevice currentDevice] setValue:[NSNumber numberWithInteger: ... ] forKey:@"orientation"];

that should work on iOS 16

Answer 7 · 2022-11-04T13:00:18.000Z

Hey guys, here's a PR with a fix for iOS16 orientation :)

The package will be updated?

Answer 8 · 2022-11-10T06:04:32.000Z

Hey guys, here's a PR with a fix for iOS16 orientation :)

any update on this? @gramiro @vinceplusplus @Girum @veerabhadrasajjan

Answer 9 · 2022-11-22T21:05:41.000Z

I think we need to take ownership of this package or use some stable fork since this lib wasn't updated for 5 years. I think it's unlikely it'll be updated in the future

Answer 10 · 2022-11-23T08:54:03.000Z

@codal-mpawar I'm not the owner of the library so I guess we need to do what @kesha-antonov is proposing. As a temporary solution I'm just patching my projects.

Answer 11 · 2022-11-24T07:28:05.000Z

@gramiro it's working locally but it's not working on the release mode.

below is patch package.
diff --git a/node_modules/react-native-orientation/iOS/RCTOrientation/Orientation.m b/node_modules/react-native-orientation/iOS/RCTOrientation/Orientation.m
index 15e9927..67940a4 100644
--- a/node_modules/react-native-orientation/iOS/RCTOrientation/Orientation.m
+++ b/node_modules/react-native-orientation/iOS/RCTOrientation/Orientation.m
@@ -39,15 +39,38 @@ + (BOOL)requiresMainQueueSetup
return YES;
}

+- (void)dispatchOrientationChangeEvent:(UIDeviceOrientation)orientation {

[self.bridge.eventDispatcher sendDeviceEventWithName:@"specificOrientationDidChange"

                                           body:@{@"specificOrientation": [self getSpecificOrientationStr:orientation]}];

[self.bridge.eventDispatcher sendDeviceEventWithName:@"orientationDidChange"

                                           body:@{@"orientation": [self getOrientationStr:orientation]}];

+}
+
+- (void)lockToOrientationWithMask:(UIInterfaceOrientationMask)maskOrientation interfaceOrientation:(UIInterfaceOrientation)interfaceOrientation deviceOrientation:(UIDeviceOrientation)deviceOrientation {

if (@available(iOS 16, *)) {
+#ifdef __IPHONE_16_0

   dispatch_sync(dispatch_get_main_queue(), ^{

       NSArray *array = [[[UIApplication sharedApplication] connectedScenes] allObjects];

       UIWindowScene *scene = (UIWindowScene *)array[0];

       [UIViewController attemptRotationToDeviceOrientation];

       UIWindowSceneGeometryPreferencesIOS *geometryPreferences = [[UIWindowSceneGeometryPreferencesIOS alloc] initWithInterfaceOrientations:maskOrientation];

       [scene requestGeometryUpdateWithPreferences:geometryPreferences errorHandler:^(NSError * _Nonnull error) {}];

```
   });
```

   [self dispatchOrientationChangeEvent:deviceOrientation];

+#endif

} else {

   [[NSOperationQueue mainQueue] addOperationWithBlock:^ {

       [[UIDevice currentDevice] beginGeneratingDeviceOrientationNotifications];

       [[UIDevice currentDevice] setValue:[NSNumber numberWithInteger:interfaceOrientation] forKey:@"orientation"];

```
   }];
```
}
+}

(void)deviceOrientationDidChange:(NSNotification *)notification
{
UIDeviceOrientation orientation = [[UIDevice currentDevice] orientation];
[self.bridge.eventDispatcher sendDeviceEventWithName:@"specificOrientationDidChange"

                                         body:@{@"specificOrientation": [self getSpecificOrientationStr:orientation]}];

[self.bridge.eventDispatcher sendDeviceEventWithName:@"orientationDidChange"

                                         body:@{@"orientation": [self getOrientationStr:orientation]}];

UIDeviceOrientation orientation = [[UIDevice currentDevice] orientation];
[self dispatchOrientationChangeEvent:orientation];
}

(NSString *)getOrientationStr: (UIDeviceOrientation)orientation {
@@ -156,12 +179,8 @@ - (NSString *)getSpecificOrientationStr: (UIDeviceOrientation)orientation {
#if DEBUG
NSLog(@"Locked to Portrait");
#endif
[Orientation setOrientation:UIInterfaceOrientationMaskPortrait];
[[NSOperationQueue mainQueue] addOperationWithBlock:^ {
[[UIDevice currentDevice] beginGeneratingDeviceOrientationNotifications];
[[UIDevice currentDevice] setValue:[NSNumber numberWithInteger: UIInterfaceOrientationPortrait] forKey:@"orientation"];
}];

[Orientation setOrientation:UIInterfaceOrientationMaskPortrait];
[self lockToOrientationWithMask:UIInterfaceOrientationMaskPortrait interfaceOrientation:UIInterfaceOrientationPortrait deviceOrientation:UIDeviceOrientationPortrait];
}

RCT_EXPORT_METHOD(lockToLandscape)
@@ -172,17 +191,11 @@ - (NSString *)getSpecificOrientationStr: (UIDeviceOrientation)orientation {
UIDeviceOrientation orientation = [[UIDevice currentDevice] orientation];
NSString *orientationStr = [self getSpecificOrientationStr:orientation];
if ([orientationStr isEqualToString:@"LANDSCAPE-LEFT"]) {

[Orientation setOrientation:UIInterfaceOrientationMaskLandscape];
[[NSOperationQueue mainQueue] addOperationWithBlock:^ {

 [[UIDevice currentDevice] beginGeneratingDeviceOrientationNotifications];

 [[UIDevice currentDevice] setValue:[NSNumber numberWithInteger: UIInterfaceOrientationLandscapeRight] forKey:@"orientation"];

}];

 [Orientation setOrientation:UIInterfaceOrientationMaskLandscapeRight];

 [self lockToOrientationWithMask:UIInterfaceOrientationMaskLandscapeRight interfaceOrientation:UIInterfaceOrientationLandscapeRight deviceOrientation:UIDeviceOrientationLandscapeRight];

} else {

[Orientation setOrientation:UIInterfaceOrientationMaskLandscape];
[[NSOperationQueue mainQueue] addOperationWithBlock:^ {

 [[UIDevice currentDevice] beginGeneratingDeviceOrientationNotifications];

 [[UIDevice currentDevice] setValue:[NSNumber numberWithInteger: UIInterfaceOrientationLandscapeLeft] forKey:@"orientation"];

}];

 [Orientation setOrientation:UIInterfaceOrientationMaskLandscapeLeft];

 [self lockToOrientationWithMask:UIInterfaceOrientationMaskLandscapeLeft interfaceOrientation:UIInterfaceOrientationLandscapeLeft deviceOrientation:UIDeviceOrientationLandscapeLeft];

}
}

@@ -192,11 +205,7 @@ - (NSString *)getSpecificOrientationStr: (UIDeviceOrientation)orientation {
NSLog(@"Locked to Landscape Left");
#endif
[Orientation setOrientation:UIInterfaceOrientationMaskLandscapeLeft];

[[NSOperationQueue mainQueue] addOperationWithBlock:^ {

   [[UIDevice currentDevice] beginGeneratingDeviceOrientationNotifications];

   [[UIDevice currentDevice] setValue:[NSNumber numberWithInteger: UIInterfaceOrientationLandscapeLeft] forKey:@"orientation"];

}];

[self lockToOrientationWithMask:UIInterfaceOrientationMaskLandscapeLeft interfaceOrientation:UIInterfaceOrientationLandscapeLeft deviceOrientation:UIDeviceOrientationLandscapeLeft];
}

RCT_EXPORT_METHOD(lockToLandscapeRight)
@@ -204,13 +213,8 @@ - (NSString *)getSpecificOrientationStr: (UIDeviceOrientation)orientation {
#if DEBUG
NSLog(@"Locked to Landscape Right");
#endif

[Orientation setOrientation:UIInterfaceOrientationMaskLandscapeRight];
[[NSOperationQueue mainQueue] addOperationWithBlock:^ {
// this seems counter intuitive
[[UIDevice currentDevice] beginGeneratingDeviceOrientationNotifications];
[[UIDevice currentDevice] setValue:[NSNumber numberWithInteger: UIInterfaceOrientationLandscapeRight] forKey:@"orientation"];
}];

[Orientation setOrientation:UIInterfaceOrientationMaskLandscapeRight];
[self lockToOrientationWithMask:UIInterfaceOrientationMaskLandscapeRight interfaceOrientation:UIInterfaceOrientationLandscapeRight deviceOrientation:UIDeviceOrientationLandscapeRight];
}

RCT_EXPORT_METHOD(unlockAllOrientations)
@@ -218,9 +222,9 @@ - (NSString *)getSpecificOrientationStr: (UIDeviceOrientation)orientation {
#if DEBUG
NSLog(@"Unlock All Orientations");
#endif

UIDeviceOrientation orientation = [[UIDevice currentDevice] orientation];
[Orientation setOrientation:UIInterfaceOrientationMaskAllButUpsideDown];
-// AppDelegate *delegate = (AppDelegate *)[[UIApplication sharedApplication] delegate];
-// delegate.orientation = 3;
[self lockToOrientationWithMask:UIInterfaceOrientationMaskAllButUpsideDown interfaceOrientation:UIInterfaceOrientationMaskAllButUpsideDown deviceOrientation:orientation];
}

(NSDictionary *)constantsToExport

Answer 12 · 2022-12-06T12:44:19.000Z

if you are still building with iOS 15 sdk, could try going into Orientation.m, and add
[UIViewController attemptRotationToDeviceOrientation];
below every
[[UIDevice currentDevice] setValue:[NSNumber numberWithInteger: ... ] forKey:@"orientation"];
that should work on iOS 16

can you confirm that is working in real device not in simulator

Answer 13 · 2022-12-07T04:47:07.000Z

@pawansharmaAccolite, yes, it is

Answer 14 · 2022-12-07T15:16:03.000Z

yes same here i made this patch its working fine at local but when i hit on pipeline its not working

Answer 15 · 2022-12-08T02:23:03.000Z

@pawansharmaAccolite if it’s working locally probably you haven’t had the patch applied on the pipeline. you will need to either fork this repo and add the change or use something like https://www.npmjs.com/package/patch-package to check in locally applied patches to your app repo. Also if it’s the fix I mentioned, i think it will only work if you are building with Xcode 14/iOS 15 sdk, so make sure your pipeline is using the same setup

Answer 16 · 2022-12-08T04:43:45.000Z

@pawansharmaAccolite if it’s working locally probably you haven’t had the patch applied on the pipeline. you will need to either fork this repo and add the change or use something like https://www.npmjs.com/package/patch-package to check in locally applied patches to your app repo. Also if it’s the fix I mentioned, i think it will only work if you are building with Xcode 14/iOS 15 sdk, so make sure your pipeline is using the same setup

yes i face same issue in pipeline
but guys i am okay with that it does not work on ios 16 but it should work on ios 14 and 15 because. my pipeline using xcode 13.1 to deploy but still i don't understand why its not working in real device
i don't have access of real device which have ios 14 or 15. Only device i have is ios 16

that's why i asked for help that its really working in real device which have ios 14 or 15

Answer 17 · 2023-01-03T11:43:36.000Z

if you are still building with iOS 15 sdk, could try going into Orientation.m, and add
[UIViewController attemptRotationToDeviceOrientation];
below every
[[UIDevice currentDevice] setValue:[NSNumber numberWithInteger: ... ] forKey:@"orientation"];
that should work on iOS 16

@vinceplusplus Can you please share more info because i added with this way and its not working on released mode.

Answer 18 · 2023-01-19T05:42:29.000Z

Pipeline should have the same setup (macOS version (13.1) and xcode (14.2)), then it is working fine locally as well as on real devices.

Answer 19 · 2023-01-24T16:29:43.000Z

Any fixes?

Answer 20 · 2023-02-09T10:15:37.000Z

@codal-mpawar thanks for the fix.

FYI, this line causes occasional crashes :
UIWindowScene *scene = (UIWindowScene *)array[0];

Crash:
*** -[__NSArrayI_Transfer objectAtIndexedSubscript:]: index 0 beyond bounds for empty array

I think it's because I call this function when the app is still initialising.

Answer 21 · 2023-02-24T01:36:40.000Z

do you already have a solution for this?
Im also experiencing it huhuhu

Answer 22 · 2023-04-23T06:56:59.000Z

hey,guys,i have solved the problem for iOS 16 system,and it work fine in ios 16 system. in Orientation.m i add this code

1.0
//适配iOS16系统
-(void)adapterIOS16SystemWithOrientation:(UIInterfaceOrientationMask)Orientation{
if (@available(iOS 16.0, *)) {
UIWindow *originalKeyWindow = [[UIApplication sharedApplication] keyWindow];
if (originalKeyWindow != nil) {
[originalKeyWindow.rootViewController setNeedsUpdateOfSupportedInterfaceOrientations];
NSArray *array = [[[UIApplication sharedApplication] connectedScenes] allObjects];
UIWindowScene *scene = (UIWindowScene *)array[0];
UIWindowSceneGeometryPreferencesIOS *geometryPreferences = [[UIWindowSceneGeometryPreferencesIOS alloc] initWithInterfaceOrientations:Orientation];
[scene requestGeometryUpdateWithPreferences:geometryPreferences
errorHandler:^(NSError * _Nonnull error) {
NSLog(@"error---%@", error.description);
}];
}
}
}

2.0
and call where needed ,for example:
RCT_EXPORT_METHOD(lockToLandscapeRight)
{
#if DEBUG
NSLog(@"Locked to Landscape Right");
#endif
[[NSOperationQueue mainQueue] addOperationWithBlock:^ {
[Orientation setOrientation:UIInterfaceOrientationMaskLandscapeRight];

  //    // this seems counter intuitive
      [[UIDevice currentDevice] beginGeneratingDeviceOrientationNotifications];
        [[UIDevice currentDevice] setValue:[NSNumber numberWithInteger: UIInterfaceOrientationPortrait] forKey:@"orientation"];
      [[UIDevice currentDevice] setValue:[NSNumber numberWithInteger: UIInterfaceOrientationLandscapeRight] forKey:@"orientation"];
    [self adapterIOS16SystemWithOrientation:UIInterfaceOrientationMaskLandscapeRight];

}];
}
3.0 it work fine good luck

Answer 23 · 2023-04-26T19:42:14.000Z

do you already have a solution for this? Im also experiencing it huhuhu

@tipaniclayr30 Sorry you didn't tag me, so I didn't see your question. I just added a safeguard:

NSArray *array = [[[UIApplication sharedApplication] connectedScenes] allObjects];
if (array.count > 0) {
      UIWindowScene *scene = (UIWindowScene *)array[0];
      [UIViewController attemptRotationToDeviceOrientation];
       UIWindowSceneGeometryPreferencesIOS *geometryPreferences = [[UIWindowSceneGeometryPreferencesIOS alloc] initWithInterfaceOrientations:maskOrientation];
       [scene requestGeometryUpdateWithPreferences:geometryPreferences errorHandler:^(NSError * _Nonnull error) {}];
 }

Answer 24 · 2024-05-16T02:52:32.000Z

I've added: react-native-orientation-locker
then in file App.js
import Orientation from "react-native-orientation-locker";

useEffect(() => { Orientation.lockToPortrait(); return () => { Orientation.unlockAllOrientations(); }; }, []);