onDragStopped / onDragStarted are not properly `update`d

[Nek-12]

Inside this state, the values of onDragStarted and onDragStopped are passed downwards to a DisposableEffect. But that effect doesn't properly apply rememberUpdatedState and captures outdated versions of these lambdas in its block. As a result, the sortableList variable and all other captured variables in composition which are passed to the store in the code below are not correct.

modifier = Modifier.draggableHandle(
    enabled = state.sortingModeEnabled,
    onDragStarted = {
        haptics.performHapticFeedback(HapticFeedbackType.GestureThresholdActivate)
    },
    onDragStopped = {
        intent(FinishedReordering(sortableList))
        haptics.performHapticFeedback(HapticFeedbackType.GestureEnd)
    }
)

To fix this, properly call rememberUpdatedState on all variable references that are captured into/ passed outside the composition

[michalgwo]

I have the same problem when I want to read the index in onDragStarted and onDragStopped. The issue is reproducible on the demo app on SimpleReorderableLazyColumnScreen. You just need to add logging in onDragStarted and onDragStopped. As you can see in the video, the index of Item #0 stays 0, even when it's not 0 anymore. It looks like other elements (that are not dragged) get the updated index value.

reorderable.mp4