From a0cdd7cb0ed3caaefb7540aacd3c40f6c8eb08ef Mon Sep 17 00:00:00 2001
From: Raphael <raphael@joindiaspora.com>
Date: Sat, 3 Jul 2010 13:14:29 -0700
Subject: [PATCH] RS; Refactored publisher, tested it, it seems to work
 perfectly, but I could be wrong.

---
 public/javascripts/publisher.js | 36 ++++++++++-----------------------
 1 file changed, 11 insertions(+), 25 deletions(-)

diff --git a/public/javascripts/publisher.js b/public/javascripts/publisher.js
index dd5094aee..d3fad10c9 100644
--- a/public/javascripts/publisher.js
+++ b/public/javascripts/publisher.js
@@ -1,33 +1,19 @@
   $(document).ready( function() {
 
-    $("#publisher_content_pickers .status_message").click(function(){
-      if( $("#new_status_message").css("display") == "none" ) {
+    $("#publisher_content_pickers .status_message").click(selectPublisherTab);
+
+    $("#publisher_content_pickers .bookmark").click(selectPublisherTab);
+
+    $("#publisher_content_pickers  .blog").click(selectPublisherTab);
+
+    function selectPublisherTab(){
+      var form_id = "#new_" + this.className
+      if( $(form_id).css("display") == "none" ) {
         $("#publisher_content_pickers").children("li").removeClass("selected");
         $("#publisher_form form").fadeOut(50);
 
         $(this).toggleClass("selected");
-        $("#new_status_message").delay(50).fadeIn(200);
+        $(form_id).delay(50).fadeIn(200);
       }
-    });
-
-    $("#publisher_content_pickers .bookmark").click(function(){
-      if( $("#new_bookmark").css("display") == "none" ) {
-        $("#publisher_content_pickers").children("li").removeClass("selected");
-        $("#publisher_form form").fadeOut(50);
-
-        $(this).toggleClass("selected");
-        $("#new_bookmark").delay(50).fadeIn(200);
-      }
-    });
-
-    $("#publisher_content_pickers  .blog").click(function(){
-      if( $("#new_blog").css("display") == "none" ) {
-        $("#publisher_content_pickers").children("li").removeClass("selected");
-        $("#publisher_form form").fadeOut(50);
-
-        $(this).toggleClass("selected");
-        $("#new_blog").delay(50).fadeIn(200);
-      }
-    });
-
+    }
   });