meh - already supported, but buggy!

[git.ikiwiki.info.git] / doc / todo / Allow_per-page_template_selection.mdwn

diff --git a/doc/todo/Allow_per-page_template_selection.mdwn b/doc/todo/Allow_per-page_template_selection.mdwn
index c915982f5ab36cab4d1525a7acb1b06d9bcd0ebb..231ccf502338ceb60997f8f6e960d50c5cb1cbca 100644 (file)
--- a/doc/todo/Allow_per-page_template_selection.mdwn
+++ b/doc/todo/Allow_per-page_template_selection.mdwn
@@ -11,14 +11,14 @@ Joey provided a nice suggestion for implementing this feature, ("I
 would probably add a hook that allowed overriding the default template
 constuction and returning a template object"). I did start looking
 into that, but finally I wimped out and just put the following hack
-into the genpage() function in Render.pm:
+into the `genpage()` function in Render.pm:
 
 
        if ($page eq 'index') {
                $template->param(suppresstitle => 1);
        }
 
-That lets me use a <TMPL_UNLESS SUPPRESSTITLE> in my template to get
+That lets me use a `<TMPL_UNLESS SUPPRESSTITLE>` in my template to get
 the effect I want. I don't think that's anything that upstream should
 pick-up as is, (maybe with an appropriate configuration option, but
 then again allowing for per-page template selection would be more
@@ -27,3 +27,17 @@ pursue implementing this feature further myself.
 
 But I'd still happily switch to using this feature if someone were to
 implement it.
+
+UPDATE: My latest hack is as follows:
+
+       if ($page eq 'index') {
+               $template->param(toplevelindex => 1);
+       }
+
+And that's something that I'm not actually embarrassed to suggest
+could be accepted upstream. Joey, what do you think? And would a
+proper patch be helpful (it really just adds those lines to the right
+place).
+
+> Fully implemented as a templatefile hook and a [[plugins/pagetemplate]]
+> plugin. --[[Joey]] [[done]]