In the mobile browser, it will shake when scrolling to the top or bottom.

[chenhebing]

Such as the title, list will shake when scrolling to the top or bottom, but react-virtualized is not happened.

You can view the video below.

video url

[bvaughn]

This looks like Safari elastic scrolling and not something this library in particular is doing.

[blenoski]

I am observing similar behavior on Safari iOS (not seeing in Chrome iOS). FWIW, found a commit from react-virtualized that seems like it was related to this problem: https://github.com/bvaughn/react-virtualized/pull/616/files

[bvaughn]

Want to try that out in react-window @blenoski ?

[blenoski]

@bvaughn
I will give it a try and report back to the issue.

I also have to update my comment 2 back, I was able to reproduce on Chrome iOS. Attaching a screen capture from iPhone7 for reference:

[bvaughn]

Cool, thanks! 😁

[blenoski]

Update:
I did some quick hacking and was able to confirm that clamping scrollTop to [0, totalHeight - lastRowHeight] eliminates the observed flickering on iOS (this was in the _onScrollVertical function in createListComponent.js

It does require knowing the current height of the innerContainer element plus the height of the last row which I imagine will be the tricky part. Also, assuming similar issues would exist for horizontal scrolling but have not yet been able to reproduce the flickering there.

Anyhow, going to pull down the repo and dig into the code a little to see if I can figure this out :)

[blenoski]

Update (cc. @bvaughn)

Have a PR up here (#250).
Scoped the PR to just vertical scrolling on lists but I was able to verify it removes the issue shown in the GIF three posts back.

[EyMaddis]

I just noticed the same. This issue is basically this old react-virtualized issue.

[bvaughn]

Worth noting that the linked RV commit was reverted in bvaughn/react-virtualized#616 so the long-standing RV solution to this is just to prevent scroll offsets less than 0. I think I'm comfortable making that change to RW as well, although I'll need to make sure it doesn't break RTL.

Also may be worth mentioning this comment:

I suspect that the bug lies somewhere within the iOS Webkit code and handling of the scroll bouncing events, but it is unclear. Disabling the bouncing on the scroll view at the wkwebview level does fixes the problem and produces a nice scrolling solution.

[bvaughn]

Bug fix released in 1.8.2 - https://github.com/bvaughn/react-window/blob/master/CHANGELOG.md#182