po: let's walk the last steps needed before merge!

[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
 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);
        }
 
 
 
        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
 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.
 
 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]]