Update comment

This commit is contained in:
RMidhunSuresh 2021-11-03 14:17:12 +05:30
parent d0b34ef2ca
commit dab3ba9f74

View file

@ -64,7 +64,6 @@ export class TimelineViewModel extends ViewModel {
so don't count the GapTile as an update. so don't count the GapTile as an update.
Usually happens when onScroll() is triggered by a non-timeline UI change, Usually happens when onScroll() is triggered by a non-timeline UI change,
eg: SessionStatusView being rendered eg: SessionStatusView being rendered
Also don't count updates from the user as these may update without triggering onScroll.
*/ */
hasSeenUpdate = true; hasSeenUpdate = true;
} }
@ -92,7 +91,12 @@ export class TimelineViewModel extends ViewModel {
this._watchMap.set(gapTile, false); this._watchMap.set(gapTile, false);
return; return;
} }
if (!hasSeenUpdate || !this._watchMap.get(gapTile)) { /*
We may have seen an update, but did the corresponding call to watchForGapFill from
that update already return?
*/
const hasNextInvocationReturned = !this._watchMap.get(gapTile);
if (!hasSeenUpdate || hasNextInvocationReturned) {
this.watchForGapFill(gapTile.notifyVisible(), gapTile, depth + 1); this.watchForGapFill(gapTile.notifyVisible(), gapTile, depth + 1);
} }
} }