From 9f23d16b37e3b61bcc37133d40e67f938e4f8eec Mon Sep 17 00:00:00 2001
From: llj <lingjun.li1@gmail.com>
Date: Fri, 20 May 2016 10:56:39 +0800
Subject: [PATCH] [side nav] fix

---
 static/scripts/app/views/side-nav.js | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/static/scripts/app/views/side-nav.js b/static/scripts/app/views/side-nav.js
index a37c12688e..ea768152cd 100644
--- a/static/scripts/app/views/side-nav.js
+++ b/static/scripts/app/views/side-nav.js
@@ -60,8 +60,10 @@ define([
 
             if ($icon.hasClass('icon-caret-down')) {
                 Cookie.set('group_expanded', 'true');
+                this.data.show_group_list = true;
             } else {
                 Cookie.set('group_expanded', 'false');
+                this.data.show_group_list = false;
             }
             return false;
         },
@@ -118,14 +120,12 @@ define([
             if (options) {
                 $.extend(this.data, options);
             }
-            this.data.show_group_list = $('#group-nav .grp-list:visible').length ? true : false;
 
             if (this.$clickedTab) {
                 // The user click a link and this.$clickedTab is set by visitLink()
                 this.$('.tab-cur').removeClass('tab-cur');
                 this.$clickedTab.addClass('tab-cur');
                 this.$clickedTab = null;
-                return;
             } else {
                 // the first time the side nav is rendered or the side nav is re-rendered
                 // when dismiss a group, leave a group