fix(array-virtual-repeat-strategy): handle instance changes with less…

… items than in view slot Fixes #41
aurelia · Apr 7, 2016 · 4eb9ca0 · 4eb9ca0
1 parent cc11b8e
commit 4eb9ca0
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/array-virtual-repeat-strategy.js b/src/array-virtual-repeat-strategy.js
@@ -32,7 +32,7 @@ export class ArrayVirtualRepeatStrategy extends ArrayRepeatStrategy {
     let viewsLength = repeat.viewCount();
     let first = repeat._getIndexOfFirstView();
     // remove unneeded views.
-    while (viewsLength > repeat._viewsLength) {
+    while (viewsLength > itemsLength) {
       viewsLength--;
       repeat.removeView(viewsLength, true);
     }

diff --git a/src/virtual-repeat.js b/src/virtual-repeat.js
@@ -126,7 +126,7 @@ export class VirtualRepeat extends AbstractRepeater {
     }
     let items = this.items;
     this.strategy = this.strategyLocator.getStrategy(items);
-    if (items.length > 0) {
+    if (items.length > 0 && this.viewCount() === 0) {
       this.strategy.createFirstItem(this);
     }
     this._calcInitialHeights(items.length);