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

Infinite scroll only works if there are enough results to fill the screen (and create a scrollbar) #84

Open
ohadschn opened this issue Nov 26, 2017 · 2 comments

Comments

@ohadschn
Copy link

Say I filter my feed by something (text or mix/track) and few (or no) results come up.

If there aren't enough results to fill the current screen and create a scrollbar, I can't invoke infinite scrolling to try and load more.

Perhaps a manual "load more" button would be the easiest way around this.

@chriszhangusc
Copy link
Collaborator

Hi @ohadschn, could you please provide the case where you can not load more content? I can't reproduce it.

@ohadschn
Copy link
Author

ohadschn commented Nov 27, 2017

Sure:

  1. Go to the main site and put drumcode in the search filter. For me it finds 3 results.
  2. Since there is no scrollbar (only 3 results), you can't scroll down to load more.
  3. Now clear the filter, scroll down (invoke infinite scroll a few times), and put drumcode in the filter again
  4. Note how you have more results now. Ideally, you should have been able to invoke the infinite scroll even when you only had 3 results.

This also happened to me when I filtered my home feed for mixes because I had too many reposts of regular tracks on my feed. So I had to cancel the mix filter, scroll down a few times, then bring it back, and possibly repeat that until I had enough mixes to overflow the on-screen list and create a scrollbar.

The point is, it doesn't make sense IMO for the ability to scroll infinitely to be dependent on the existence of a scrollbar (which is dependent on your zoom level, window size, etc).

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

No branches or pull requests

2 participants