web commit by Fred
[ikiwiki.git] / doc / bugs / Add_a_footer_div_on_all_pages_to_improve_theming.mdwn
1 The following patch adds a footer div on all pages to ease CSS themeing.
2 Indeed, the misc.tmpl, recentchanges.tmpl and editpage.tmpl templates lack such a div.
3
4 > So, the problem with this is that the default css inserts a horizontal
5 > line at the top of the footer div, and putting an empty footer on these
6 > other pages looks a bit weird. Any idea how to get around that?
7 > --[[Joey]]
8
9 >> Sorry I didn't see that. It definitely looks weird. We could add text  
10 >> in all footers or change the CSS stylesheet, but it's not clean IMHO.
11
12 >> The idea was about to ease themeing by giving all the pages the same
13 >> structure. The current structure is the following one:
14
15 >> div header - div actions ... div content - div footer (sometimes)
16
17 >> So we could add some new divs in all templates. By default, they will  
18 >> be empty and no CSS will be defined for them. This way ikiwiki
19 >> standard appearance is not changed but themeing will be eased.
20 >> The new page structure could be:
21
22 >> * div banner (to show up a logo for example)
23
24 >> * div content-wrap containing div header, div actions, ... div content
25 >> and div footer
26
27 >> * div realfooter (to put credits, "Powered by ikiwiki", "Valid XHTML"...)
28
29 >> From my tests, it works: Just adding the divs, without touching the stylesheet,
30 >> doesn't change ikiwiki appearance. And themeing is eased :-)
31
32 >> I can update the patch, if you want to follow and test this idea. --Fred
33
34 This patch is against current svn.
35
36         Index: templates/recentchanges.tmpl
37         ===================================================================
38         --- templates/recentchanges.tmpl        (révision 3543)
39         +++ templates/recentchanges.tmpl        (copie de travail)
40         @@ -64,6 +64,8 @@
41          </div>
42          
43          <!-- from <TMPL_VAR NAME=WIKINAME> -->
44         +<div id="footer">
45         +</div>
46          
47          </body>
48          </html>
49         Index: templates/editpage.tmpl
50         ===================================================================
51         --- templates/editpage.tmpl     (révision 3543)
52         +++ templates/editpage.tmpl     (copie de travail)
53         @@ -86,5 +86,7 @@
54          <TMPL_VAR PAGE_PREVIEW>
55          </div>
56          </TMPL_IF>
57         +<div id="footer">
58         +</div>
59          </body>
60          </html>
61         Index: templates/misc.tmpl
62         ===================================================================
63         --- templates/misc.tmpl (révision 3543)
64         +++ templates/misc.tmpl (copie de travail)
65         @@ -22,6 +22,8 @@
66         <div id="content">
67          <TMPL_VAR PAGEBODY>
68          </div>
69         +<div id="footer">
70         +</div>
71          
72          </body>
73          </html>