Fix placeholder update

tinydesk · May 19, 2016 · 781443a · 781443a
1 parent b8ebaba
commit 781443a
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 17 deletions.
diff --git a/dist/tileview.js b/dist/tileview.js
@@ -169,14 +169,10 @@
                         }
                     }
                     function setPlaceholder() {
-                        var newHeightStart = Math.max(startRow * scope.options.tileSize[sizeDimension], 0);
-                        var newHeightEnd = Math.max((rowCount - endRow) * scope.options.tileSize[sizeDimension], 0);
-                        if (newHeightStart !== heightStart || newHeightEnd !== heightEnd) {
-                            placeholderStart.css(sizeDimension, newHeightStart + 'px');
-                            placeholderEnd.css(sizeDimension, newHeightEnd + 'px');
-                            heightStart = newHeightStart;
-                            heightEnd = newHeightEnd;
-                        }
+                        heightStart = Math.max(startRow * scope.options.tileSize[sizeDimension], 0);
+                        heightEnd = Math.max((rowCount - endRow) * scope.options.tileSize[sizeDimension], 0);
+                        placeholderStart.css(sizeDimension, heightStart + 'px');
+                        placeholderEnd.css(sizeDimension, heightEnd + 'px');
                     }
                     function createElements(diff) {
                         updateVisibleRows();

diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "angular-tileview",
-  "version": "0.5.7",
+  "version": "0.5.8",
   "description": "A tileview for angular",
   "main": "gulpfile.js",
   "scripts": {

diff --git a/src/tileview.ts b/src/tileview.ts
@@ -191,14 +191,10 @@ declare const angular: any;
           }
 
         function setPlaceholder() {
-          const newHeightStart = Math.max(startRow * scope.options.tileSize[sizeDimension], 0);
-          const newHeightEnd = Math.max((rowCount - endRow) * scope.options.tileSize[sizeDimension], 0);
-          if (newHeightStart !== heightStart || newHeightEnd !== heightEnd) {
-            placeholderStart.css(sizeDimension, newHeightStart + 'px');
-            placeholderEnd.css(sizeDimension, newHeightEnd + 'px');
-            heightStart = newHeightStart;
-            heightEnd = newHeightEnd;
-          }
+          heightStart = Math.max(startRow * scope.options.tileSize[sizeDimension], 0);
+          heightEnd = Math.max((rowCount - endRow) * scope.options.tileSize[sizeDimension], 0);
+          placeholderStart.css(sizeDimension, heightStart + 'px');
+          placeholderEnd.css(sizeDimension, heightEnd + 'px');
         }
 
         function createElements(diff) {