Fixing a divide-by-zero error

CHANGELOG.md

@@ -1,3 +1,8 @@
+Version 1.1.8 (2017/09/15)
+=================
+
+  * Fixed a divide-by-zero issue that could prevent items from rendering
+
 Version 1.1.7 (2016/03/08)
 =================
 

Gruntfile.js

@@ -13,7 +13,7 @@ module.exports = function(grunt){
 					banner: [
 						'//',
 						'// Copyright Kamil Pękala http://github.com/kamilkp',
-						'// Angular Virtual Scroll Repeat v1.1.7 2016/03/08',
+						'// Angular Virtual Scroll Repeat v1.1.8 2017/09/15',
 						'//',
 						''
 					].join('\n')
@@ -47,4 +47,4 @@ module.exports = function(grunt){
 	grunt.registerTask('test', 'Run unit tests', ['jshint', 'min', 'karma:unit']);
 	grunt.registerTask('default', ['test']);
 	grunt.registerTask('travis', ['jshint', 'min', 'karma:travis']);
-};
+};

README.md

@@ -1,4 +1,4 @@
-angular-vs-repeat v1.1.7
+angular-vs-repeat v1.1.8
 =================
 
 Looking for a version for Angular 2?

bower.json

@@ -1,6 +1,6 @@
 {
   "name": "angular-vs-repeat",
-  "version": "1.1.7",
+  "version": "1.1.8",
   "description": "Virtual Scroll for AngularJS ngRepeat directive",
   "homepage": "http://kamilkp.github.io/angular-vs-repeat",
   "main": "src/angular-vs-repeat.js",

package.json

@@ -1,6 +1,6 @@
 {
   "name": "angular-vs-repeat",
-  "version": "1.1.7",
+  "version": "1.1.8",
   "description": "Virtual Scroll for AngularJS ngRepeat directive",
   "homepage": "http://kamilkp.github.io/angular-vs-repeat",
   "author": {
@@ -26,7 +26,10 @@
     "karma-firefox-launcher": "~0.1.3"
   },
   "scripts": {
-    "test": "grunt travis"
+    "test": "grunt travis --force"
   },
-  "dependencies": {}
+  "dependencies": {
+    "grunt-cli": "^1.2.0",
+    "travis": "^0.1.1"
+  }
 }

src/angular-vs-repeat.js

@@ -1,6 +1,6 @@
 //
 // Copyright Kamil Pękala http://github.com/kamilkp
-// Angular Virtual Scroll Repeat v1.1.7 2016/03/08
+// Angular Virtual Scroll Repeat v1.1.8 2017/09/15
 //
 
 (function(window, angular) {
@@ -512,7 +512,7 @@
                                     originalLength
                                 );
                             }
-                            else {
+                            else if ($scope.elementSize > 0) {
                                 __startIndex = Math.max(
                                     Math.floor(
                                         ($scrollPosition - $scope.offsetBefore - scrollOffset) / $scope.elementSize
@@ -527,6 +527,12 @@
                                     originalLength
                                 );
                             }
+                            else {
+                                // Don't try to divide by zero, just default to the size of the
+                                // array
+                                __startIndex = 0;
+                                __endIndex = originalLength;
+                            }
 
                             _minStartIndex = Math.min(__startIndex, _minStartIndex);
                             _maxEndIndex = Math.max(__endIndex, _maxEndIndex);