From e18009b0dd9f6825b474acfc61f1b42605d44bfe Mon Sep 17 00:00:00 2001
From: Alexis Lahouze <xals@lahouze.org>
Date: Wed, 15 Jul 2015 18:28:56 +0200
Subject: [PATCH] Cleanup account frontend.

---
 accountant/frontend/static/js/accounts.js     |  8 ++---
 .../frontend/static/templates/accounts.html   | 30 ++++++++++++++++++-
 2 files changed, 33 insertions(+), 5 deletions(-)

diff --git a/accountant/frontend/static/js/accounts.js b/accountant/frontend/static/js/accounts.js
index 537419f..06e2829 100644
--- a/accountant/frontend/static/js/accounts.js
+++ b/accountant/frontend/static/js/accounts.js
@@ -29,8 +29,8 @@ accountantApp
 
 .controller(
     "AccountController", [
-        "$scope", "$rootScope", "$window", "$routeParams", "Accounts",
-        function($scope, $rootScope, $window, $routeParams, Accounts) {
+        "$scope", "$rootScope", "$routeParams", "Accounts",
+        function($scope, $rootScope, $routeParams, Accounts) {
 
     $scope.accountClass = function(account) {
       if(account == $scope.selectedAccount) {
@@ -56,7 +56,7 @@ accountantApp
     $scope.createAccount = function(account) {
       account.$save(function(account) {
         // Reset new account.
-        $scope.resetAccount(account);
+        $scope.cancelEditAccount(account);
 
         $scope.$emit("accountCreatedEvent", account); 
       });
@@ -84,7 +84,7 @@ accountantApp
      * Start account edition.
      */
     $scope.editAccount = function(account) {
-      account.editing = true;
+      account.edit = true;
     };
 
     /*
diff --git a/accountant/frontend/static/templates/accounts.html b/accountant/frontend/static/templates/accounts.html
index b74ea7a..18193ee 100644
--- a/accountant/frontend/static/templates/accounts.html
+++ b/accountant/frontend/static/templates/accounts.html
@@ -60,7 +60,7 @@
         </td>
       </tr>
 
-      <tr id="account_[[account.id]]" class="form-inline" ng-repeat="account in accounts">
+      <tr id="account_[[account.id]]" class="form-inline" ng-if="!account.edit" ng-repeat-start="account in accounts">
         <td><a href="account/[[ account.id ]]/entries">[[ account.name ]]</a></td>
         <td>[[ account.current ]]</td>
         <td>[[ account.pointed ]]</td>
@@ -84,6 +84,34 @@
           </div>
         </td>
       </tr>
+
+      <tr id="account_[[account.id]]" class="form-inline" ng-if="account.edit" ng-repeat-end>
+        <td>
+          <input type="text" class="form-control" ng-model="account.name" />
+        </td>
+
+        <td>[[ account.current ]]</td>
+        <td>[[ account.pointed ]]</td>
+
+        <td>
+          <input type="text" class="form-control"
+            ng-model="account.authorized_overdraft" />
+        </td>
+
+        <td>
+          <div class="btn-group">
+            <button class="btn btn-xs btn-success"
+              ng-click="saveAccount(account)">
+              <span class="fa fa-floppy-o"></span>
+            </button>
+
+            <button class="btn btn-xs btn-default"
+              ng-click="cancelEditAccount(account)">
+              <span class="fa fa-times"></span>
+            </button>
+          </div>
+        </td>
+      </tr>
     </tbody>
   </table>
 </div>