refactor account

src/js/controller/account.js

@@ -12,6 +12,13 @@ define(function(require) {
     var AccountCtrl = function($scope) {
         emailDao = appController._emailDao;
 
+        $scope.state.account = {
+            open: false,
+            toggle: function(to) {
+                this.open = to;
+            }
+        };
+
         //
         // scope variables
         //

src/js/controller/navigation.js

@@ -43,13 +43,6 @@ define(function(require) {
             $scope.state.nav.toggle(false);
         };
 
-        $scope.openAccount = function() {
-            $scope.accountOpen = true;
-        };
-        $scope.closeAccount = function() {
-            $scope.accountOpen = false;
-        };
-
         //
         // Outbox checker
         //
@@ -216,8 +209,6 @@ define(function(require) {
     ngModule.directive('keyShortcuts', function() {
         return function(scope, elm) {
             elm.bind('keydown', function(e) {
-                var cs = scope.$$childTail;
-
                 // global state is not yet set, ignore keybaord shortcuts
                 if (!scope.state) {
                     return;
@@ -238,10 +229,10 @@ define(function(require) {
                     e.preventDefault();
                     scope.state.writer.close();
 
-                } else if (e.keyCode === 27 && cs.accountOpen) {
+                } else if (e.keyCode === 27 && scope.state.account.open) {
                     // escape -> close account view
                     e.preventDefault();
-                    cs.closeAccount();
+                    scope.state.account.toggle(false);
 
                 } else if (e.keyCode === 83 && scope.state.writer && !scope.state.writer.open && scope.state.mailList.synchronize) {
                     // s -> sync folder

src/tpl/account.html

@@ -1,7 +1,7 @@
 <div class="lightbox-body" ng-controller="AccountCtrl">
   <header>
     <h2>Account</h2>
-    <button class="close" ng-click="closeAccount()" data-action="lightbox-close">&#xe007;</button>
+    <button class="close" ng-click="state.account.toggle(false)" data-action="lightbox-close">&#xe007;</button>
   </header>
 
   <div class="content">

src/tpl/desktop.html

@@ -22,7 +22,7 @@
 <div class="lightbox-overlay" ng-class="{'show': state.writer.open}">
   <div class="lightbox lightbox-effect" ng-include="'tpl/write.html'"></div>
 </div>
-<div class="lightbox-overlay" ng-class="{'show': accountOpen}">
+<div class="lightbox-overlay" ng-class="{'show': state.account.open}">
   <div class="lightbox lightbox-effect" ng-include="'tpl/account.html'"></div>
 </div>
 <div class="lightbox-overlay" ng-class="{'show': state.dialog.open}">

src/tpl/navigation.html

@@ -13,7 +13,7 @@
   </ul>
 
   <ul class="nav-secondary">
-    <li><a href="#" ng-click="openAccount(); $event.preventDefault()">Account</a></li>
+    <li><a href="#" ng-click="state.account.toggle(true); $event.preventDefault()">Account</a></li>
     <li><a href="#">About whiteout.io</a></li>
     <li><a href="#">Help</a></li>
   </ul>