ikiwiki (3.20130711) unstable; urgency=low
[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 >>> Sure, go ahead --[[Joey]]
35
36 >>>>Here is an updated patch against current svn. --Fred
37
38         Index: templates/recentchanges.tmpl
39         ===================================================================
40         --- templates/recentchanges.tmpl        (révision 3575)
41         +++ templates/recentchanges.tmpl        (copie de travail)
42         @@ -12,7 +12,11 @@
43          </TMPL_IF>
44          </head>
45          <body>
46         +<div id="banner">
47         +</div>
48          
49         +<div id="content-wrap">
50         +
51          <div class="header">
52          <span>
53          <TMPL_VAR INDEXLINK>/ <TMPL_VAR TITLE>
54         @@ -65,5 +69,10 @@
55          
56          <!-- from <TMPL_VAR NAME=WIKINAME> -->
57          
58         +</div>
59         +
60         +<div id="realfooter">
61         +</div>
62         +
63          </body>
64          </html>
65         Index: templates/page.tmpl
66         ===================================================================
67         --- templates/page.tmpl (révision 3575)
68         +++ templates/page.tmpl (copie de travail)
69         @@ -13,7 +13,11 @@
70          <TMPL_IF NAME="META"><TMPL_VAR META></TMPL_IF>
71          </head>
72          <body>
73         +<div id="banner">
74         +</div>
75          
76         +<div id="content-wrap">
77         +
78          <div class="header">
79          <span>
80          <TMPL_LOOP NAME="PARENTLINKS">
81         @@ -95,5 +99,10 @@
82          <TMPL_IF EXTRAFOOTER><TMPL_VAR EXTRAFOOTER></TMPL_IF>
83          </div>
84          
85         +</div>
86         +
87         +<div id="realfooter">
88         +</div>
89         +
90          </body>
91          </html>
92         Index: templates/editpage.tmpl
93         ===================================================================
94         --- templates/editpage.tmpl     (révision 3575)
95         +++ templates/editpage.tmpl     (copie de travail)
96         @@ -12,6 +12,11 @@
97          </TMPL_IF>
98          </head>
99          <body>
100         +<div id="banner">
101         +</div>
102         +
103         +<div id="content-wrap">
104         +
105          <TMPL_IF NAME="PAGE_CONFLICT">
106          <p>
107          <b>Your changes conflict with other changes made to the page.</b>
108         @@ -86,5 +91,11 @@
109          <TMPL_VAR PAGE_PREVIEW>
110          </div>
111          </TMPL_IF>
112         +
113         +</div>
114         +
115         +<div id="realfooter">
116         +</div>
117         +
118          </body>
119          </html>
120         Index: templates/misc.tmpl
121         ===================================================================
122         --- templates/misc.tmpl (révision 3575)
123         +++ templates/misc.tmpl (copie de travail)
124         @@ -12,7 +12,11 @@
125          </TMPL_IF>
126          </head>
127          <body>
128         +<div id="banner">
129         +</div>
130          
131         +<div id="content-wrap">
132         +
133          <div class="header">
134          <span>
135          <TMPL_VAR INDEXLINK>/ <TMPL_VAR TITLE>
136         @@ -23,5 +27,10 @@
137          <TMPL_VAR PAGEBODY>
138          </div>
139          
140         +</div>
141         +
142         +<div id="realfooter">
143         +</div>
144         +
145          </body>
146          </html>
147
148 > I took a more intrusive approach to avoid ugly names like "realfooter".
149 > [[done]] --[[Joey]]