From 9c8bc0b8c0e94d0e4ff83893b958b4cf28d759e7 Mon Sep 17 00:00:00 2001
From: "Benjamin J. Balter" <ben@balter.com>
Date: Sat, 8 Dec 2012 12:28:52 -0500
Subject: [PATCH] Slightly more intuitive saving UI, tries to fix #240.

On save, move focus to the commit message dialog, to encouage users to enter message.

In order to make this process more fluid, the default commit message was moved to `placeholder`, and is swapped in for `val()` on save if none is present.

May make sense to have someone with more design/UI sense take a pass, perhaps the commit box border changes on focus, the commit button changes color, etc. to stand out?
---
 _includes/templates/post._ | 2 +-
 _includes/views/post.js    | 5 +++--
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/_includes/templates/post._ b/_includes/templates/post._
index d666cdeb7..627f32342 100644
--- a/_includes/templates/post._
+++ b/_includes/templates/post._
@@ -12,7 +12,7 @@
     </div>
 
     <div class="fr menu-item save-state">
-      <input type="text" class="commit-message" value="Updated."/>
+      <input type="text" class="commit-message" value=""/>
       <div class="state fl">
         <% if (writeable) { %><a class='toggle-options button' href='#'>&nbsp;</a><% } %>
         
diff --git a/_includes/views/post.js b/_includes/views/post.js
index 7c40d90ab..dd3311062 100644
--- a/_includes/views/post.js
+++ b/_includes/views/post.js
@@ -57,7 +57,7 @@ views.Post = Backbone.View.extend({
 
   _toggleCommit: function() {
     if (!this.$('.document-menu').hasClass('commit')) {
-      this.$('.commit-message').val("Updated "+$('input.filepath').val());  
+      this.$('.commit-message').attr( 'placeholder', "Updated "+$('input.filepath').val());
     }
 
     this.hideMeta();
@@ -69,6 +69,7 @@ views.Post = Backbone.View.extend({
     this.showDiff();
     this.$('.surface').toggle();
     this.$('.diff-wrapper').toggle();
+    this.$('.commit-message').focus();  
 
     return false;
   },
@@ -306,7 +307,7 @@ views.Post = Backbone.View.extend({
         filepath = $('input.filepath').val(),
         filename = _.extractFilename(filepath)[1],
         filecontent = this.serialize(),
-        message = this.$('.commit-message').val(),
+        message = this.$('.commit-message').val() || this.$('.commit-message').attr('placeholder'),
         method = this.model.writeable ? this.saveFile : this.sendPatch;
 
     // Update content