IllegalStateException with BottomSheetScaffold when sheetState is expanded after leaving and naving back to screen [277774594]

Fixed

Bug

Status Update

No update yet.

Description

lu...@gmail.com

created issue #1

Apr 12, 2023 12:49PM

Jetpack Compose version: 1.5.0-alpha01 Jetpack Compose component used: 1.5.0-alpha01 Android Studio Build: Android Studio Electric Eel | 2022.1.1 Patch 2 Kotlin version: 18.10

Steps to Reproduce or Code Sample to Reproduce: Create a BottomSheetScaffold with a NavHost inside sheetContent, if we laeve the screen with drawer expanded and nav back to screen with BottomSheetScaffold, app crash

Checkout following sample: https://github.com/Guimareshh/ComposePlayground/tree/feature/bottom-navigation
Launch the app, open Drawer Tab from bottomBar
Expand Drawer, open Explore Tab from bottomBar
Nav back to Drawer Tab from bottomBar: app crash

Stack trace (if applicable):

E/AndroidRuntime: FATAL EXCEPTION: main
    Process: com.compose.playground, PID: 15308
    java.lang.IllegalStateException: The offset was read before being initialized. Did you access the offset in a phase before layout, like effects or composition?
        at androidx.compose.material.SwipeableV2State.requireOffset(SwipeableV2.kt:208)
        at androidx.compose.material.BottomSheetState.requireOffset(BottomSheetScaffold.kt:159)
        at androidx.compose.material.BottomSheetScaffoldKt$BottomSheetScaffold$child$1$3$1.invoke(BottomSheetScaffold.kt:369)
        at androidx.compose.material.BottomSheetScaffoldKt$BottomSheetScaffold$child$1$3$1.invoke(BottomSheetScaffold.kt:369)
        at androidx.compose.material.BottomSheetScaffoldKt$BottomSheetScaffoldLayout$1$1.invoke-0kLqBqw(BottomSheetScaffold.kt:495)
        at androidx.compose.material.BottomSheetScaffoldKt$BottomSheetScaffoldLayout$1$1.invoke(BottomSheetScaffold.kt:488)
        at androidx.compose.ui.layout.LayoutNodeSubcompositionsState$createMeasurePolicy$1.measure-3p2s80s(SubcomposeLayout.kt:598)
        at androidx.compose.ui.node.InnerNodeCoordinator.measure-BRTryo0(InnerNodeCoordinator.kt:103)
        at androidx.compose.ui.node.LayoutNodeLayoutDelegate$performMeasure$2.invoke(LayoutNodeLayoutDelegate.kt:1090)
        at androidx.compose.ui.node.LayoutNodeLayoutDelegate$performMeasure$2.invoke(LayoutNodeLayoutDelegate.kt:1086)
        at androidx.compose.runtime.snapshots.Snapshot$Companion.observe(Snapshot.kt:2200)
        at androidx.compose.runtime.snapshots.SnapshotStateObserver$observeReads$1$1.invoke(SnapshotStateObserver.kt:234)
        at androidx.compose.runtime.snapshots.SnapshotStateObserver$observeReads$1$1.invoke(SnapshotStateObserver.kt:230)
        at androidx.compose.runtime.SnapshotStateKt__DerivedStateKt.observeDerivedStateRecalculations(DerivedState.kt:341)
        at androidx.compose.runtime.SnapshotStateKt.observeDerivedStateRecalculations(Unknown Source:1)
        at androidx.compose.runtime.snapshots.SnapshotStateObserver.observeReads(SnapshotStateObserver.kt:230)
        at androidx.compose.ui.node.OwnerSnapshotObserver.observeReads$ui_release(OwnerSnapshotObserver.kt:120)
        at androidx.compose.ui.node.OwnerSnapshotObserver.observeMeasureSnapshotReads$ui_release(OwnerSnapshotObserver.kt:107)
        at androidx.compose.ui.node.LayoutNodeLayoutDelegate.performMeasure-BRTryo0(LayoutNodeLayoutDelegate.kt:1086)
        at androidx.compose.ui.node.LayoutNodeLayoutDelegate.access$performMeasure-BRTryo0(LayoutNodeLayoutDelegate.kt:36)
        at androidx.compose.ui.node.LayoutNodeLayoutDelegate$MeasurePassDelegate.remeasure-BRTryo0(LayoutNodeLayoutDelegate.kt:342)
        at androidx.compose.ui.node.LayoutNodeLayoutDelegate$MeasurePassDelegate.measure-BRTryo0(LayoutNodeLayoutDelegate.kt:321)
        at androidx.compose.foundation.layout.BoxKt$boxMeasurePolicy$1.measure-3p2s80s(Box.kt:115)
        at androidx.compose.ui.node.InnerNodeCoordinator.measure-BRTryo0(InnerNodeCoordinator.kt:103)
        at androidx.compose.ui.graphics.SimpleGraphicsLayerModifier.measure-3p2s80s(GraphicsLayerModifier.kt:635)
        at androidx.compose.ui.node.LayoutModifierNodeCoordinator.measure-BRTryo0(LayoutModifierNodeCoordinator.kt:155)
        at androidx.compose.foundation.layout.FillModifier.measure-3p2s80s(Size.kt:658)
        at androidx.compose.ui.node.BackwardsCompatNode.measure-3p2s80s(BackwardsCompatNode.kt:323)
        at androidx.compose.ui.node.LayoutModifierNodeCoordinator.measure-BRTryo0(LayoutModifierNodeCoordinator.kt:155)
        at androidx.compose.ui.node.LayoutNodeLayoutDelegate$performMeasure$2.invoke(LayoutNodeLayoutDelegate.kt:1090)
        at androidx.compose.ui.node.LayoutNodeLayoutDelegate$performMeasure$2.invoke(LayoutNodeLayoutDelegate.kt:1086)
        at androidx.compose.runtime.snapshots.Snapshot$Companion.observe(Snapshot.kt:2200)
        at androidx.compose.runtime.snapshots.SnapshotStateObserver$observeReads$1$1.invoke(SnapshotStateObserver.kt:234)
        at androidx.compose.runtime.snapshots.SnapshotStateObserver$observeReads$1$1.invoke(SnapshotStateObserver.kt:230)
        at androidx.compose.runtime.SnapshotStateKt__DerivedStateKt.observeDerivedStateRecalculations(DerivedState.kt:341)

Comments

js...@google.com <js...@google.com> #2Apr 12, 2023 03:20PM

Looks related to b/266780234 except that this reproducer shows the error in 1.5.0-alpha01 (that bug claims to be fixed in 1.5.0-alpha01)

jo...@google.com <jo...@google.com> #3Apr 17, 2023 08:01AM

Marked as fixed.

Thanks! b/266780234 is about ModalBottomSheetLayout and was addressed in 1.5.0-alpha01.

This issue in BottomSheetScaffold has been fixed in aosp/2517255 and the fix will be released as part of the next release.

Issue 277774594

Description

Issue summary

Comments

js...@google.com <js...@google.com> #2Apr 12, 2023 03:20PM

jo...@google.com <jo...@google.com> #3Apr 17, 2023 08:01AM

Add comment

Issue metadata