You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by mi...@apache.org on 2023/01/15 10:18:22 UTC

[maven-fluido-skin] branch master updated: Don't print CSS class for
  • if none has been provided

    • This is an automated email from the ASF dual-hosted git repository.

michaelo pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/maven-fluido-skin.git


The following commit(s) were added to refs/heads/master by this push:
     new dff9ce7  Don't print CSS class for <li> if none has been provided
dff9ce7 is described below

commit dff9ce70674c4ba9f51abcbf5812de7931a99d46
Author: Michael Osipov <mi...@apache.org>
AuthorDate: Sun Jan 15 11:18:11 2023 +0100

    Don't print CSS class for <li> if none has been provided
---
 src/it/sidebar/verify.groovy                     | 2 +-
 src/it/topbar/verify.groovy                      | 2 +-
 src/main/resources/META-INF/maven/site-macros.vm | 4 ++++
 3 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/src/it/sidebar/verify.groovy b/src/it/sidebar/verify.groovy
index 6d022e7..66defc8 100644
--- a/src/it/sidebar/verify.groovy
+++ b/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 --git a/src/it/topbar/verify.groovy b/src/it/topbar/verify.groovy
index 98248ac..9cef326 100644
--- a/src/it/topbar/verify.groovy
+++ b/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 --git a/src/main/resources/META-INF/maven/site-macros.vm b/src/main/resources/META-INF/maven/site-macros.vm
index 1f51f94..6c3635d 100644
--- a/src/main/resources/META-INF/maven/site-macros.vm
+++ b/src/main/resources/META-INF/maven/site-macros.vm
@@ -224,7 +224,11 @@
 #*    *##set ( $currentItemHref = './' )
 #*  *##end
 ##
+#*  *##if( $cssClass )
       <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>