fix: prevent crash when resizing container height on Android

[gorhom]

close #251

Motivation

for some reason when when passing a parameter from an array, in android it will crash

// CRASH
requestAnimationFrame(() => runOnUI(animateToPoint)(snapPoints[currentIndexRef.current]));

// FIX
const newSnapPoint = snapPoints[currentIndexRef.current];
requestAnimationFrame(() => runOnUI(animateToPoint)(newSnapPoint));

Installation

yarn add ssh://[email protected]:gorhom/react-native-bottom-sheet#fix/prevent-crash-on-container-height-resizing

[gorhom]

hi @tankers746, could you please test this pr :)

[tankers746]

Thanks thats fixed the issue
However the style change to BottomSheetContainer.tsx is now causing the sheet to be cut off. This is however only happening on my device with a notch.

[gorhom]

the container will always align to the top, and that what will happened when you override its height

[tankers746]

This is a problem with the automatic container height.

However I can see now this is just a case of RN not playing nice with my translucent status bar, for some reason the onLayout height in BottomSheetContainer is excluding the height of the status bar.

The change above does change the existing behaviour but I can work around it.

[gorhom]

if you provide a container height it will no longer do automatic layout measuring