Skip to content

Commit bdb0a58

Browse files
committed
remove single item from top: Padding cache logic
1 parent c12a2e6 commit bdb0a58

File tree

3 files changed

+6
-6
lines changed

3 files changed

+6
-6
lines changed

src/modules/adapter.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -163,7 +163,7 @@ class Adapter {
163163
}
164164
// remove single item
165165
if(!newItems.length) {
166-
this.viewport.removeCacheItem(index, true);
166+
this.viewport.removeCacheItem(index, index === this.buffer.minIndex);
167167
}
168168
}
169169

src/modules/padding.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,9 +15,9 @@ class CacheProto {
1515
this.sort((a, b) => ((a.index < b.index) ? -1 : ((a.index > b.index) ? 1 : 0)));
1616
}
1717

18-
remove(argument) {
18+
remove(argument, _isTop) {
1919
const index = argument % 1 === 0 ? argument : argument.scope.$index;
20-
const isTop = argument % 1 === 0 ? false : argument._op === 'isTop';
20+
const isTop = argument % 1 === 0 ? _isTop : argument._op === 'isTop';
2121
for (let i = this.length - 1; i >= 0; i--) {
2222
if (this[i].index === index) {
2323
this.splice(i, 1);

src/modules/viewport.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -182,9 +182,9 @@ export default function Viewport(elementRoutines, buffer, element, viewportContr
182182
bottomPadding.cache.clear();
183183
},
184184

185-
removeCacheItem(item) {
186-
topPadding.cache.remove(item);
187-
bottomPadding.cache.remove(item);
185+
removeCacheItem(item, isTop) {
186+
topPadding.cache.remove(item, isTop);
187+
bottomPadding.cache.remove(item, isTop);
188188
},
189189

190190
removeItem(item) {

0 commit comments

Comments
 (0)