-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
DrawModifierNode is not synced with LazyColumn scroll position #4740
Comments
Looks like it's only on the top/bottom elements during overscrolling? Screen.Recording.2024-05-03.at.16.32.01.mov@elijah-semyonov, |
@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 I also noticed that this actually works fine on the simulator - so there is no such problem in your video. |
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 |
Sure, I updated the code to make the problem more visible. And here is the video: IMG_0012.MP4Set the speed to 0.5 to see that the drawn elements (yellow boxes) are not aligned to the list items when scrolling. |
@igordmn Native iOS works with negative (or above the max value) scrolls. Such values are breaking The lag looks interesting though, did we include main-loop sequence fix in beta03? |
Yes, it is included |
Possibly fixed by JetBrains/compose-multiplatform-core#1356; worth trying. |
@m-sasha it's indeed fixed by that fix. RPReplay_Final1715848901.MP4 |
Could you check the one with the yellow squares too? It's much more visible there. |
Sure! It's not present there either: RPReplay_Final1715932651.MP4 |
Nice, what version will the fix come in? |
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. |
Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks. |
Describe the bug
Drawing done in
drawBehind
/drawWithCache
/drawWithContent
is not synced with the LazyColumn scroll position.Please see the video attached: when scrolling, the position of the drawn icons (circles) is unstable relative to the items in the LazyColumn.
The icons are drawn using the
drawBehind
modifier.Affected platforms
Versions
To Reproduce
Here is a code snippet to reproduce.
Steps and/or the code snippet to reproduce the behavior:
Expected behavior
Drawing done in
drawBehind
/drawWithCache
/drawWithContent
is synced with the LazyColumn scroll position.Video
IMG_0006.MP4
Additional context
Works fine on Android.
The text was updated successfully, but these errors were encountered: