fix(android): remeasure coordinator layout on configuration change

Author: lodev09

android/src/main/java/com/lodev09/truesheet/TrueSheetView.kt

@@ -278,19 +278,8 @@ class TrueSheetView(private val reactContext: ThemedReactContext) :
       // Attach coordinator to the root container
       rootContainerView = findRootContainerView()
       viewController.coordinatorLayout?.let { coordinator ->
-        rootContainerView?.let { container ->
-          container.addView(coordinator)
-
-          // Manually measure and layout for React Native views (they don't layout native children)
-          container.post {
-            if (container.width > 0 && container.height > 0) {
-              val widthSpec = View.MeasureSpec.makeMeasureSpec(container.width, View.MeasureSpec.EXACTLY)
-              val heightSpec = View.MeasureSpec.makeMeasureSpec(container.height, View.MeasureSpec.EXACTLY)
-              coordinator.measure(widthSpec, heightSpec)
-              coordinator.layout(0, 0, container.width, container.height)
-            }
-          }
-        }
+        rootContainerView?.addView(coordinator)
+        coordinator.post { measureCoordinatorLayout() }
       }
 
       // Register with observer to track sheet stack hierarchy
@@ -447,6 +436,7 @@ class TrueSheetView(private val reactContext: ThemedReactContext) :
 
   override fun viewControllerDidChangeSize(width: Int, height: Int) {
     updateState(width, height)
+    measureCoordinatorLayout()
   }
 
   override fun viewControllerWillFocus() {
@@ -491,6 +481,19 @@ class TrueSheetView(private val reactContext: ThemedReactContext) :
 
   // ==================== Private Helpers ====================
 
+  private fun measureCoordinatorLayout() {
+    val coordinator = viewController.coordinatorLayout ?: return
+    val width = viewController.screenWidth
+    val height = viewController.realScreenHeight
+
+    if (width > 0 && height > 0) {
+      val widthSpec = View.MeasureSpec.makeMeasureSpec(width, View.MeasureSpec.EXACTLY)
+      val heightSpec = View.MeasureSpec.makeMeasureSpec(height, View.MeasureSpec.EXACTLY)
+      coordinator.measure(widthSpec, heightSpec)
+      coordinator.layout(0, 0, width, height)
+    }
+  }
+
   /**
    * Find the root container view for presenting the sheet.
    * This traverses up the view hierarchy to find the content view (android.R.id.content)

android/src/main/java/com/lodev09/truesheet/TrueSheetViewController.kt

@@ -361,6 +361,13 @@ class TrueSheetViewController(private val reactContext: ThemedReactContext) :
     }
   }
 
+  override fun coordinatorLayoutDidChangeConfiguration() {
+    if (!isPresented) return
+
+    updateStateDimensions()
+    sheetView?.let { emitChangePositionDelegate(it.top, realtime = false) }
+  }
+
   // =============================================================================
   // MARK: - TrueSheetDimViewDelegate
   // =============================================================================
@@ -664,15 +671,7 @@ class TrueSheetViewController(private val reactContext: ThemedReactContext) :
       animate = isPresented
     )
 
-    val offset = if (expandedOffset == 0) topInset else 0
-    val newHeight = realScreenHeight - expandedOffset - offset
-    val newWidth = minOf(screenWidth, DEFAULT_MAX_WIDTH.dpToPx().toInt())
-
-    if (lastStateWidth != newWidth || lastStateHeight != newHeight) {
-      lastStateWidth = newWidth
-      lastStateHeight = newHeight
-      delegate?.viewControllerDidChangeSize(newWidth, newHeight)
-    }
+    updateStateDimensions(expandedOffset)
 
     if (isPresented) {
       setStateForDetentIndex(currentDetentIndex)
@@ -922,6 +921,19 @@ class TrueSheetViewController(private val reactContext: ThemedReactContext) :
   // MARK: - Detent Helpers
   // =============================================================================
 
+  private fun updateStateDimensions(expandedOffset: Int? = null) {
+    val offset = expandedOffset ?: (realScreenHeight - detentCalculator.getDetentHeight(detents.last()))
+    val topOffset = if (offset == 0) topInset else 0
+    val newHeight = realScreenHeight - offset - topOffset
+    val newWidth = minOf(screenWidth, DEFAULT_MAX_WIDTH.dpToPx().toInt())
+
+    if (lastStateWidth != newWidth || lastStateHeight != newHeight) {
+      lastStateWidth = newWidth
+      lastStateHeight = newHeight
+      delegate?.viewControllerDidChangeSize(newWidth, newHeight)
+    }
+  }
+
   fun translateSheet(translationY: Int) {
     val sheet = sheetView ?: return
 
@@ -966,22 +978,6 @@ class TrueSheetViewController(private val reactContext: ThemedReactContext) :
     (getTag(R.id.react_test_id) as? String)?.let { info.viewIdResourceName = it }
   }
 
-  override fun onSizeChanged(w: Int, h: Int, oldw: Int, oldh: Int) {
-    super.onSizeChanged(w, h, oldw, oldh)
-
-    if (w == oldw && h == oldh) return
-    if (!isPresented) return
-
-    // Skip reconfiguration if expanded and only height changed (e.g., keyboard)
-    if (h + topInset >= screenHeight && isExpanded && oldw == w) return
-
-    post {
-      setupSheetDetents()
-      positionFooter()
-      sheetView?.let { emitChangePositionDelegate(it.top, realtime = false) }
-    }
-  }
-
   // =============================================================================
   // MARK: - RootView Touch Handling
   // =============================================================================

android/src/main/java/com/lodev09/truesheet/core/TrueSheetCoordinatorLayout.kt

@@ -2,6 +2,7 @@ package com.lodev09.truesheet.core
 
 import android.annotation.SuppressLint
 import android.content.Context
+import android.content.res.Configuration
 import android.view.MotionEvent
 import android.view.ViewConfiguration
 import android.view.ViewGroup
@@ -12,6 +13,7 @@ import com.facebook.react.uimanager.ReactPointerEventsView
 
 interface TrueSheetCoordinatorLayoutDelegate {
   fun coordinatorLayoutDidLayout(changed: Boolean)
+  fun coordinatorLayoutDidChangeConfiguration()
 }
 
 /**
@@ -53,6 +55,11 @@ class TrueSheetCoordinatorLayout(context: Context) :
     delegate?.coordinatorLayoutDidLayout(changed)
   }
 
+  override fun onConfigurationChanged(newConfig: Configuration?) {
+    super.onConfigurationChanged(newConfig)
+    delegate?.coordinatorLayoutDidChangeConfiguration()
+  }
+
   override val pointerEvents: PointerEvents
     get() = PointerEvents.BOX_NONE
 

example/shared/src/screens/MapScreen.tsx

@@ -192,11 +192,11 @@ const MapScreenInner = ({
             onPress={() => sheetRef.current?.dismiss()}
           />
         </View>
-        <BasicSheet ref={basicSheet} onNavigateToModal={onNavigateToModal} />
-        <PromptSheet ref={promptSheet} />
-        <ScrollViewSheet ref={scrollViewSheet} />
-        <GestureSheet ref={gestureSheet} />
       </ReanimatedTrueSheet>
+      <BasicSheet ref={basicSheet} onNavigateToModal={onNavigateToModal} />
+      <PromptSheet ref={promptSheet} />
+      <ScrollViewSheet ref={scrollViewSheet} />
+      <GestureSheet ref={gestureSheet} />
       <FlatListSheet ref={flatListSheet} />
     </View>
   );