Use jQuery chaining
authorJohn Firebaugh <john.firebaugh@gmail.com>
Thu, 29 Nov 2012 15:33:04 +0000 (07:33 -0800)
committerTom Hughes <tom@compton.nu>
Thu, 29 Nov 2012 18:29:50 +0000 (18:29 +0000)
app/assets/javascripts/sidebar.js

index 00ec299a1f74f859f1da33d8fd2705387e08463a..912197b9f2c380cfcc4732cf7033901d5b8987bf 100644 (file)
@@ -5,18 +5,12 @@ function openSidebar(options) {
 
   if (options.title) { $("#sidebar_title").html(options.title); }
 
-  if (options.width) { $("#sidebar").width(options.width); }
-  else { $("#sidebar").width("30%"); }
-
-  $("#sidebar").css("display", "block");
-
-  $("#sidebar").trigger("opened");
-};
+  $("#sidebar").width(options.width || "30%");
+  $("#sidebar").css("display", "block").trigger("opened");
+}
 
 function closeSidebar() {
-  $("#sidebar").css("display", "none");
-
-  $("#sidebar").trigger("closed");
+  $("#sidebar").css("display", "none").trigger("closed");
 }
 
 $(document).ready(function () {