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

Adds horizontal scrolling. #142

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Adds horizontal scrolling. #142

wants to merge 1 commit into from

Conversation

mhoffmeyerDC
Copy link

CoffeeScript implementation from the discussion in #69.

I followed the existing pattern for binding scope.infiniteScrollHorizontal to scrollHorizontal with $watch. I don't think this is needed since it's not going to change after init. Same for useDocumentBottom. Would either of these need to evaluated after load? Could we save a few listeners with just a simple assignment?

@graingert
Copy link
Collaborator

@mhoffmeyerDC please update the tests with this feature.

@ghost
Copy link

ghost commented Oct 19, 2016

Greetings! is this work in progress or horizontal infinite scroll will never happen?
cheers

@LoganLehman
Copy link

Any news on this? @mhoffmeyerDC - did you create the tests for these? @graingert - Would it be difficult to get this into ngInfiniteScroll soon? Even without @mhoffmeyerDC responding?

@keo64
Copy link

keo64 commented Apr 24, 2018

Would be great to include this feature to release, guys!

@yannakyp
Copy link

Much needed feature, would be great to add this into our next release :)

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.

5 participants