DrawModifierNode is not synced with LazyColumn scroll position about compose-multiplatform HOT 12 CLOSED

Sure, I updated the code to make the problem more visible.

And here is the video:

Set the speed to 0.5 to see that the drawn elements (yellow boxes) are not aligned to the list items when scrolling.

from compose-multiplatform.

It will definitely be in 1.7, possibly earlier in 1.6.11. You can also see it in our dev builds, which we release every few days.

from compose-multiplatform.

when scrolling, the position of the drawn icons (circles) is unstable relative to the items in the LazyColumn.

Looks like it's only on the top/bottom elements during overscrolling?

@elijah-semyonov, listState.layoutInfo.visibleItemsInfo doesn't take into account the overscroll effect.

from compose-multiplatform.

@igordmn No, the problem with the overscroll effect is a separate issue that also doesn't work on Android (there is a feature request that might help fix this).

This issue is about drawings done in DrawModifierNode not being synced with the scroll position of LazyColumn when scrolled in any position (with the overscroll effect disabled).

I also noticed that this actually works fine on the simulator - so there is no such problem in your video.
I can only reproduce the bug on a real device (iPhone 12 iOS 17.4.1).

from compose-multiplatform.

I can only reproduce the bug on a real device

Could you post a video with scrolling in the middle? In the video you posted it is difficult to distinguish the desync because of overscoll and the desync because of other reasons

from compose-multiplatform.

@elijah-semyonov, listState.layoutInfo.visibleItemsInfo doesn't take into account the overscroll effect.

@igordmn
That makes sense. Overscroll effect on iOS is based on applying Modifier.offset to the scroll content and tracks its own state, it doesn't know anything about the implementation of the particular scroll it's integrated with (whether it's a TextField scroll, .scrollable or LazyList.

Native iOS works with negative (or above the max value) scrolls. Such values are breaking common logic invariants, so supporting it in the same way as native iOS does it would be extremely intrusive.

The lag looks interesting though, did we include main-loop sequence fix in beta03?

from compose-multiplatform.

The lag looks interesting though, did we include JetBrains/compose-multiplatform-core#1260 in beta03?

Yes, it is included

from compose-multiplatform.

Possibly fixed by JetBrains/compose-multiplatform-core#1356; worth trying.

from compose-multiplatform.

@m-sasha it's indeed fixed by that fix.

from compose-multiplatform.

Could you check the one with the yellow squares too? It's much more visible there.

from compose-multiplatform.

Sure! It's not present there either:

from compose-multiplatform.

Nice, what version will the fix come in?

from compose-multiplatform.