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

Add new infiniteScrollReset parameter to allow for easier list updates #277

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

tricinel
Copy link

This parameter is needed if you want to reuse the same infinite scroll directive and your items will completely change. A use case is a file browser where you navigate the folder structure and you want all the levels of the hierarchy to benefit from infinite scroll. Without a reset such as this, as soon as you scroll to the bottom of the list, subsequent lists that are not bigger than that will no longer have infinite scroll.

You can call the reset by adding an event listener and setting the reset to true. e.g.

<div infinite-scroll="addMoreItems()" infinite-scroll-listen-for-event="list:changed" infinite-scroll-reset="true">
  [...]
</div>

tricinel added 2 commits April 15, 2016 15:12
This parameter is needed if you want to reuse the same infinite scroll directive and your items will completely change. A use case is a file browser where you navigate the folder structure and you want all the levels of the hierarchy to benefit from infinite scroll. Without a reset such as this, as soon as you scroll to the bottom of the list, subsequent lists that are not bigger than that will no longer have infinite scroll.

You can call the reset by adding an event listener and setting the reset to true. e.g. <div infinite-scroll="addMoreItems()" infinite-scroll-listen-for-event="list:changed" infinite-scroll-reset="true">
  [...]
</div>
@graingert
Copy link
Collaborator

@tricinel please add tests and documentation for this change, and squash + rebase.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants