Fix style-guide warnings

closes #23 #27
IonicBrazil · Jul 4, 2015 · 2dda1f7 · 2dda1f7
1 parent 4dfbe48
commit 2dda1f7
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 6 deletions.
diff --git a/src/shared/configure-status-bar.js b/src/shared/configure-status-bar.js
@@ -13,7 +13,7 @@ function configStatusBar($ionicPlatform) {
    */
   function platformReady() {
     if (window.StatusBar) {
-      StatusBar.styleDefault()
+      StatusBar.styleDefault();
     }
   }
 }
diff --git a/src/shared/ig-button-loading/ig-button-loading.controller.js b/src/shared/ig-button-loading/ig-button-loading.controller.js
@@ -9,32 +9,45 @@ var DEFAULT_LOADING_OPTIONS = {
   hideOnStateChange: false,
   delay: 0,
   duration: undefined
-}
+};
 
 /**
  * Controller Definition
  */
 function IGButtonLoadingController($scope, $state, $timeout, $ionicLoading) {
 
-  console.log($scope.ig);
-  $scope.ig = ionic.extend(DEFAULT_LOADING_OPTIONS, $scope.ig);
+  $scope.ig = $scope.ig;
   console.log($scope.ig);
 
-  $scope.$on('$destroy', function () {
+  $scope.$on('$destroy', destroy);
+
+  /**
+   * Scope Destroy
+   */
+  function destroy() {
     console.log('igLoading $destroy');
-  });
+  }
 
+  /**
+   * Show loading
+   */
   function show() {
     $ionicLoading.show(DEFAULT_LOADING_OPTIONS);
   }
 
+  /**
+   * Show loading with no backdrop option
+   */
   function showNoBackdrop() {
     $ionicLoading.show({
       template: $scope.title,
       noBackdrop: true
     });
   }
 
+  /**
+   * Hide loading
+   */
   function hideLoading() {
     $ionicLoading.show();
   }