nowinandroid icon indicating copy to clipboard operation
nowinandroid copied to clipboard

Fix the scroll bar thumb jumping when at the end of the LazyStaggeredList with more than one lane and improve smothness of the scroll

Open rodrigodiasferreira opened this issue 1 year ago • 5 comments

What I have done and why Bug fix: scroll bar thumb jumping when at the end of the LazyStaggeredList with more than one lane, issue described in: https://github.com/android/nowinandroid/issues/1526.

rodrigodiasferreira avatar Jul 15 '24 19:07 rodrigodiasferreira

I also transfered the improvement on the smoothness of the scroll from the PR: https://github.com/android/nowinandroid/pull/1550, into this one, as the generic abstraction was not accepted.

rodrigodiasferreira avatar Aug 11 '24 16:08 rodrigodiasferreira

Could you add the result screen video?

Jaehwa-Noh avatar Aug 11 '24 23:08 Jaehwa-Noh

Hi @Jaehwa-Noh , sure.

Before: before.webm

After: after.webm

rodrigodiasferreira avatar Aug 12 '24 03:08 rodrigodiasferreira

Changes look good to me, I don't work on this project anymore so leaving to the maintainers to approve and merge.

tunjid avatar Aug 12 '24 06:08 tunjid

Really nice workaround. I have just checked and found out that fix from Google was included in androidx.compose.foundation:foundation:1.8.0-alpha01.

Now we are using 1.8.0-alpha07, so we can just retest the latest code to make sure the issue no longer exists.

anhtuannd avatar Jan 13 '25 13:01 anhtuannd