-
Notifications
You must be signed in to change notification settings - Fork 728
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
[RFR] Infinite scroll: avoid to load multiple time at the same time #1322
Conversation
9b0cec7
to
d809f5a
Compare
package.json
Outdated
@@ -52,6 +52,7 @@ | |||
"babel-preset-stage-2": "^6.13.0", | |||
"chai": "^3.3.0", | |||
"css-loader": "^0.25.0", | |||
"diff": "^3.2.0", |
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.
^
are bad
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.
Even in dev deps ?
My opinion is:
~
for client projects^
is OK for open-source
@@ -1,10 +1,14 @@ | |||
import angular from 'angular'; | |||
|
|||
export default function maDatagridInfinitePagination($window, $document) { | |||
const isDownScrolling = wheelEvent => { |
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.
isScrollingDown
?
}; | ||
|
||
// Trigger the scroll at least once | ||
// In this case, we'll avoid that a pagination of a few element (< 10) |
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.
Hard to understand, can you rephrase this or have a longer explanation ?
Fix #681
Supersedes #1068
wheel
event