fix(loader): Disable loader by default

Set visibility of loader to false by default and remove unnecessary loader code. re #286
sensebox · Nov 26, 2018 · b56092b · b56092b
1 parent 5483a4b
commit b56092b
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 8 deletions.
diff --git a/app/components/loader.directive.js b/app/components/loader.directive.js
@@ -45,7 +45,7 @@
     }
 
     function activate () {
-      vm.visible = true;
+      vm.visible = false;
       vm.messageText = 'Fetching boxes and measurements';
     }
 

diff --git a/app/scripts/controllers/account.dashboard.js b/app/scripts/controllers/account.dashboard.js
@@ -5,9 +5,9 @@
     .module('openSenseMapApp')
     .controller('AccountDashboardController', AccountDashboardController);
 
-  AccountDashboardController.$inject = ['AccountService', 'LocalStorageService', '$scope', '$rootScope'];
+  AccountDashboardController.$inject = ['AccountService', 'LocalStorageService', '$scope'];
 
-  function AccountDashboardController (AccountService, LocalStorageService, $scope, $rootScope) {
+  function AccountDashboardController (AccountService, LocalStorageService, $scope) {
     var vm = this;
     var localStorageKey = 'osem.dashboard.listStyle';
     var localStorageOrderByKey = 'osem.dashboard.orderBy';
@@ -21,8 +21,6 @@
     ////
 
     function activate () {
-      $rootScope.$broadcast('osemLoaderVisibility', { visible: false });
-
       var listStyleFromLocalStorage = LocalStorageService.getValue(localStorageKey);
       if (listStyleFromLocalStorage) {
         vm.listStyle = listStyleFromLocalStorage;

diff --git a/app/scripts/controllers/info.js b/app/scripts/controllers/info.js
@@ -5,8 +5,8 @@
     .module('openSenseMapApp')
     .controller('InfoController', InfoController);
 
-  InfoController.$inject = ['$rootScope', '$state', 'app'];
-  function InfoController ($rootScope, $state, app) {
+  InfoController.$inject = ['$state', 'app'];
+  function InfoController ($state, app) {
     var vm = this;
     vm.app = app;
     vm.tabActive = {
@@ -22,7 +22,6 @@
     ////////////////
 
     function activate () {
-      $rootScope.$broadcast('osemLoaderVisibility', { visible: false });
       changeActiveTab($state.current.url.split('/')[1]);
     }