this.requestAnimationFrame is not a function appears when going in and out of fullscreen mode

I have gone through these following points

• Check latest documentation: https://docs.nativebase.io/
• Check for existing open/closed issues for a possible duplicate before creating a new issue: https://github.com/GeekyAnts/NativeBase/issues
• Use the latest NativeBase release: https://github.com/GeekyAnts/NativeBase/releases
• Check examples from NativeBase KitchenSink https://github.com/GeekyAnts/NativeBase-KitchenSink
• [] For discussion purpose make use of NativeBase Slack: http://slack.nativebase.io/
• [] For queries related to theme, check Theme Variables from Docs and live NativeBase Theme Editor http://nativebase.io/customizer/

Issue Description

When playing a video in our app and we go in and out of fullscreen mode the simulator throws the error this.requestAnimationFrame is not a function When using a prior version of native-base (2.13.12) this bug did not exist in our app. Once we upgraded to native-base version 2.13.13, the bug started to appear.

We’ve forked the Native-base repo and implemented the following workaround in the node_modules/native-base/src/basic/Tabs/index.js file:

And in the node_modules/native-base/dist/src/basic/Tabs/index.js file:

node, npm, react-native, react and native-base version, expo version if used, xcode version

node: 10.15.3 react-native: 0.60.0 react: 16.8.6 native-base: 2.13.13 xcode: 11.1 (11A1027)

Expected behaviour

Should not throw an error when going in and out of fullscreen mode

Actual behaviour

throws an error when going in and out of fullscreen mode

Steps to reproduce

Launch app and play a video Then click fullscreen button to play video in fullscreen mode

Is the bug present in both iOS and Android or in any one of them?

Yes, bug is present in iOS and Android.

Any other additional info which would help us debug the issue quicker.

###Important

If you want your issue to be looked at quicker, attach a snack reproducible with your issue. Makes it easier for us!

[elilambnz]

@tejparekh if you don’t want to mess around with manually changing files, you could temporarily import @dgeorgiev’s handy PR into npm in place of native-base: "native-base": "Healthyco/NativeBase#feature/fix-request-animation"

Or npm i/yarn add Healthyco/NativeBase#feature/fix-request-animation

Just remember to subscribe to PR #3218 changes and revert back to importing native-base once it’s merged in.

[dgeorgiev]

Hey, pushed a PR with a fix for this https://github.com/GeekyAnts/NativeBase/pull/3218