wamo/Qvitter
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Duplicate items appeared when scrolling to bottom of feed

Browse Source 
At least on the "notifications" feed page many users on quitter.es reported
that their notifications were duplicated when scrolling down.

It appears that qvitter.js fetches notices but does not check which notices
were already there, so I did a quick fix and set the feed to only fetch
notices which had a max_id of the last item _minus one_.

max_id is inclusive in GNU social.
This commit is contained in:
Mikael Nordfeldth 2015-01-19 15:41:52 +01:00
parent 259651e671
commit ce4b99e2a5
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
js/qvitter.js
View File

@ -1051,7 +1051,7 @@ $(window).scroll(function() {
} }
// normal streams // normal streams
else { else {
var getVars = qOrAmp(window.currentStream) + 'max_id=' + $('#feed-body').children('.stream-item').last().attr('data-quitter-id-in-stream'); var getVars = qOrAmp(window.currentStream) + 'max_id=' + ($('#feed-body').children('.stream-item').last().attr('data-quitter-id-in-stream')-1);
} }
display_spinner('#footer-spinner-container'); display_spinner('#footer-spinner-container');