Refactor out function to call digest only when safe

Alex--wu · Nov 15, 2013 · 7745223 · 7745223
1 parent c364028
commit 7745223
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/src/directives/ng-viewport.js b/src/directives/ng-viewport.js
@@ -3,6 +3,11 @@ ngGridDirectives.directive('ngViewport', [function() {
         var isMouseWheelActive;
         var prevScollLeft;
         var prevScollTop = 0;
+        var ensureDigest = function() {
+            if (!$scope.$root.$$phase) {
+                $scope.$digest();
+            }
+        };
         elm.bind('scroll', function(evt) {
             var scrollLeft = evt.target.scrollLeft,
                 scrollTop = evt.target.scrollTop;
@@ -11,9 +16,7 @@ ngGridDirectives.directive('ngViewport', [function() {
             }
             $scope.adjustScrollLeft(scrollLeft);
             $scope.adjustScrollTop(scrollTop);
-            if (!$scope.$root.$$phase) {
-                $scope.$digest();
-            }
+            ensureDigest();
             prevScollLeft = scrollLeft;
             prevScollTop = scrollTop;
             isMouseWheelActive = false;