[Bug]: maxDynamicContentSize not enforced on keyboard open

[gkueny]

Version

v5

Platforms

iOS

What happened?

For a modal with dynamicSizing enabled (enableDynamicSizing). When we focus an input in the modal, it will not respect given maxDynamicContentSize.

Reproduction steps

• setup aBottomSheet with some BottomSheetTextInput
• set BottomSheet enableDynamicSizing props to true
• set a maxDynamicContentSize
• set keyboardBehavior to interactive
• focus Input

On Focus, keyboard will appear and BottomSheet modal might exceed given maxDynamicContentSize

Here is a video of the reproduction where I set maxDynamicContentSize to avoid modal to prevent from slipping under the phone's top edge

ScreenRecording_03-24-2026.17-20-26_1.MP4

Reproduction sample

https://snack.expo.dev/@gkuenyme/bottomsheet-reproduction---maxdynamiccontentsize-not-enforced-on-keyboard-open

[gkueny]

Maybe related to #2255

[github-actions]

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days.

[gkueny]

not stale

[krisgerhard]

@gkueny I ran into the same issue. Try this 👇 - worked for me.

diff --git a/node_modules/@gorhom/bottom-sheet/src/components/bottomSheet/BottomSheet.tsx b/node_modules/@gorhom/bottom-sheet/src/components/bottomSheet/BottomSheet.tsx
index fc52033..526acc1 100644
--- a/node_modules/@gorhom/bottom-sheet/src/components/bottomSheet/BottomSheet.tsx
+++ b/node_modules/@gorhom/bottom-sheet/src/components/bottomSheet/BottomSheet.tsx
@@ -312,10 +312,17 @@ const BottomSheetComponent = forwardRef<BottomSheet, BottomSheetProps>(
 
       // extended position with keyboard =
       // container height - (sheet height + keyboard height in root container)
+      // Patch (gorhom #2638): clamp so the sheet does not exceed maxDynamicContentSize
+      // when the keyboard pushes it up. Without this, the sheet's effective height
+      // grows by keyboardHeightInContainer, ignoring the configured cap.
       const keyboardHeightInContainer =
         animatedKeyboardState.get().heightWithinContainer;
+      const minPositionFromMaxSize =
+        maxDynamicContentSize !== undefined
+          ? Math.max(0, containerHeight - maxDynamicContentSize)
+          : 0;
       const extendedPositionWithKeyboard = Math.max(
-        0,
+        minPositionFromMaxSize,
         containerHeight -
           (animatedSheetHeight.value + keyboardHeightInContainer)
       );
@@ -348,6 +355,7 @@ const BottomSheetComponent = forwardRef<BottomSheet, BottomSheetProps>(
       animatedSheetHeight,
       isInTemporaryPosition,
       keyboardBehavior,
+      maxDynamicContentSize,
     ]);
     const {
       state: animatedScrollableState,
@@ -856,7 +864,17 @@ const BottomSheetComponent = forwardRef<BottomSheet, BottomSheetProps>(
           isInTemporaryPosition.value = true;
           const keyboardHeightInContainer =
             animatedKeyboardState.get().heightWithinContainer;
-          return Math.max(0, highestDetentPosition - keyboardHeightInContainer);
+          // Patch (gorhom #2638): clamp so the sheet does not exceed
+          // maxDynamicContentSize when the keyboard pushes it up.
+          const { containerHeight } = animatedLayoutState.get();
+          const minPositionFromMaxSize =
+            maxDynamicContentSize !== undefined
+              ? Math.max(0, containerHeight - maxDynamicContentSize)
+              : 0;
+          return Math.max(
+            minPositionFromMaxSize,
+            highestDetentPosition - keyboardHeightInContainer
+          );
         }
 
         /**
@@ -908,6 +926,7 @@ const BottomSheetComponent = forwardRef<BottomSheet, BottomSheetProps>(
         animatedHandleGestureState,
         animatedAnimationState,
         animatedKeyboardState,
+        animatedLayoutState,
         animatedPosition,
         animatedDetentsState,
         isInTemporaryPosition,
@@ -916,6 +935,7 @@ const BottomSheetComponent = forwardRef<BottomSheet, BottomSheetProps>(
         keyboardBlurBehavior,
         _providedIndex,
         android_keyboardInputMode,
+        maxDynamicContentSize,
       ]
     );

[github-actions]

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days.

[krisgerhard]

not stale

[github-actions]

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days.

[github-actions]

This issue was closed because it has been stalled for 5 days with no activity.