Skip to content
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

[Firefox/IE] - OnDemandGrid - Wrong scrollbar length on large dataset, unreliable getScrollPosition/scrollTo #1384

Open
TiBeN opened this issue Jun 26, 2017 · 0 comments

Comments

@TiBeN
Copy link

TiBeN commented Jun 26, 2017

Up to a certain amount of total rows, the scrollbar does not reflect the length of the total dataset but, i think, only loaded data rows.

When the scrollbar is put at the bottom, this makes the grid to go to the end of the loaded results, trigger the load of next following rows then move the scrollbar at the center.

This makes impossible to "navigate" quickly at the end of the dataset and, worst in our scenario, makes dgrid.getScrollPosition() and dgrid.scrollTo() totally unreliable. For example, calling many times
dgrid.scrollTo({y: 5000}) makes each times the grid to move to an undeterministic position.

This can be simply reproduced with dgrid/test/OnDemand_promises.html by replacing at line 68:

for (i = 1; i <= 200; i++) {

by

for (i = 1; i <= 800000; i++) {

I hope this will not freeze your computer, because the dstore is memory, this is memory consuming. But this is quickest hack i found to reproduce the bug.

800000 is the value that makes firefox 54 on Linux and Internet Explorer 11 to behave incorrectly for this test. It has been noticed in Firefox 49 in Windows with a value of 1200000. Chrome does not seems to suffer from this bug (or it is more tolerant?).

This issue has been observed with dgrid 1.1.0 and 1.2.0

In our project this value is different (maybe simply because data is different).

@TiBeN TiBeN changed the title [Firefox/IE] - OnDemandGrid - Wrong scrollbar length on large dataset [Firefox/IE] - OnDemandGrid - Wrong scrollbar length on large dataset, unreliable getScrollPosition/scrollTo Jun 26, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant