diff options
author | Anonymous <anonymous@none> | 2010-10-11 11:45:38 +0200 |
---|---|---|
committer | anonymous <anonymous@none> | 2010-10-11 11:46:29 +0200 |
commit | 2ef899d9f94768d714b81ff5bc157e5f2c521395 (patch) | |
tree | b097f1d87e76e827e6fc3dcebf8960295f3ca6e4 | |
parent | 2c0d2799b752c81c4fe45d569d1931157ef947ec (diff) |
Bugreport
-rw-r--r-- | doc/bugs/class_parameter_of_img_directive_behave_not_as_documented.mdwn | 29 |
1 files changed, 29 insertions, 0 deletions
diff --git a/doc/bugs/class_parameter_of_img_directive_behave_not_as_documented.mdwn b/doc/bugs/class_parameter_of_img_directive_behave_not_as_documented.mdwn new file mode 100644 index 000000000..10ba15066 --- /dev/null +++ b/doc/bugs/class_parameter_of_img_directive_behave_not_as_documented.mdwn @@ -0,0 +1,29 @@ +On [[ikiwiki/directive/img/]] I read that + +> You can also pass alt, title, class, align, id, hspace, and vspace +> parameters. These are passed through unchanged to the html img tag. + +but when I pass `class="myclass"` to an img directive, I obtain + + <img class="myclass img" ... + +I found that this behaviour was added in commit f6db10d: + +> img: Add a margin around images displayed by this directive. +> +> Particularly important for floating images, which could before be placed +> uncomfortably close to text. + +which adds to img.pm: + + if (exists $params{class}) { + $params{class}.=" img"; + } + else { + $params{class}="img"; + } + +I would prefer if the `img` class were only added if no class attribute is +passed. + +If you keep the current behaviour, please document it. |