Don't print CSS class for <li> if none has been provided

Author: michael-o

Diff for: src/it/sidebar/verify.groovy

@@ -28,4 +28,4 @@ assert html.contains( '<li class="active"><a><span class="none"></span>About</a>
 assert html.contains( '<li><a href="summary.html" title="Summary"><span class="none"></span>Summary</a>' )
 assert html.contains( '<li><a href="plugins.html" title="Plugins"><span class="none"></span>Plugins</a></li>' )
 // breadcrumb
-assert html.contains( '<li class=""><a href="https://www.apache.org/" class="externalLink" title="Apache">Apache</a><span class="divider">/</span></li>' )
+assert html.contains( '<li><a href="https://www.apache.org/" class="externalLink" title="Apache">Apache</a><span class="divider">/</span></li>' )

Diff for: src/it/topbar/verify.groovy

@@ -28,4 +28,4 @@ assert html.contains( '<li><a title="About">About</a></li>' )
 assert html.contains( '<li><a href="summary.html" title="Summary">Summary</a></li>' )
 assert html.contains( '<li><a href="plugins.html" title="Plugins">Plugins</a></li>' )
 // breadcrumb
-assert html.contains( '<li class=""><a href="https://www.apache.org/" class="externalLink" title="Apache">Apache</a><span class="divider">/</span></li>' )
+assert html.contains( '<li><a href="https://www.apache.org/" class="externalLink" title="Apache">Apache</a><span class="divider">/</span></li>' )

Diff for: src/main/resources/META-INF/maven/site-macros.vm

@@ -224,7 +224,11 @@
 #*    *##set ( $currentItemHref = './' )
 #*  *##end
 ##
+#*  *##if( !$cssClass.isEmpty() )
       <li class="$cssClass">##
+#*  *##else
+      <li>##
+#end
 #*  *##link( $currentItemHref $item.name $item.target $item.img $item.position $item.alt $item.border $item.width $item.height false )
 #*  *#<span class="divider">$breadcrumbDivider</span>##
 #*  *#</li>