fix: don't wrap content with BottomSheetView when enableDynamicSizing is true

[jeanregisser]

Hey,

This PR is to get the discussion started on the following problem:
We're using react-navigation-bottom-sheet with enableDynamicSizing set to true but want to use BottomSheetScrollView (or other Gorhom components: BottomSheetFlatList, BottomSheetSectionList, etc)
within the screen to get proper sizing and all features that go with it (scroll down to dismiss, etc).

Currently with the latest version of react-navigation-bottom-sheet (0.3.2), this causes sizing issues and blank renders.
I tracked it down to having both BottomSheetView (in this lib) and BottomSheetScrollView (in our own screen) nested.
They somehow conflict.

As a workaround I've patched it to remove the BottomSheetView wrapping: https://github.com/valora-inc/wallet/blob/bbd0e39dcc1420a704fa866113994e9a20c3bfa4/patches/%40th3rdwave%2Breact-navigation-bottom-sheet%2B0.3.2.patch

This works pretty well for us, but we must ensure the screens use one of the Gorhom components.

I'm not sure about the "right" solution to avoid this patch. Here are possible options:

1. force the consumers of this lib to wrap their screens with the Gohrom component they need
2. iterate over the children to see if one of them is a Gohrom component
   • not sure it would work for all cases, especially if the screen nests another bottom sheet that doesn't need to be a screen
3. add a new property for devs to set when they don't want the default wrapping with BottomSheetView
4. other?

Looking forward to finding a solution that will work for all users of this lib.

[Rag0n]

Thanks for the PR - I'm having the same issue and wrapping everything in BottomSheetView is definitely not a right approach.

Hey @janicduplessis, what do you think?

[vipierozan99]

same issue here, maybe we could add a sheetWrapper prop and where you pass in the Gohrom component?

<BottomSheet
  screenOptions={{
    sheetWrapper: BottomSheetView
  }}
/>

[vipierozan99]

Also, would it make sense to add this? as it seems like this lib assumes the old default of undefined=false

enableDynamicSizing = true, // same default as lib

https://github.com/th3rdwave/react-navigation-bottom-sheet/blob/954bcf38fa4503f018585d6cca1d854cab07841f/src/BottomSheetView.tsx#L141