Use abs position when checking if element is visible in viewport #437
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
While implementing ember-infinity in my project I noticed that the
<InfinityLoader>
would be in view but would not load new records. One simple way of testing this was to have an endpoint that returns 1 record per page. ember-infinity loads the first page,<InfinityLoader>
remains in view, and no new records are loaded.This PR contains the fix I came up with which resolve the issue.
clientHeight
doesn't tell us anything about where theviewPortElem
is located in the window relative to wherethis.elem
is located. Using absolute values ensures that_debounceScrolledToBottom()
is always called when the<InfinityLoader>
component is in view.