From 5a84918099dc4ed2ce3918677a79d31a1b5ef08e Mon Sep 17 00:00:00 2001
From: Billiam <billiamthesecond@gmail.com>
Date: Sun, 10 Mar 2013 18:05:15 -0500
Subject: [PATCH] Fix repeated classes header in sidenav

Fixes multiple <h3>classes</h3> navigation menus appearing in navigation
sidebar when using modules.

Removes magic -1 value and module counting in favor of more explicit
boolean, only appending class heading if necessary.
---
 templates/default/publish.js | 21 ++++++++++++---------
 1 file changed, 12 insertions(+), 9 deletions(-)

diff --git a/templates/default/publish.js b/templates/default/publish.js
index 44fb9d20..3ea89bb7 100644
--- a/templates/default/publish.js
+++ b/templates/default/publish.js
@@ -164,7 +164,9 @@ function generateSourceFiles(sourceFiles) {
  */
 function buildNav(members) {
     var nav = '<h2><a href="index.html">Index</a></h2>',
-        seen = {};
+        seen = {},
+        hasClassList = false,
+        classNav = '';
 
     if (members.modules.length) {
         nav += '<h3>Modules</h3><ul>';
@@ -191,25 +193,26 @@ function buildNav(members) {
     }
 
     if (members.classes.length) {
-        var moduleClasses = 0;
+
         members.classes.forEach(function(c) {
             var moduleSameName = find({kind: 'module', longname: c.longname});
             if (moduleSameName.length) {
                 c.name = c.name.replace('module:', 'require("')+'")';
-                moduleClasses++;
                 moduleSameName[0].module = c;
             }
-            if (moduleClasses !== -1 && moduleClasses < members.classes.length) {
-                nav += '<h3>Classes</h3><ul>';
-                moduleClasses = -1;
-            }
+
             if ( !hasOwnProp.call(seen, c.longname) ) {
-                nav += '<li>'+linkto(c.longname, c.name)+'</li>';
+                hasClassList = true;
+                classNav += '<li>'+linkto(c.longname, c.name)+'</li>';
             }
             seen[c.longname] = true;
         });
         
-        nav += '</ul>';
+        if (hasClassList) {
+            nav += '<h3>Classes</h3><ul>';
+            nav += classNav;
+            nav += '</ul>';
+        }
     }
 
     if (members.events.length) {