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 fixed - the UIScrollView will break on iOS10 when building by XCode9 #1896

Merged
merged 1 commit into from
Sep 24, 2017
Merged

Bug fixed - the UIScrollView will break on iOS10 when building by XCode9 #1896

merged 1 commit into from
Sep 24, 2017

Conversation

gameboyVito
Copy link
Contributor

This only happens when you are using [email protected], and build your APP by XCode9. When you run your APP in iOS10 or below, it will throw unrecognized selector error. The new iOS11 has new design on StatusBar which behaves weird on UIScrollView, so that the react-native team added this new selector to fix it. This PR will help to address this problem when you are also using the amazing react-native-navigation.

Here's the error details:

Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: '-[RCTCustomScrollView setContentInsetAdjustmentBehavior:]: unrecognized selector sent to instance 0x7ffab9891a00'

Reference:
facebook/react-native#15957
https://github.com/wix/react-native-navigation/issues/1886

This only happens when you are using [email protected], and your XCode is in version 9.0
@SudoPlz
Copy link
Collaborator

SudoPlz commented Sep 22, 2017

I had that problem w/ XCode9 and react-native 0.46.3 so it does not only happen on 0.48.3

@manishoo
Copy link

Can anyone please merge this ?

@gran33 gran33 merged commit 77b5c64 into wix:master Sep 24, 2017
@gran33
Copy link
Contributor

gran33 commented Sep 24, 2017

Thanks man!

junedomingo pushed a commit to junedomingo/react-native-navigation that referenced this pull request Sep 29, 2017
…de9 (wix#1896)

This only happens when you are using [email protected], and your XCode is in version 9.0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants