From 308919a3050316d1323a52169dc526b485c79009 Mon Sep 17 00:00:00 2001
From: Javier <javier@bitpay.com>
Date: Fri, 29 Jan 2016 15:21:42 -0300
Subject: [PATCH] scroll up when error occurs

---
 public/views/create.html         |  6 +++---
 public/views/walletHome.html     |  2 +-
 src/js/controllers/create.js     |  8 +++++++-
 src/js/controllers/walletHome.js | 13 +++++++++----
 4 files changed, 20 insertions(+), 9 deletions(-)

diff --git a/public/views/create.html b/public/views/create.html
index 98d3e88a8..adaef7f9a 100644
--- a/public/views/create.html
+++ b/public/views/create.html
@@ -49,13 +49,13 @@
   </div>
 
   <form name="setupForm" ng-submit="create.create(setupForm)" novalidate>
-    <div class="box-notification m20b" ng-show="create.error">
+    <div class="box-notification m20b" id="notification" ng-show="create.error">
       <span class="text-warning">
         {{create.error|translate}}
       </span>
     </div>
     <div class="row">
-      <div class="large-12 columns"> 
+      <div class="large-12 columns">
 
         <div ng-hide="create.hideWalletName">
           <label><span translate>Wallet name</span>
@@ -169,7 +169,7 @@
                 <span>Testnet</span>
                 <switch id="network-name" name="isTestnet" ng-model="isTestnet" class="green right m5t m10b"></switch>
               </label>
-            </div> 
+            </div>
           </div> <!-- columns -->
         </div> <!-- advanced -->
 
diff --git a/public/views/walletHome.html b/public/views/walletHome.html
index 77b5e2037..e34fec099 100644
--- a/public/views/walletHome.html
+++ b/public/views/walletHome.html
@@ -395,7 +395,7 @@
           <qr-scanner on-scan="home.onQrCodeScanned(data)"></qr-scanner>
         </div>
       </div>
-      <div class="box-notification m20t" ng-show="home.error" ng-click="home.resetError()">
+      <div class="box-notification m20t" id="notification" ng-show="home.error" ng-click="home.resetError()">
         <span class="text-warning">
           {{home.error|translate}}
         </span>
diff --git a/src/js/controllers/create.js b/src/js/controllers/create.js
index 1433ccdb9..48a3c815a 100644
--- a/src/js/controllers/create.js
+++ b/src/js/controllers/create.js
@@ -1,7 +1,7 @@
 'use strict';
 
 angular.module('copayApp.controllers').controller('createController',
-  function($scope, $rootScope, $location, $timeout, $log, lodash, go, profileService, configService, isCordova, gettext, ledger, trezor, isMobile, isChromeApp, isDevel, derivationPathHelper) {
+  function($scope, $location, $anchorScroll, $rootScope, $timeout, $log, lodash, go, profileService, configService, isCordova, gettext, ledger, trezor, isMobile, isChromeApp, isDevel, derivationPathHelper) {
 
     var self = this;
     var defaults = configService.getDefaults();
@@ -164,6 +164,7 @@ angular.module('copayApp.controllers').controller('createController',
           if (err) {
             $log.warn(err);
             self.error = err;
+            scrollUp();
             $timeout(function() {
               $rootScope.$apply();
             });
@@ -174,6 +175,11 @@ angular.module('copayApp.controllers').controller('createController',
       }, 100);
     }
 
+    function scrollUp(){
+      $location.hash('notification');
+      $anchorScroll();
+    };
+
     this.formFocus = function(what) {
       if (!this.isWindowsPhoneApp) return
 
diff --git a/src/js/controllers/walletHome.js b/src/js/controllers/walletHome.js
index 737849652..49ae1d1d8 100644
--- a/src/js/controllers/walletHome.js
+++ b/src/js/controllers/walletHome.js
@@ -1,6 +1,6 @@
 'use strict';
 
-angular.module('copayApp.controllers').controller('walletHomeController', function($scope, $rootScope, $timeout, $filter, $modal, $log, notification, txStatus, isCordova, isMobile, profileService, lodash, configService, rateService, storageService, bitcore, isChromeApp, gettext, gettextCatalog, nodeWebkit, addressService, ledger, bwsError, confirmDialog, txFormatService, animationService, addressbookService, go, feeService, txSignService) {
+angular.module('copayApp.controllers').controller('walletHomeController', function($scope, $location, $anchorScroll, $rootScope, $timeout, $filter, $modal, $log, notification, txStatus, isCordova, isMobile, profileService, lodash, configService, rateService, storageService, bitcore, isChromeApp, gettext, gettextCatalog, nodeWebkit, addressService, ledger, bwsError, confirmDialog, txFormatService, animationService, addressbookService, go, feeService, txSignService) {
 
   var self = this;
   window.ignoreMobilePause = false;
@@ -97,6 +97,11 @@ angular.module('copayApp.controllers').controller('walletHomeController', functi
     $rootScope.$digest();
   });
 
+  this.scrollUp = function(){
+    $location.hash('notification');
+    $anchorScroll();
+  };
+
   var accept_msg = gettextCatalog.getString('Accept');
   var cancel_msg = gettextCatalog.getString('Cancel');
   var confirm_msg = gettextCatalog.getString('Confirm');
@@ -596,7 +601,7 @@ angular.module('copayApp.controllers').controller('walletHomeController', functi
     });
   };
 
-  // Send 
+  // Send
 
   this.canShowAlternative = function() {
     return $scope.showAlternative;
@@ -739,7 +744,7 @@ angular.module('copayApp.controllers').controller('walletHomeController', functi
       fc.credentials.m > 1 ? gettextCatalog.getString('Could not create payment proposal') : gettextCatalog.getString('Could not send payment');
 
     this.error = bwsError.msg(err, prefix);
-
+    this.scrollUp();
     $timeout(function() {
       $scope.$digest();
     }, 1);
@@ -1076,7 +1081,7 @@ angular.module('copayApp.controllers').controller('walletHomeController', functi
     }
   };
 
-  // History 
+  // History
 
   function strip(number) {
     return (parseFloat(number.toPrecision(12)));