-
Notifications
You must be signed in to change notification settings - Fork 82
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
iOS 7 Compatibility #69
Comments
Ah it seems this is because its part of iOS 8. http://stackoverflow.com/questions/28113849/ios-swift-uinavigationcontroller-hidesbarsonswipe-error |
There are also 2 classes used that are not available on iOS 7: |
Hi |
Ok, will it be something you will consider addressing when merging with react-native-navigation? |
It's currently not a part of our road map. |
Seems a shame, as its minimal code changes and has no impact on later iOS versions (that I am aware of) - I am targeting iOS 7 as a minimum in my app, same as React Native. |
This is targeting less than 3% of the iOS devices in the world (which means less then 1.5% total), and will cost us more development time (also each feature will have to pass QA on 1 more device). Furthermore we will have iOS 10 soon, which means it will drop even further. We have to make compromises for our fast velocity and I think this is a no brainer. Same goes to supporting Android below api15.. Just too little market share to make it worth our time and resources. |
Fair enough, I understand your position. Thanks for taking the time to explain. |
I'm getting an error on running my app on device.
I can't find the reference to setHidesBarsOnSwipe anywhere - could this be getting called by iOS?
iPhone 4, iOS v 7.1.2
The text was updated successfully, but these errors were encountered: