Hi Tim,

Nice macro! I noticed that it generates a lot of empty class tags and class tags with extra trailing spaces though. If you dump the class string into a variable and then check if its there before adding the classes, you can eliminate the empty tags (and use the normalise-space function to kill the trailing spaces). Here's the code I used:

<xsl:variable name="itemClass">
  <xsl:if test="$isSelected = 1"><xsl:value-of select="concat($selectedClass,' ')"/></xsl:if>
  <xsl:if test="$isSelectedBranch = 1"><xsl:value-of select="concat($branchClass,' ')"/></xsl:if>
  <xsl:if test="$hasChildren = 1"><xsl:value-of select="'hasChildren '"/></xsl:if>
</xsl:variable>

<xsl:if test="string-length($itemClass) > 0"> 
  <xsl:attribute name="class">
    <xsl:value-of select="normalize-space($itemClass)"/>
  </xsl:attribute>
</xsl:if>

It doesn't seem to impact the performance from the tests I've run, and you get cleaner html output.