Is extending the original renderer needed?

[Phillaf]

The ListRenderer mentions

Extend original to fix UL attributes

I believe this is a mistake. Knp uses <li> as the current node and <ul> as children nodes. Which make sense when you render a list like this

<li> <!-- current node -->

    <a href="#">This is a menu Item</a>

    <ul> <!-- children node -->

        <li><a hrtef="#">Submenu1</a></li>
        <li><a hrtef="#">Submenu2</a></li>

    </ul> 
</li>

When rendering a full menu, the topmost element is a <ul>, and therefore uses "children node" attributes.

However, by overriding the render() method and using getAttributes() both on <ul> and <li> tags, we're making setChildrenAttribute() pointless and we limit the flexibility provided in the default renderer.