From a3ecd32264cf7ada8aa3bd04acd798e6f0ff8a90 Mon Sep 17 00:00:00 2001
From: Dennis Collinson <dennis.collective@gmail.com>
Date: Wed, 7 Mar 2012 17:47:34 -0800
Subject: [PATCH] DG DC Change dropdown text on select

---
 public/javascripts/app/forms/post_form.js     |  2 +-
 .../app/templates/aspects-dropdown.handlebars |  4 +++-
 .../app/views/aspects_dropdown_view.js        | 14 +++++++++--
 .../app/views/aspects_dropdown_view_spec.js   | 23 +++++++++++++++++--
 4 files changed, 37 insertions(+), 6 deletions(-)

diff --git a/public/javascripts/app/forms/post_form.js b/public/javascripts/app/forms/post_form.js
index 1f2a1fc56..adfde5cfb 100644
--- a/public/javascripts/app/forms/post_form.js
+++ b/public/javascripts/app/forms/post_form.js
@@ -6,7 +6,7 @@ app.forms.Post = app.forms.Base.extend({
   },
 
   formAttrs : {
-    ".text" : "text",
+    "textarea.text" : "text",
     "input.aspect_ids" : "aspect_ids"
   },
 
diff --git a/public/javascripts/app/templates/aspects-dropdown.handlebars b/public/javascripts/app/templates/aspects-dropdown.handlebars
index 14ed9dff6..1596933e3 100644
--- a/public/javascripts/app/templates/aspects-dropdown.handlebars
+++ b/public/javascripts/app/templates/aspects-dropdown.handlebars
@@ -1,6 +1,8 @@
 
 <div class="btn-group aspects_dropdown check-group">
-    <a class="btn btn-info dropdown-toggle" data-toggle="dropdown" href="#">All Aspects <span class="caret"></span></a>
+    <a class="btn btn-info dropdown-toggle" data-toggle="dropdown" href="#">
+        <span class="text"></span> <span class="caret"></span>
+    </a>
     <ul class="dropdown-menu">
         <li><a href="#" class="public" data-visibility="public">Public</a></li>
         <li><a href="#" class="all-aspects" data-visibility="all-aspects">All Aspects</a></li>
diff --git a/public/javascripts/app/views/aspects_dropdown_view.js b/public/javascripts/app/views/aspects_dropdown_view.js
index 67af22f53..263d2ab28 100644
--- a/public/javascripts/app/views/aspects_dropdown_view.js
+++ b/public/javascripts/app/views/aspects_dropdown_view.js
@@ -4,24 +4,34 @@ app.views.AspectsDropdown = app.views.Base.extend({
     "click .dropdown-menu a" : "setVisibility"
   },
 
+  postRenderTemplate : function(){
+    this.setVisibility({target : this.$("a[data-visibility='all-aspects']").first()})
+  },
+
   setVisibility : function(evt){
-    var linkVisibility = $(evt.target).data("visibility")
+    var link = $(evt.target)
       , visibilityCallbacks = {
           'public' : setPublic,
           'all-aspects' : setPrivate
         }
 
-    visibilityCallbacks[linkVisibility].call(this)
+    visibilityCallbacks[link.data("visibility") || "all-aspects"].call(this)
 
     function setPublic (){
       this.setAspectIds("public")
+      this.setDropdownText(link.text())
     }
 
     function setPrivate (){
       this.setAspectIds("all_aspects")
+      this.setDropdownText(link.text())
     }
   },
 
+  setDropdownText : function(text){
+    $.trim(this.$(".dropdown-toggle .text").text(text))
+  },
+
   setAspectIds : function(val){
     this.$("input.aspect_ids").val(val)
   }
diff --git a/spec/javascripts/app/views/aspects_dropdown_view_spec.js b/spec/javascripts/app/views/aspects_dropdown_view_spec.js
index 9baca853a..e1e4d2aaf 100644
--- a/spec/javascripts/app/views/aspects_dropdown_view_spec.js
+++ b/spec/javascripts/app/views/aspects_dropdown_view_spec.js
@@ -8,18 +8,37 @@ describe("app.views.AspectsDropdown", function(){
       this.view.render()
     })
 
+    it("defaults to All Aspects Visibility", function(){
+      expect(this.view.$("input.aspect_ids").val()).toBe("all_aspects")
+      expect($.trim(this.view.$(".dropdown-toggle .text").text())).toBe("All Aspects")
+    })
+
     describe("selecting Public", function(){
-      it("calls set aspect_ids to 'public'", function(){
+      beforeEach(function(){
         this.view.$("a[data-visibility='public']").click()
+      })
+
+      it("calls set aspect_ids to 'public'", function(){
         expect(this.view.$("input.aspect_ids").val()).toBe("public")
       })
+
+      it("sets the dropdown title to 'public'", function(){
+        expect(this.view.$(".dropdown-toggle .text").text()).toBe("Public")
+      })
     })
 
     describe("selecting All Aspects", function(){
-      it("calls set aspect_ids to 'all'", function(){
+      beforeEach(function(){
         this.view.$("a[data-visibility='all-aspects']").click()
+      })
+
+      it("calls set aspect_ids to 'all'", function(){
         expect(this.view.$("input.aspect_ids").val()).toBe("all_aspects")
       })
+
+      it("sets the dropdown title to 'public'", function(){
+        expect($.trim(this.view.$(".dropdown-toggle .text").text())).toBe("All Aspects")
+      })
     })
   })
 })
\ No newline at end of file