responses
[ikiwiki.git] / doc / patchqueue / more_class__61____34____34___for_css.mdwn
1 I'm writing my own CSS for ikiwiki. During this effort I often found the need of adding more class="" attributes to the default ikiwiki templates. This way more presentational aspects of visual formatting can be delegated to CSS and removed from the HTML structure.
2
3 In this patch I plan to collect changes in this direction.
4
5 The first, one-liner, patch is to use a "div" element with a
6 class="actions" attribute for inline page as is done with non-inlined page.
7 This way the same CSS formatting can be applied to div.actions in the CSS,
8 while at the moment it must be duplicated for a span.actions (which I
9 believe is also incorrect, since it will contain a "ul" element, not sure
10 though). In case the markup should be differentiated it will still be
11 possible relying on the fact that a div.actions is contained or not in a
12 div.inlinepage.
13
14 Here's the one-liner:
15
16 > applied --[[Joey]]
17
18 The following adds a div element with class="trailer" around the
19 meta-information added after an inlined page (namely: the post date, the
20 tags, and the actions):
21
22 The following adds a div element with class="trailer" around the meta-informati
23 on added after an inlined page (namely: the post date, the tags, and the actions):
24
25     --- inlinepage.tmpl.orig        2006-12-28 16:56:49.000000000 +0100
26     +++ inlinepage.tmpl     2006-12-28 17:02:06.000000000 +0100
27     @@ -17,6 +17,8 @@
28      </span>
29      <TMPL_VAR CONTENT>
30
31     +<div class="trailer">
32     +
33      <span class="pageinfo">
34      Posted <TMPL_VAR CTIME>
35      </span>
36     @@ -44,3 +46,5 @@
37      </TMPL_IF>
38
39      </div>
40     +
41     +</div>
42
43 > Unfortunately, the inlinepage content passes through markdown, and markdown
44 > gets confused by these nested div's and puts p's around one of them, generating
45 > broken html. If you can come up with a way to put in the div that passes
46 > the test suite, or a fix to markdown, I will accept it, but the above patch
47 > fails the test suite. --[[Joey]]