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
Progressive Load: Scroll on Basic Renderer jumps to top while scrolling #4469
Comments
Seeing the same thing with the virtual renderer (tried the basic as well). All rows have the same height, so it shouldn't be related to any issues with variable row heights. |
You can't use progressive scroll and the basic render they incompatible. The basic renderer is extremely simple and will redraw the whole table when new data is added, resetting the position. If you have a scrolling table you should be using the virtual renderer. The basic renderer is only intended for very simple tables |
Describe the bug
Basic Vertical Renderer bounces to top when using Progressive Load Scroll. This happens when new data is fetched and added from the server.
Tabulator Info
Working Example
https://jsfiddle.net/qsk5zLwh/2/
To Reproduce
renderVertical:"basic"
andprogressiveLoad:"scroll"
Expected behavior
Smooth scrolling. Fetching the next page should not reset the scroll position.
Desktop:
Additional context
BasicVerticalRenderer
rerenderRows
clears all rows then renders them again. This shrinks the container element and resets the scroll position. Documentation for renderers states thatrerenderRows
should maintain scroll position.Have encountered scrolling issues using the default virtual dom renderer like #4353 and #3654 both of which are resolved by switching to the basic renderer.
I have used the suggestion in #4424 and made the basic renderer's
rerenderRows
remember thescrollTop
before clearing and then reapplying it after rendering to resolve the issue.The text was updated successfully, but these errors were encountered: