You are right, I already saw that html "bug" when the patch that added the style parameter was added... anyway, I can't find a good reason why the simpleNavigationTag should support writing a container tag at all... can't you simply do: <div class="whatever"> <cmsu:simpleNavigation startLevel="1" endLevel="1"> </div>
I would like to fix it in one of the following ways: -1 deprecating the style attribute at all, planning to remove it in future releases -2 moving the class added by the parameter to the root ul (the output will be [ul class="level1 simple-nav"]. This could be a bit more useful but the parameter name remains a bit misleading Moving the discussion to the developer list for feedback from other developers. fabrizio On 9/27/05, Eric Everman <[email protected]> wrote: > Hi- > > I am using the simpleNavigation tag like this: > > <cmsu:simpleNavigation startLevel="1" endLevel="1" style="simple-nav" /> > > and it results is this HTML (with angle braces turned to square > braces to prevent email mess-ups): > > [span class="simple-nav"] > [ul class="level1"] > [li class="leaf"][a href="/author/Home/About.html">Child > Page </a></li><li class="leaf"><a href="/author/Home/Birds.html"] > Birds of a Feath[/a][/li] > ... > [/ul] > [/span] > > Its that [span] tag that is the problem: A UL tag cannot be nested > in a SPAN tag - thats not valid html. Could that be updated to > output a [div] tag instead? Also, that 'style' name is a bit > misleading - how about 'class' or 'class-attrib' instead? > > Thanks Again, > > Eric Everman > > ---------------------------------------------------------------- > for list details see > http://www.magnolia.info/en/magnolia/developer.html > ---------------------------------------------------------------- > ---------------------------------------------------------------- for list details see http://www.magnolia.info/en/magnolia/developer.html ----------------------------------------------------------------
