Skip to content

Commit 00189c5

Browse files
authored
Merge pull request #98 from leopittelli/master
added lazy load for whole container out of viewport
2 parents c24bc8c + 1402b33 commit 00189c5

File tree

2 files changed

+6
-0
lines changed

2 files changed

+6
-0
lines changed

src/LazyLoad.jsx

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -36,6 +36,8 @@ export default class LazyLoad extends Component {
3636

3737
add(window, 'resize', this.lazyLoadHandler);
3838
add(eventNode, 'scroll', this.lazyLoadHandler);
39+
40+
if (eventNode !== window) add(window, 'scroll', this.lazyLoadHandler);
3941
}
4042

4143
componentWillReceiveProps() {
@@ -104,6 +106,8 @@ export default class LazyLoad extends Component {
104106

105107
remove(window, 'resize', this.lazyLoadHandler);
106108
remove(eventNode, 'scroll', this.lazyLoadHandler);
109+
110+
if (eventNode !== window) remove(window, 'scroll', this.lazyLoadHandler);
107111
}
108112

109113
render() {

src/utils/inViewport.js

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,8 @@ export default function inViewport(element, container, customOffset) {
1919
bottom = top + window.innerHeight;
2020
right = left + window.innerWidth;
2121
} else {
22+
if (!inViewport(container, window, customOffset)) return false;
23+
2224
const containerPosition = getElementPosition(container);
2325

2426
top = containerPosition.top;

0 commit comments

Comments
 (0)