padding elements cache is synced with items removing

Author: dhilt

src/modules/padding.js

@@ -13,7 +13,23 @@ function Cache() {
         index: item.scope.$index,
         height: item.element.outerHeight()
       });
+      cache.sort((a, b) => ((a.index < b.index) ? -1 : ((a.index > b.index) ? 1 : 0)));
     },
+
+    remove(item) {
+      for (let i = cache.length - 1; i >= 0; i--) {
+        if (cache[i].index === item.scope.$index) {
+          cache.splice(i, 1);
+          break;
+        }
+      }
+      for (let i = cache.length - 1; i >= 0; i--) {
+        if (cache[i].index > item.scope.$index) {
+          cache[i].index--;
+        }
+      }
+    },
+
     clear() {
       cache.length = 0;
     }

src/modules/viewport.js

@@ -175,6 +175,12 @@ export default function Viewport(elementRoutines, buffer, element, viewportContr
     resetBottomPadding() {
       bottomPadding.height(0);
       bottomPadding.cache.clear();
+    },
+
+    removeItem(item) {
+      topPadding.cache.remove(item);
+      bottomPadding.cache.remove(item);
+      return buffer.remove(item);
     }
   });
 

src/ui-scroll.js

@@ -261,7 +261,7 @@ angular.module('ui.scroll', [])
             }
           });
 
-          toBeRemoved.forEach((wrapper) => promises = promises.concat(buffer.remove(wrapper)));
+          toBeRemoved.forEach((wrapper) => promises = promises.concat(viewport.removeItem(wrapper)));
 
           if (toBePrepended.length)
             toBePrepended.forEach((wrapper) => {