-
Notifications
You must be signed in to change notification settings - Fork 155
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
Flush debounceIncreasePool on loadPage if scrolling #2131
Conversation
@@ -345,7 +345,9 @@ | |||
filters: this._mapFilters(), | |||
parentItem: cache.parentItem | |||
}; | |||
|
|||
if (this.__scrollingToPendingIndex && this._debounceIncreasePool) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This new __scrollingToPendingIndex
property feels a bit hacky? Is there some other justification for adding it other than making that one data-provider test pass / should the increase pool debouncer not be flushed while the grid is scrolling to pending index?
I took a look at the failing test case and it seems to pass because of timing. If I wrap the last expect inside of a timeout of 1 second, it also fails on master due to a too large call count. I don't necessarily see it as a problem to increase the call count expectation to make the test pass with the new change in timing.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
8c6c954
to
d9cb509
Compare
Fixes #2107