search

davidbauer / Instacurate

Turn your Twitter timeline into a personalised news site, in an instant. Fetches links from your timeline and displays them in a discovery friendly design.
http://www.instacurate.com
113 stars 29 forks source link

Display message while loading new teasers after scrolling #81

Closed oliverwehn closed 11 years ago

davidbauer commented 11 years ago

Can't reproduce it. What's your example and setup?

oliverwehn commented 11 years ago

Seems to work perfectly when you instacurate a user’s timeline. Maybe an issue with hashtags only.

davidbauer commented 11 years ago

The scroll function doesn't seem to be the issue, works fine with input #ddj. But we only fetch 100 tweets with the hashtag, so if a lot of those tweets don't contain a link, we don't get a lot of teasers for instacurate.

oliverwehn commented 11 years ago

Ok, I get it. The scroll loading feels a bit weird this way as you may see no effect reaching page bottom. Maybe there sould be a message like “Fetches 100 more tweets, found 4 more links.” shown at the bottom before inserting the new teasers, if there are some.

davidbauer commented 11 years ago

You mean in the same way we have the message saying "Looking for tweeted links" first?

oliverwehn commented 11 years ago

Yeah, would make sense to display the message everytime the script is actually looking for tweeted links. And give some additional feedback that tells you the result of the action–if there are links found to be shown or not. Like: Display “Looking for tweets” -> display “Found x (more) tweets which contain y links” -> show/append teasers.

davidbauer commented 11 years ago

Will try to implement it this way.

davidbauer commented 11 years ago

Pushed a solution for this. What do you think, @oliverwehn? Positioning could be improved, especially on mobile.