refactor(scrollController): remove unused dependency

also JSLint cleanup
ionic-team · Feb 25, 2015 · 59a0ce8 · 59a0ce8
1 parent 31fda2f
commit 59a0ce8
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/js/angular/controller/scrollController.js b/js/angular/controller/scrollController.js
@@ -12,16 +12,14 @@ IonicModule
   '$document',
   '$ionicScrollDelegate',
   '$ionicHistory',
-  '$controller',
 function($scope,
          scrollViewOptions,
          $timeout,
          $window,
          $location,
          $document,
          $ionicScrollDelegate,
-         $ionicHistory,
-         $controller) {
+         $ionicHistory) {
 
   var self = this;
   // for testing

diff --git a/js/angular/directive/collectionRepeat.js b/js/angular/directive/collectionRepeat.js
@@ -267,7 +267,7 @@ function CollectionRepeatDirective($ionicCollectionManager, $parse, $window, $$r
 
       var withoutQuotes = attrValue.replace(/(\'|\"|px|%)/g, '').trim();
       var isConstant = withoutQuotes.length && !/([a-zA-Z]|\$|:|\?)/.test(withoutQuotes);
-      dimensionData.attrValue = attrValue;;
+      dimensionData.attrValue = attrValue;
 
       // If it's a constant, it's either a percent or just a constant pixel number.
       if (isConstant) {
@@ -536,7 +536,7 @@ function RepeatManagerFactory($rootScope, $window, $$rAF) {
 
         item = itemsShownMap[i] || (itemsShownMap[i] = itemsLeaving.length ? itemsLeaving.pop() :
                                     itemsPool.length ? itemsPool.shift() :
-                                    new RepeatItem())
+                                    new RepeatItem());
         itemsEntering.push(item);
         item.isShown = true;