You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
If the user has a particularly tall screen, like a tablet in portrait mode, 1 loaded page may not fill their screen. When this happens ChatView does not call onInfiniteLoad when the you attempt to scroll.
I'm currently working around this using react-measure to trigger a load any time the height inside of ChatView is less than height outside ChatView. I'd prefer not to rely on significantly increasing page size to make this event unlikely but still plausible.
Maybe isPassedThreshold should detect the scroll section being too small to possibly trigger when scrollHeight < clientHeight and return true in that case?
Reproducing should be easy. Just open the demo and use Chrome's zoom feature to zoom out to 50% or smaller until the first page loads all at once, then try to scroll up. Using latest Chrome canary, this reproduces the issue for me.
The text was updated successfully, but these errors were encountered:
I've done some experimentation with onScroll and isPassedThreshold and it seems onScroll is never triggered if the div is too small to scroll when using passive event listeners.
If the user has a particularly tall screen, like a tablet in portrait mode, 1 loaded page may not fill their screen. When this happens ChatView does not call onInfiniteLoad when the you attempt to scroll.
I'm currently working around this using react-measure to trigger a load any time the height inside of ChatView is less than height outside ChatView. I'd prefer not to rely on significantly increasing page size to make this event unlikely but still plausible.
Maybe isPassedThreshold should detect the scroll section being too small to possibly trigger when scrollHeight < clientHeight and return true in that case?
Reproducing should be easy. Just open the demo and use Chrome's zoom feature to zoom out to 50% or smaller until the first page loads all at once, then try to scroll up. Using latest Chrome canary, this reproduces the issue for me.
The text was updated successfully, but these errors were encountered: