summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAnonymous <anonymous@none>2010-10-11 11:45:38 +0200
committeranonymous <anonymous@none>2010-10-11 11:46:29 +0200
commit2ef899d9f94768d714b81ff5bc157e5f2c521395 (patch)
treeb097f1d87e76e827e6fc3dcebf8960295f3ca6e4
parent2c0d2799b752c81c4fe45d569d1931157ef947ec (diff)
Bugreport
-rw-r--r--doc/bugs/class_parameter_of_img_directive_behave_not_as_documented.mdwn29
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.