From 8a3044a7700041596b8d147084d86f427529b016 Mon Sep 17 00:00:00 2001
From: Huan Du <i@huandu.me>
Date: Tue, 4 Mar 2014 19:00:25 +0800
Subject: [PATCH] Hide sidebar if #sidebar has no children

---
 javascripts/theme.js | 16 +++++++++++-----
 1 file changed, 11 insertions(+), 5 deletions(-)

diff --git a/javascripts/theme.js b/javascripts/theme.js
index 1a2db0a..b98f911 100644
--- a/javascripts/theme.js
+++ b/javascripts/theme.js
@@ -11,15 +11,21 @@
       "autofocus.time_entry": true
     };
 
-    var sidebar_btn = $('<div id="sidebar_btn">&nbsp;</div>'),
+    var sidebar_btn = $('<div id="sidebar_btn">&nbsp;</div>'),                                                                                                                    
         elem = $('#main:not(.nosidebar) #sidebar');
 
     sidebar_btn.on('click', toggle_sidebar);
 
-    if (elem != undefined){
-      elem.before(sidebar_btn);
-      if ($.cookie('hide_sidebar') == 'yes'){
-        $('#main').toggleClass('nosidebar');
+    if (elem.length){
+      // show sidebar if it has something.
+      if ($("#sidebar").children().length) {
+          elem.before(sidebar_btn);
+          if ($.cookie('hide_sidebar') == 'yes'){
+            $('#main').toggleClass('nosidebar');
+          }   
+      } else {
+          // if #sidebar is empty, hide it completely.
+          $("#main").toggleClass("nosidebar");
       }
     }