From e28be27bf3a265714859d826143e3178acff33b4 Mon Sep 17 00:00:00 2001
From: Matias Alejo Garcia <ematiu@gmail.com>
Date: Thu, 17 Apr 2014 12:37:10 -0300
Subject: [PATCH 1/4] better signin page

---
 index.html                      | 55 +++++++++++++++++++++------------
 js/controllers/signin.js        |  7 ++---
 js/models/core/WalletFactory.js |  1 -
 3 files changed, 37 insertions(+), 26 deletions(-)

diff --git a/index.html b/index.html
index 8ebc2f12c..f5d0c1db6 100644
--- a/index.html
+++ b/index.html
@@ -64,17 +64,6 @@
           Connecting to wallet...
         </div>
         <div ng-show="!loading">
-          <div class="row">
-            <div class="large-6 columns">
-              <h3>Open a Existing Wallet</h3>
-              <select class="form-control" ng-model="selectedWalletId" ng-options="walletId for walletId in listWalletIds()">
-              </select>
-            </div>
-            <div class="large-3 columns">
-              <button class="button primary expand round" type="button" ng-click="open(selectedWalletId)">Open</button>
-            </div>
-          </div>
-          <hr>
           <div class="row">
             <div class="large-6 columns">
               <h3>Join a Network Wallet</h3>
@@ -85,14 +74,35 @@
             </div>
           </div>
           <hr>
-          <div class="row">
-            <div class="large-6 columns">
-              <h3>Create a New Wallet</h3>
-            </div>
-            <div class="large-3 columns">
-              <button class="button secondary expand round" ng-click="create()">Create</button>
+          <div ng-show="walletIds.length === 0">
+            <div class="row">
+              <div class="large-6 columns">
+                <h3>Create a New Wallet</h3>
+              </div>
+              <div class="large-3 columns">
+                <button class="button secondary expand round" ng-click="create()">Create</button>
+              </div>
             </div>
           </div>
+          <div ng-show="walletIds.length>0">
+            <div class="row">
+              <div class="large-6 columns">
+                <h3>Open a Existing Wallet</h3>
+                <select class="form-control" ng-model="selectedWalletId" ng-options="walletId for walletId in walletIds">
+                </select>
+              </div>
+              <div class="large-3 columns">
+                <button class="button primary expand round" type="button" ng-click="open(selectedWalletId)">Open</button>
+              </div>
+            </div>
+            <div class="row">
+              <div class="text-center">
+                <a ng-click="create()">Or create a new wallet</a>
+              </div>
+            </div>
+          </div>
+          <hr>
+
         </div>
       </div>
     </script>
@@ -127,6 +137,11 @@
               </button>
             </div>
           </div>
+          <div class="row">
+            <div class="text-center">
+              <a href="#signin">Go back</a>
+            </div>
+          </div>
         </div>
       </div>
     </script>
@@ -152,11 +167,11 @@
           </ul>
         </div>
         <div class="large-6 columns p70l">
-          <h3 class="panel-title">Copayers ({{$root.wallet.network.connectedPeers.length}}/{{$root.wallet.publicKeyRing.requiredCopayers}})</h3>
+          <h3 class="panel-title">Online Copayers: {{$root.wallet.network.connectedPeers.length}}</h3>
           <ul class="no-bullet">
             <li class="panel" ng-repeat="copayer in $root.wallet.network.connectedPeers">
-              <span ng-if="copayer == $root.peerId"> You ({{$root.peerId}})<i class="fi-check size-24"></i></span>
-              <span ng-if="copayer != $root.peerId">{{copayer}}</span>
+              <span ng-if="copayer == $root.wallet.network.peerId"> You ({{copayer}})<i class="fi-check size-24"></i></span>
+              <span ng-if="copayer !== $root.wallet.network.peerId">{{copayer}}</span>
               <span>
                   <i class="fi-check size-16 panel-sign right p5h br100"></i>
               </span>
