It looks like this suggestion was put into the current code, but we still get empty ul if there are docs that have umbracoNaviHide set, so I changed the test to this:
<xsl:if test="$parentNode/*[@isDoc][string(umbracoNaviHide) != '1'] or ($calculatedMenuDepth = 1 and $forceHome)">
Thanks Claire! This should really be included. For me it messed up a lot more than just the semantics. The "isBranch" variable also needs this condition (if you use it). Like this:
Don't render empty ul
What
Don't render empty ul
Why
Solution
Proposed solution:
surround ul element with conditional
<xsl:if test="$parentNode/node or $calculatedMednuDepth = 1 and $forceHome">
<ul>
...
Means:
Only render ul-menu if there are children ;
with exception if forceHome is selected and it's the 1'st level (the actual home link)
It looks like this suggestion was put into the current code, but we still get empty ul if there are docs that have umbracoNaviHide set, so I changed the test to this:
Thanks Claire! This should really be included. For me it messed up a lot more than just the semantics. The "isBranch" variable also needs this condition (if you use it). Like this:
<xsl:variable name="isBranch"> <xsl:choose> <xsl:when test="$currentPage/ancestor-or-self::*[@isDoc][@id = $currentNodeID]/child::*[@isDoc][string(umbracoNaviHide) != '1'] and string(count(descendant::umbracoNaviHide[text() = '0'])) != '0'">1</xsl:when> </xsl:choose> </xsl:variable>is working on a reply...