fixed nested sub menu issue

impressivewebs · Jun 28, 2012 · df00685 · df00685
1 parent 6c85997
commit df00685
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 14 deletions.
diff --git a/css/styles.css b/css/styles.css
@@ -48,11 +48,11 @@ a:hover, a:active, a:focus {
 }
 
 .toc-holder li li a:link, .toc-holder li li a:visited {
-	padding-left: 50px;
+	padding-left: 60px;
 }
 
 .toc-holder li li li a:link, .toc-holder li li li a:visited {
-	padding-left: 100px;
+	padding-left: 90px;
 }
 
 .toc-holder li a:hover {

diff --git a/js/jquery.fixedTOC.js b/js/jquery.fixedTOC.js
@@ -57,7 +57,7 @@
                             top: '-' + s.tocHeight + 'px'
                         }, s.menuSpeed, function () {
                             base.$el.addClass(s.tocUpClass);
-                            $('.toc-sub').slideUp(0);
+                            s.tocSub.slideUp(0);
                         });
 
                     }
@@ -69,17 +69,10 @@
 
             doOpenItem : function () {
 
-                $('.toc-h1>a').on('click', function () {
+                $('.toc-h1>a, .toc-sub a').on('click', function () {
 
-                    s.tocSub.each(function () {
+                    $(this).parent().siblings('.toc-h1').find('.toc-sub').stop().slideUp().addClass('closed');
-
+                    $(this).next('.toc-sub').stop().slideToggle().toggleClass('closed');
-                        if (!$(this).hasClass('closed')) {
-                            $(this).stop().slideUp().addClass('closed');
-                        }
-
-                    });
-
-                    $(this).parent().find('.toc-sub').stop().slideToggle().removeClass('closed');
 
                     return false;
 
@@ -168,7 +161,7 @@
         topLink          : $('#top-link'),
         currHash         : null,
         // customizable settings
-        menuOpens        : 'click',
+		menuOpens		 : 'click',
         scrollSpeed      : 1000,
         menuSpeed        : 300,
         useSubMenus      : true,