diff --git a/js/controllers/signin.js b/js/controllers/signin.js
index a636dbab2..2e571f328 100644
--- a/js/controllers/signin.js
+++ b/js/controllers/signin.js
@@ -9,11 +9,8 @@ angular.module('copay.signin').controller('SigninController',
 //    $rootScope.peerId = peerData ? peerData.peerId : null;
     $scope.loading = false;
 
-    $scope.selectedWalletId = false;
-
-    $scope.listWalletIds = function() {
-      return walletFactory.getWalletIds();
-    };
+    $scope.walletIds = walletFactory.getWalletIds();
+    $scope.selectedWalletId = $scope.walletIds?$scope.walletIds.shift():null;
 
     $scope.create = function() {
       $location.path('setup');
diff --git a/js/models/core/WalletFactory.js b/js/models/core/WalletFactory.js
index a871b4968..bd87ec1fd 100644
--- a/js/models/core/WalletFactory.js
+++ b/js/models/core/WalletFactory.js
@@ -156,7 +156,6 @@ WalletFactory.prototype.openRemote = function(peedId) {
   opts.totalCopayers = totalCopayers;
   var w   = new Wallet(opts);
   w.store();
-  this.addWalletId(w.id);
   return w;
 };
 

From a51ed6c66d270beb802b5481134fdb2f3847364c Mon Sep 17 00:00:00 2001
From: Matias Alejo Garcia <ematiu@gmail.com>
Date: Thu, 17 Apr 2014 12:58:54 -0300
Subject: [PATCH 2/4] fix walletID list

---
 index.html                      | 2 +-
 js/controllers/signin.js        | 3 ++-
 js/models/storage/LocalPlain.js | 6 +++++-
 3 files changed, 8 insertions(+), 3 deletions(-)

diff --git a/index.html b/index.html
index f5d0c1db6..e494bb438 100644
--- a/index.html
+++ b/index.html
@@ -74,7 +74,7 @@
             </div>
           </div>
           <hr>
-          <div ng-show="walletIds.length === 0">
+          <div ng-show="!walletIds.length">
             <div class="row">
               <div class="large-6 columns">
                 <h3>Create a New Wallet</h3>
diff --git a/js/controllers/signin.js b/js/controllers/signin.js
index 2e571f328..357cc7acc 100644
--- a/js/controllers/signin.js
+++ b/js/controllers/signin.js
@@ -10,7 +10,8 @@ angular.module('copay.signin').controller('SigninController',
     $scope.loading = false;
 
     $scope.walletIds = walletFactory.getWalletIds();
-    $scope.selectedWalletId = $scope.walletIds?$scope.walletIds.shift():null;
+
+    $scope.selectedWalletId = $scope.walletIds.length ? $scope.walletIds[0]:null;
 
     $scope.create = function() {
       $location.path('setup');
diff --git a/js/models/storage/LocalPlain.js b/js/models/storage/LocalPlain.js
index d32234899..d5434f91b 100644
--- a/js/models/storage/LocalPlain.js
+++ b/js/models/storage/LocalPlain.js
@@ -54,13 +54,17 @@ Storage.prototype.remove = function(walletId, k) {
 
 Storage.prototype.getWalletIds = function() {
   var walletIds = [];
+  var uniq = {};
 
   for (var i = 0; i < localStorage.length; i++) {
     var key = localStorage.key(i);
     var split = key.split('::');
     if (split.length == 2) {
       var walletId = split[0];
-      walletIds.push(walletId);
+      if (typeof uniq[walletId] === 'undefined' ) {
+        walletIds.push(walletId);
+        uniq[walletId] = 1;
+      }
     }
   } 
 

From 9733305aeaf081a217b45320e03ea5068ff50bd2 Mon Sep 17 00:00:00 2001
From: Matias Alejo Garcia <ematiu@gmail.com>
Date: Thu, 17 Apr 2014 13:43:34 -0300
Subject: [PATCH 3/4] ux fixes

---
 index.html | 37 +++++++++++++++++++++++++++----------
 1 file changed, 27 insertions(+), 10 deletions(-)

diff --git a/index.html b/index.html
index e494bb438..2ccedad3e 100644
--- a/index.html
+++ b/index.html
@@ -51,6 +51,17 @@
       <a href="#" ng-click="clearFlashMessage()" class="button tiny">Dismiss</a>
     </div>
 
+
+    <div ng-if='$root.wallet && !$root.wallet.publicKeyRing.isComplete() && !loading' class="panel callout radius" >
+
+      <p class="panel panel-sign"> <i class="fi-alert size-40"></i> 
+        NOTE: Your wallet is not complete yet. 
+    {{$root.wallet.publicKeyRing.totalCopayers - $root.wallet.publicKeyRing.registeredCopayers() }} keys are missing. Ask your copayers to join your session: <b>{{$root.wallet.network.peerId}}</b>
+      
+      </p>
+    </div>
+
+
   </div>
 
   <div class="row">
@@ -158,16 +169,14 @@
               <p class="panel panel-sign"> <i class="fi-page-copy size-22"></i> </p>
             </div>
           </div>
-          <ul class="no-bullet" style="font-size: 0.7rem;">
-            <li> [DEBUG] Pubkeys that you have: {{$root.wallet.publicKeyRing.registeredCopayers()}} 
-            <li> [DEBUG] WalletId: {{$root.wallet.id}} 
-            <li class="panel" style="word-wrap: break-word;" ng-repeat="pub in $root.wallet.publicKeyRing.copayersBIP32">
-                ${{pub.extendedPublicKeyString()}}
-            </li>
-          </ul>
         </div>
         <div class="large-6 columns p70l">
           <h3 class="panel-title">Online Copayers: {{$root.wallet.network.connectedPeers.length}}</h3>
+
+          <p class="panel panel-sign" ng-show="$root.wallet.publicKeyRing.requiredCopayers > $root.wallet.network.connectedPeers.length"> <i class="fi-alert size-28"></i> 
+          {{$root.wallet.publicKeyRing.requiredCopayers}} copayers needed for signning transactions
+
+
           <ul class="no-bullet">
             <li class="panel" ng-repeat="copayer in $root.wallet.network.connectedPeers">
               <span ng-if="copayer == $root.wallet.network.peerId"> You ({{copayer}})<i class="fi-check size-24"></i></span>
@@ -177,9 +186,6 @@
               </span>
             </li>
           </ul>
-          <div class="text-center">
-            <button href="#/home" class="button secondary round right">Go to home</button>
-          </div>
         </div>
       </div>
     </script>
@@ -324,6 +330,17 @@
         </div>
       </div>
     </script>
+    <div ng-show="$root.wallet"  style="margin-top:200px">
+      <hr>
+      <p> 
+        [DEBUG] WalletId: {{$root.wallet.id}} 
+        <ul class="no-bullet">
+          <li class="panel" style="word-wrap: break-word;" ng-repeat="pub in $root.wallet.publicKeyRing.copayersBIP32">
+          ${{pub.extendedPublicKeyString()}}
+        </ul>
+    </div>
+
+
 <!-- / <div class="large-4 columns box-backup">Backup to Dropbox</div>
           // <div class="large-4 columns box-backup">Backup to email</div> -->
     <!-- NOT FOUND -->

From fc7354e93a3a996f3da025dd0a9dad78602d3997 Mon Sep 17 00:00:00 2001
From: Matias Alejo Garcia <ematiu@gmail.com>
Date: Thu, 17 Apr 2014 14:28:50 -0300
Subject: [PATCH 4/4] import option (not working yet)

---
 index.html | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/index.html b/index.html
index 2ccedad3e..d10b58a89 100644
--- a/index.html
+++ b/index.html
@@ -84,6 +84,7 @@
               <button class="button primary expand round" type="button" ng-click="join(connectionId)">Join</button>
             </div>
           </div>
+
           <hr>
           <div ng-show="!walletIds.length">
             <div class="row">
@@ -94,6 +95,12 @@
                 <button class="button secondary expand round" ng-click="create()">Create</button>
               </div>
             </div>
+            <hr>
+            <div class="row">
+              <div class="text-center">
+                <a ng-click="import()">Import from file</a>
+              </div>
+            </div>
           </div>
           <div ng-show="walletIds.length>0">
             <div class="row">
@@ -108,7 +115,8 @@
             </div>
             <div class="row">
               <div class="text-center">
-                <a ng-click="create()">Or create a new wallet</a>
+                <a ng-click="create()">Create a new wallet</a>
+                &middot; <a ng-click="import()">Import from file</a>
               </div>
             </div>
           </div>