Zim

Code review comment for lp:~fenryxo/zim/templates-pagemenu

Revision history for this message
Jaap Karssenberg (jaap.karssenberg) wrote :

Did some rework and afraid that changed how the options work. But it is more in line now with the index page that can be generated, so I propose keeping it like this. If you really want the style you had before maybe we can add a "sparse" option ?

* collapse now mimics the standard index, so it shows all of the toplevel pages and pages in any expanded sub page
* hide_empty now only hides pages that don't have content OR children, this is needed because otherwise namspaces were the parent has no text disappear.

« Back to merge proposal