]> git.vanrenterghem.biz Git - git.ikiwiki.info.git/blob - doc/todo/beef_up_sidebar_to_allow_for_multiple_sidebars.mdwn
bugfixes
[git.ikiwiki.info.git] / doc / todo / beef_up_sidebar_to_allow_for_multiple_sidebars.mdwn
1 Maybe sidebar could be beefed up to take the name of a sidebar, such that I could use multiple sidebars in the same wiki. For instance, the default name would be 'sidebar', meaning the plugin looks for `sidebar.pm` and fills in the `sidebar` slot, but I might also want a footer in `footer.pm`, filling the template's `footer` slot.
3 One good way (if possible) would be to provide a directive like `\[[!sidebar
4 id=sidebar]]` which would cause the file, in which it occurred to fill the
5 slot `SIDEBAR` in the template: basically, a page `foo.mdwn` says
6 `\[[!fillslot slot=myslot]]` and then its contents should go into `<TMPL_VAR
7 SLOT_MYSLOT>` for all pages. Ideally, this can then be overridden, so if
8 `/bar/foo.mdwn` also references `myslot` then pages under `/bar` should get
9 those contents instead.
12 --[[madduck]]
14 > In mine I just copied sidebar out and made some extra "sidebars", but they go elsewhere. Ugly hack, but it works. --[[simonraven]]
16 >> Here a simple [[patch]] for multiple sidebars. Not too fancy but better than having multiple copies of the sidebar plugin. --[[jeanprivat]]
18 >>> I made a [[git]] branch for it [[!template id=gitbranch branch="privat/multiple_sidebars" author="[[jeanprivat]]"]] --[[jeanprivat]]
20 <pre>
21 --- /usr/share/perl5/IkiWiki/Plugin/sidebar.pm  2010-02-11 22:53:17.000000000 -0500
22 +++ plugins/IkiWiki/Plugin/sidebar.pm   2010-02-27 09:54:12.524412391 -0500
23 @@ -19,12 +19,20 @@
24                         safe => 1,
25                         rebuild => 1,
26                 },
27 +               active_sidebars => {
28 +                       type => "string",
29 +                       example => qw(sidebar banner footer),
30 +                       description => "Which sidebars must be activated and processed.",
31 +                       safe => 1,
32 +                       rebuild => 1
33 +               },
34  }
35  
36 -sub sidebar_content ($) {
37 +sub sidebar_content ($$) {
38         my $page=shift;
39 +       my $sidebar=shift;
40         
41 -       my $sidebar_page=bestlink($page, "sidebar") || return;
42 +       my $sidebar_page=bestlink($page, $sidebar) || return;
43         my $sidebar_file=$pagesources{$sidebar_page} || return;
44         my $sidebar_type=pagetype($sidebar_file);
45         
46 @@ -49,11 +57,17 @@
47  
48         my $page=$params{page};
49         my $template=$params{template};
50 -       
51 -       if ($template->query(name => "sidebar")) {
52 -               my $content=sidebar_content($page);
53 -               if (defined $content && length $content) {
54 -                       $template->param(sidebar => $content);
55 +
56 +       my @sidebars;
57 +       if (defined $config{active_sidebars} && length $config{active_sidebars}) { @sidebars = @{$config{active_sidebars}}; }
58 +       else { @sidebars = qw(sidebar); }
59 +
60 +       foreach my $sidebar (@sidebars) {
61 +               if ($template->query(name => $sidebar)) {
62 +                       my $content=sidebar_content($page, $sidebar);
63 +                       if (defined $content && length $content) {
64 +                               $template->param($sidebar => $content);
65 +                       }
66                 }
67         }
68  }
69 </pre>
71 [[!tag wishlist]]