FlatList Compatibility + fix for setState on unmount issue

[curtismenmuir]

Context

• I was looking to use this lib within a component being rendered as a FlatList item. When using a large list I started to see lots of 'react-native-text-ticker: could not calculate metrics', 'nodehandle_not_found' errors and the text-ticker was displaying weird behavior with text animations (The smallest list I seen this occur was ~30 items).

Change Description

• Added disabled field so that the animation can be stopped. I have used this with the FlatList's onViewableItemsChanged callback so that any item which is not currently on screen will not animate.
• Added makeCancellable() to allow promises to be cancelled in componentWillUnmount(). This will cleanup any unresolved promises to resolve the unmounted setState warning. resolve and reject have been nulled to help with GC. NOTE: This is not a perfect fix, but I believe that it is the best approach for now. More Details: https://github.com/reactjs/reactjs.org/pull/2562

[deanhet]

Good catch on the warning! Thanks for contributing and updating the docs too with the additional prop. I've tried your branch locally and everything seems to work like you said. Will merge and push out an update to the package now. 👍🏻

[dimbslmh]

@curtismenmuir

Warning: componentWillMount has been renamed, and is not recommended for use. See https://fb.me/react-unsafe-component-lifecycles for details.

• Move code with side effects to componentDidMount, and set initial state in the constructor.
• Rename componentWillMount to UNSAFEcomponentWillMount to suppress this warning in non-strict mode. In React 17.x, only the UNSAFE name will work. To rename all deprecated lifecycles to their new names, you can run npx react-codemod rename-unsafe-lifecycles in your project source folder.

Please update the following components: TextMarquee

[curtismenmuir]

@dimbslmh Good catch, updated initialization to be set in constructor. See new PR: https://github.com/deanhet/react-native-text-ticker/pull/66