plone/Products.CMFPlone — pull request #36 by danjacka (2012-11-25 08:50:27+00:00)
Title: Fix sitemap rendering

This pull request renders the sitemap using a browser view page template, rather than `portlet_navtree_macro.pt` from the `plone_deprecated` skin.

`plone_deprecated` is inactive in Plone 4.3, so currently (4.3a2) the sitemap is broken (https://dev.plone.org/ticket/13178)

I have taken the opportunity to improve sitemap rendering by:
- recursing over the content tree in Python, not in the template;
- removing logic associated with topLevel and bottomLevel, which do not affect the sitemap (see https://github.com/plone/Products.CMFPlone/commit/b2f55ebbea90d2227e93c3e79418249069fb8283);
- making the template match the navigation portlet template as far as possible.

The first commit reproduces the 4.2 sitemap markup the new way; the second commit changes the markup, mainly to add content type icons. Adding content type icons was my original reason for tinkering with the sitemap ... (see https://github.com/plone/Products.CMFPlone/pull/25)