From 7591f0155b4965b40894539d36e5981361328a93 Mon Sep 17 00:00:00 2001
From: Max Ogden <max@maxogden.com>
Date: Wed, 13 Jul 2011 20:45:48 -0700
Subject: [PATCH] better form UX

---
 attachments/pages/index.html | 43 +++++++++++++++++++-----------------
 attachments/script/site.js   |  7 ++++++
 attachments/script/util.js   | 10 ---------
 3 files changed, 30 insertions(+), 30 deletions(-)

diff --git a/attachments/pages/index.html b/attachments/pages/index.html
index 1bbbf333..e4755c06 100644
--- a/attachments/pages/index.html
+++ b/attachments/pages/index.html
@@ -144,26 +144,29 @@
   	</div>
   	<div class="dialog-body">
   		<div class="grid-layout layout-tight layout-full">
-        <table id="sign-in-form" class="form-table">
-          <tbody>
-            <tr>
-              <th>
-                <label for="username">Username</label>
-              </th>
-              <td>
-                <input type="text" size="25" id="username-input" name="username">
-              </td>
-            </tr>
-            <tr>
-              <th>
-                <label for="password">Password</label>
-              </th>
-              <td>
-                <input type="password" size="25" id="password-input" name="password">
-              </td>
-            </tr>
-          </tbody>
-        </table>
+        <form name="sign-in-form" id="sign-in-form">
+          <table class="form-table">
+            <tbody>
+              <tr>
+                <th>
+                  <label for="username">Username</label>
+                </th>
+                <td>
+                  <input type="text" size="25" id="username-input" name="username">
+                </td>
+              </tr>
+              <tr>
+                <th>
+                  <label for="password">Password</label>
+                </th>
+                <td>
+                  <input type="password" size="25" id="password-input" name="password">
+                </td>
+              </tr>
+              <input type="submit" style="height: 0px; width: 0px; border: none; padding: 0px;" hidefocus="true" />
+            </tbody>
+          </table>
+        </form>
   		</div>
   	</div>
   	<div class="dialog-footer">
diff --git a/attachments/script/site.js b/attachments/script/site.js
index 30e6cd74..ffdbd74c 100644
--- a/attachments/script/site.js
+++ b/attachments/script/site.js
@@ -83,6 +83,11 @@ app.after = {
   exportActions: removalist.handleMenuClick,
   columnActions: removalist.handleMenuClick,
   signIn: function() {
+    $('.dialog-content #username-input').focus();
+    $('.dialog-content').find('#sign-in-form').submit(function(e) {
+      $('.dialog-content .okButton').click();
+      return false;
+    })
     $('.dialog-content .cancelButton').click(function(e) {
       util.hide('dialog');
     })
@@ -97,6 +102,8 @@ app.after = {
       couch.login(credentials).then(function(response) {
         util.notify("Signed in");
         util.render('controls', 'project-controls', {text: "Sign out"});
+      }, function(error) {
+        if (error.statusText === "error") util.notify(JSON.parse(error.responseText).reason);
       })
     })
   },
diff --git a/attachments/script/util.js b/attachments/script/util.js
index c8713896..b3dc9be4 100644
--- a/attachments/script/util.js
+++ b/attachments/script/util.js
@@ -123,16 +123,6 @@ var util = function() {
       $('.persist').each(function(i, el) {
         localStorage.removeItem($(el).attr('id'));
       })
-    },
-    init: function() {
-      if (Modernizr.localstorage) {
-        util.persist.restore();
-
-        $('.persist').keyup(function(e) {
-          var inputId = $(e.target).attr('id');
-          util.persist.save(inputId);
-        })
-      }
     }
   }